pespin submitted this change.

View Change



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified
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(-)

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 change 39190. To unsubscribe, or for help writing mail filters, visit settings.

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@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>