From dd0badf16b40bb56dace44ed32d53ce33bca40e0 Mon Sep 17 00:00:00 2001 From: George Joseph Date: Wed, 13 Jul 2022 12:38:10 -0600 Subject: [PATCH] geoloc_eprofile.c: Fix setting of loc_src in set_loc_src() line 196: loc_src = '\0'; should have been line 196: *loc_src = '\0'; The issue was caught by the gcc optimizer complaining that loc_src had a zero length because the pointer itself was being set to NULL instead of the _contents_ of the pointer being set to the NULL terminator. ASTERISK-30138 Reported-by: Sean Bright Change-Id: Id247be113cc8510f043ca053d5b4f5f3d32acd29 --- res/res_geolocation/geoloc_eprofile.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/res/res_geolocation/geoloc_eprofile.c b/res/res_geolocation/geoloc_eprofile.c index 64244f4424..af0e89c8e4 100644 --- a/res/res_geolocation/geoloc_eprofile.c +++ b/res/res_geolocation/geoloc_eprofile.c @@ -193,7 +193,7 @@ static int set_loc_src(struct ast_geoloc_eprofile *eprofile, const char *uri, co loc_src = strchr(local_uri, ';'); if (loc_src) { - loc_src = '\0'; + *loc_src = '\0'; loc_src++; } -- 2.47.2