pespin has uploaded this change for review.

View Change

pcap-server: Avoid closing cli conn if writing to disk fails

Related: SYS#7374
Change-Id: I06b20a17fae064a74e2acdc4549df400ad9c4930
---
M src/osmo_server_core.c
1 file changed, 3 insertions(+), 2 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-pcap refs/changes/72/39772/1
diff --git a/src/osmo_server_core.c b/src/osmo_server_core.c
index 9629423..cebc70a 100644
--- a/src/osmo_server_core.c
+++ b/src/osmo_server_core.c
@@ -547,8 +547,9 @@
rc = osmo_pcap_wr_file_write_msgb(conn->wrf, msg);
if (rc < 0) {
LOGP(DSERVER, LOGL_ERROR, "%s: Failed writing to file\n", conn->name);
- /* msgb will be freed by caller */
- return -1;
+ msgb_free(msg);
+ /* Avoid closing conn. msgb won't be freed by caller */
+ return 0;
}
update_last_write(conn, now);
/* msgb is now owned by conn->wrf. */

To view, visit change 39772. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newchange
Gerrit-Project: osmo-pcap
Gerrit-Branch: master
Gerrit-Change-Id: I06b20a17fae064a74e2acdc4549df400ad9c4930
Gerrit-Change-Number: 39772
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>