Cool VL Viewer forum
http://sldev.free.fr/forum/

Way to crash client
http://sldev.free.fr/forum/viewtopic.php?f=4&t=536
Page 1 of 1

Author:  WebGearbox [ 2011-08-21 21:51:12 ]
Post subject:  Way to crash client

To immediately close CoolVL - just press Ctrl+U and select PSD file...

Sure PSD format is not supported by sl. But if often click it occasionally and it crashes... Could you please make coolVL to recognize PSD file and say its unsupported - instead of immediate closure?

Author:  Henri Beauchamp [ 2011-08-21 23:13:18 ]
Post subject:  Re: Way to crash client

I tried to reproduce the crash, but the Cool VL Viewer properly reports a failure to decode the PSD image I used:
Attachment:
PSDupload.png
PSDupload.png [ 27.94 KiB | Viewed 1574 times ]


Also, I fail to see how you can accidentally select an image with an unrecognized format, since by default, and unless your force the filter to "all files", the file selector is passed a filter only allowing to select PNG, TGA, BMP and JPG files.

This said, the viewer must support TGA files, and since TGA files are without a "magic number" in their header (which is just a collection of pointers and size related numbers describing the image width, height, channels, etc), the viewer cannot recognize a random unsupported format from a TGA file and will attempt to load it as a TGA file anyway, which, depending on the values it finds in what it thinks is a legit TGA header, might lead to a crash. Some sanity checks are made, but they can't reasonably cover all possible cases.

Simply be more careful when overriding the file selector filter and trying to force the viewer to "digest" a format it doesn't support...

I will also note that this is in no way a Cool VL Viewer specific bug (so you should reproduce it with an official viewer and report it on the SL JIRA if you want it to get fixed some day).

Page 1 of 1 All times are UTC
Powered by phpBB® Forum Software © phpBB Group
https://www.phpbb.com/