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 <pes...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: osmith <osm...@sysmocom.de>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
