crash in mapogssld.c -> msSLDGetGraphicSymbol()

[pf@…]

I found a problem in mapogssld.c in context of using external Graphics.
There is a bug in mapogcsld.c; No limit check for creating new symbols in
context of sld with externalGraphic references. Further it loads images which
are allready available because the symbols use the unique name instead of the
graphic name.
I suggest to use  extGraphicName as SymbolName instead of the Unique
Name of  created TmpFileName. This will reduce the amount of
created Symbols, and will also reduce the number of calls to
msHTTPGetFile(). Find attached the modified functions based on code of
current 4.10.0 Release.
I also insert a LIMIT Check for static symbolset.symbol[] in
msSLDGetGraphicSymbol to avoid a crash caused by inserting new Symbols.
Kindest regards,
Peter

[pf@…]

modified functions msSLDGetGraphicSymbol() and msSLDParseExternalGraphic() from mapogcsld.c

[pf@…]

*** Bug 1985 has been marked as a duplicate of this bug. ***