Skip to content

Commit

Permalink
msLoadMap(): fix nullptr dereference when using LATLON keyword
Browse files Browse the repository at this point in the history
As far as I can see, this was broken in the nominal case since
f595e91

So this feature is not used in msautotest, nor documented
(https://mapserver.org/search.html?q=LATLON doesn't return anything
relevant)
Probably that the mapObj::latlon member would be better initialized by taking
the geographic CRS of the mapObj::projection.
  • Loading branch information
rouault authored and github-actions[bot] committed Oct 6, 2022
1 parent f60c84e commit 8bcd196
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion mapfile.c
Original file line number Diff line number Diff line change
Expand Up @@ -6383,7 +6383,6 @@ static int loadMapInternal(mapObj *map)
if((map->interlace = getSymbol(2, MS_ON,MS_OFF)) == -1) return MS_FAILURE;
break;
case(LATLON):
msFreeProjectionExceptContext(&map->latlon);
if(loadProjection(&map->latlon) == -1) return MS_FAILURE;
break;
case(LAYER):
Expand Down

0 comments on commit 8bcd196

Please sign in to comment.