bankd_client_fsm: close IPA client connection before re-establishing it

ipa_client_conn_destroy() really only destroys the object, but not close
the underlying file descriptor.  This leads to old connections
lingering around, which in turn (in case of the remsim client) will
occupy banksim worker threads.

Change-Id: If87f4bbc133e4dc812fa96a75e8495bad65275aa
This commit is contained in:
Harald Welte 2019-11-07 13:00:39 +01:00
parent da1854c2d7
commit 13ebf436ac
1 changed files with 1 additions and 0 deletions

View File

@ -152,6 +152,7 @@ static void bdc_st_reestablish_onenter(struct osmo_fsm_inst *fi, uint32_t prev_s
/* re-create bankd_conn */
if (bc->bankd_conn) {
LOGPFSML(fi, LOGL_INFO, "Destroying existing connection to bankd\n");
ipa_client_conn_close(bc->bankd_conn);
ipa_client_conn_destroy(bc->bankd_conn);
bc->bankd_conn = NULL;
}