Skip to content
Success

#27000 (Feb 19, 2026, 5:35:49 PM)

Started 2 hr 43 min ago
Took 2 sec on build4-deb12build-ansible

Started by upstream project gerrit-osmo-remsim build number 108
originally caused by:

This run spent:

  • 8.7 sec waiting;
  • 2 sec build duration;
  • 10 sec total from scheduled to completion.
Revision: 00dcb6b47ca67653edf2279a914954403e53b819
Repository: $GERRIT_REPO_URL
  • master
rspro_server: fix releasing timed out clients

Fix crash on reconnects of clients if they timed out.

The duplicate check will use conn->peer of the old peer,
but conn->peer is NULL because rspro_client_conn_destroy() never destroys
the connection if conn->peer is valid when rspro_client_conn_destroy() is called.

void rspro_client_conn_destroy(conn)
{
if (conn->peer) {
peer = conn->peer;
conn->peer = NULL;
osmo_stream_srv_destroy(peer); /* calls sock_closed_cb()
}
[..]
}

int sock_closed_cb(peer)
{
[..]
if (conn->peer) {
osmo_fsm_inst_dispatch(conn->fi, CLNTC_E_TCP_DOWN, NULL); /* calls in the end rspro_client_conn_destroy() */
}
return 0;
}

Re-organize the clean up:
* rspro_client_conn_destroy() will be only called by the FSM clean up
* closed_cb will inform the fi to clean up

Fixes: 8cfe1d808a57 ("Use new osmo_ipa_ka_fsm_inst APIs from libosmo-netif")
Related: OS#6957
Change-Id: I1f7faf5ffdd909362c492ab434b63fa7e79ada95
lynxis at