fixeria has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/32640 )
Change subject: cosmetic: e1inp_ipa_bts_rsl_close_n(): cache ipa_client_conn ......................................................................
cosmetic: e1inp_ipa_bts_rsl_close_n(): cache ipa_client_conn
Change-Id: I9bdb4b1778d702c97890149a9f98590ae8ec54be --- M src/input/ipaccess.c 1 file changed, 14 insertions(+), 3 deletions(-)
Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved
diff --git a/src/input/ipaccess.c b/src/input/ipaccess.c index 8a52591..114389e 100644 --- a/src/input/ipaccess.c +++ b/src/input/ipaccess.c @@ -1206,6 +1206,7 @@ /* Close the underlying IPA TCP socket of an RSL link */ int e1inp_ipa_bts_rsl_close_n(struct e1inp_line *line, uint8_t trx_nr) { + struct ipa_client_conn *conn; struct ipaccess_line *il;
if (E1INP_SIGN_RSL+trx_nr-1 >= NUM_E1_TS) { @@ -1218,9 +1219,10 @@ if (!il) return 0; /* Nothing to do, no lines created */
- if (il->ipa_cli[1 + trx_nr]) { - ipa_client_conn_close(il->ipa_cli[1 + trx_nr]); - ipa_client_conn_destroy(il->ipa_cli[1 + trx_nr]); + conn = il->ipa_cli[1 + trx_nr]; + if (conn != NULL) { + ipa_client_conn_close(conn); + ipa_client_conn_destroy(conn); il->ipa_cli[1 + trx_nr] = NULL; } return 0;