Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash when msLoadMSRasterBufferFromFile() fails to open file #4265

Closed
mapserver-bot opened this issue Apr 4, 2012 · 4 comments
Closed

Crash when msLoadMSRasterBufferFromFile() fails to open file #4265

mapserver-bot opened this issue Apr 4, 2012 · 4 comments
Assignees
Milestone

Comments

@mapserver-bot
Copy link

Reporter: woodbri
Date: 2012/03/29 - 21:34
Trac URL: http://trac.osgeo.org/mapserver/ticket/4265
mapserver crashes when it can not open a symbol

[Thu Mar 29 15:23:20 2012].235133 msDrawLabelCache(): labelcache_map_edge_buffer = 10
[Thu Mar 29 15:23:20 2012].243273 msLoadMSRasterBufferFromFile(): General error message. unable to open file /u/data/tiger2011-maps/indistrial for reading
mapserv: mapagg.cpp:571: int agg2RenderPixmapSymbol(imageObj*, double, double, symbolObj*, symbolStyleObj*): Assertion `pixmap->type == MS_BUFFER_BYTE_RGBA' failed.
Aborted
@ghost ghost assigned tbonfort Apr 4, 2012
@tbonfort
Copy link
Member

tbonfort commented Apr 6, 2012

@woodbri Looking through the code, it seems that all the calls loading an image file now check for the return value, which might not have been the case before. If you are able to retest this please confirm this is the case.

If you do manage to crash with an unreadable file, please provide a backtrace.

@woodbri
Copy link

woodbri commented Apr 6, 2012

@tbonfort here are mapfiles that can reproduce this error and shows some related problems.
http://imaptools.com:8080/dl/bug4265.zip
See the README file.

@tbonfort
Copy link
Member

tbonfort commented Apr 6, 2012

loading svg symbols by filename without defining a proper symbol is not supported. this shortcut is only available for pixmap symbols.

@woodbri
Copy link

woodbri commented Apr 6, 2012

ok, an appropriate error message and handling is fine. This should also get transferred to doc so it is reflected there. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants