]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
geoloc_eprofile.c: Fix setting of loc_src in set_loc_src()
authorGeorge Joseph <gjoseph@digium.com>
Wed, 13 Jul 2022 18:38:10 +0000 (12:38 -0600)
committerGeorge Joseph <gjoseph@digium.com>
Wed, 13 Jul 2022 18:43:35 +0000 (13:43 -0500)
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

index 64244f44245108f1166751d512d306f5571cb774..af0e89c8e41e77fca3deffa941dbb4e5be4efae7 100644 (file)
@@ -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++;
        }