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 dmorissette, 17 years ago

The same issue was found in msSLDParseLineSymbolizer(), msSLDParsePolygonSymbolizer(), msSLDParsePolygonSymbolizer(), msSLDParsePointSymbolizer() and msSLDParseTextSymbolizer

comment:2 by assefa, 17 years ago

Milestone: 5.0 release5.2 release

comment:3 by dmorissette, 14 years ago

Cc: assefa added
Milestone: 5.6 release6.0 release
Owner: changed from assefa to aboudreault

comment:4 by aboudreault, 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 assefa, 14 years ago

Cc: aboudreault added
Owner: changed from aboudreault to assefa

Thanks Alan, I will give it a look early next week and close.

Note: See TracTickets for help on using tickets.