Opened 17 years ago
Last modified 14 years ago
#2165 new defect
Need to return MS_SUCCESS/MS_FAILURE in msSLDParseRasterSymbolizer()
Reported by: | dmorissette | Owned by: | assefa |
---|---|---|---|
Priority: | normal | Milestone: | 6.0 release |
Component: | WMS Server | Version: | 5.0 |
Severity: | normal | Keywords: | |
Cc: | assefa, aboudreault |
Description
While working on ticket #302, I noticed that msSLDParseRasterSymbolizer() in mapogcsld.c has a void return type and doesn't return any indication of success/failure so I am unable to propagate possible errors to the caller.
msSLDParseRasterSymbolizer() needs to be fixed to return MS_SUCCESS/MS_FAILURE and the callers need to be modified to check its return value and propagate error situations to their own callers, etc.
Change History (5)
comment:1 by , 17 years ago
comment:2 by , 17 years ago
Milestone: | 5.0 release → 5.2 release |
---|
comment:3 by , 14 years ago
Cc: | added |
---|---|
Milestone: | 5.6 release → 6.0 release |
Owner: | changed from | to
comment:4 by , 14 years ago
I have modified SLD functions to return proper values. However, I haven't modified the function calls. They were not checking the return values and still don't.
Assefa, will let you close the ticket if everything is ok or let me know after you revisited the ticket if there are other issues you me to fix.
Fixed and committed r9611.
comment:5 by , 14 years ago
Cc: | added |
---|---|
Owner: | changed from | to
Thanks Alan, I will give it a look early next week and close.
The same issue was found in msSLDParseLineSymbolizer(), msSLDParsePolygonSymbolizer(), msSLDParsePolygonSymbolizer(), msSLDParsePointSymbolizer() and msSLDParseTextSymbolizer