pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcap/+/39190?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: pcap-server: Use osmo_talloc_replace_string() to replace base-path ...................................................................... pcap-server: Use osmo_talloc_replace_string() to replace base-path Change-Id: I9ba1b7202d3e6a06cfd6efd6ed773b53007a99c4 --- M src/osmo_server_vty.c 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/src/osmo_server_vty.c b/src/osmo_server_vty.c index 3191cda..6a44da6 100644 --- a/src/osmo_server_vty.c +++ b/src/osmo_server_vty.c @@ -149,8 +149,7 @@ "base-path PATH", "Base path for log files\n" "Path\n") { - talloc_free(pcap_server->base_path); - pcap_server->base_path = talloc_strdup(pcap_server, argv[0]); + osmo_talloc_replace_string(pcap_server, &pcap_server->base_path, argv[0]); return CMD_SUCCESS; } -- To view, visit https://gerrit.osmocom.org/c/osmo-pcap/+/39190?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-pcap Gerrit-Branch: master Gerrit-Change-Id: I9ba1b7202d3e6a06cfd6efd6ed773b53007a99c4 Gerrit-Change-Number: 39190 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: pespin <pespin(a)sysmocom.de>