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 <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>

Reply via email to