[mapserver-dev] When to mark an error as "reported"?

[Eichner, Andreas - SID]

Hi devs,

I just found that msResetErrorList() in maperror.c does not reset the "isreported" field to MS_FALSE. This can cause strange error reporting behavior when in FastCGI mode. While checking the uses of this field I found that msWriteError() and msWriteErrorXML() mark errors as reported while msWriteErrorImage() does not.
The problem is that msWriteErrorImage() does not access the errorObj directly but uses msGetErrorString(). So my question is: is it OK to let msGetErrorString() mark the errors as reported or should this be done in msWriteErrorImage()?
I think it's truly a bug but I'm not sure what the best solution would be. Any ideas?

Regards,
Andreas