Opened 20 years ago

Closed 19 years ago

#1002 closed defect (fixed)

msSaveImage() failures for GetCoverage don't result in error message

Reported by: warmerdam Owned by: sdlime
Priority: high Milestone:
Component: WCS Server Version: 4.3
Severity: normal Keywords:
Cc:

Description

Steve,

I was doing a little WCS test, and accidentally ran with a WEB IMAGEPATH
set to a directory that didn't exist.  This resulted in the msSaveImage()
failing because it could not create the temporary file.  

An msSetError() call was made down in msSaveImageGDAL(), but the error was
never turned into an exception "up the stack". 

I *think* that this should likely be done in msWCSGetCoverage() shortly
after the msSaveImage() call if the returned status is not MS_SUCCESS.   Looking
at the code I notice that msDrawRasterLayerLow() returning an error also
does not result in any sort of exception.  Should it?

Change History (1)

comment:1 by fwarmerdam, 19 years ago

Resolution: fixed
Status: newclosed
I have corrected both these by throwing a WCS exception if these errors occur.

Note that if msSaveImage() fails, the exception will have already been
improperly typed with the image result mime type but at that point there is
little we can do to fix the problem.

Note: See TracTickets for help on using tickets.