pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-uecups/+/40793?usp=email ) ( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: cups_client: Log conn lost before tear down subprocesses ...................................................................... cups_client: Log conn lost before tear down subprocesses Change-Id: I32b56fddc32b95cf15c30b2c9cfc272a8678857b --- M daemon/cups_client.c 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: osmith: Looks good to me, approved laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/daemon/cups_client.c b/daemon/cups_client.c index b72ebd7..868480c 100644 --- a/daemon/cups_client.c +++ b/daemon/cups_client.c @@ -617,14 +617,14 @@ struct gtp_daemon *d = cc->d; struct subprocess *p, *p2; + LOGCC(cc, LOGL_INFO, "UECUPS connection lost\n"); + /* kill + forget about all subprocesses of this client */ /* We need no locking here as the subprocess list is only used from the main thread */ llist_for_each_entry_safe(p, p2, &d->subprocesses, list) { if (p->cups_client == cc) subprocess_destroy(p, SIGKILL); } - - LOGCC(cc, LOGL_INFO, "UECUPS connection lost\n"); llist_del(&cc->list); return 0; } -- To view, visit https://gerrit.osmocom.org/c/osmo-uecups/+/40793?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-uecups Gerrit-Branch: master Gerrit-Change-Id: I32b56fddc32b95cf15c30b2c9cfc272a8678857b Gerrit-Change-Number: 40793 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>