[Vm-dev] JPEGReadWriter2Plugin crash fixed

[David T. Lewis]

P?hi,

Excellent! Thank you very much.

I put this on Mantis also: http://bugs.squeak.org/view.php?id=7691

The fix is for the Slang in VMMaker, so we will need to apply it there.
I'll look into it over the weekend if nobody has gotten to it sooner :)

Bravo for fixing an old and serious bug!

Dave


On Fri, Apr 27, 2012 at 10:02:43AM +0200, P?hi Andr?s wrote:
>  
> Hi,
> 
> I have found the origin of various crashes in the Squeak VM, which is related to the JPEGReadWriter2Plugin.
> The hunt begun with Juan's Cuis 4.0 release, see the emails exchanged below and the description.
> 
> I have attached the fixed C source, but the fix should go in the VM sources, because the code is
> generated as I see. Please include the fix in the VM sources.
> 
> Regards,
> Andras Pahi
> 
> ----- Original Message ----- 
> From: Juan Vuletich (mail lists) 
> To: P?hi Andr?s 
> Sent: Thursday, April 26, 2012 9:26 PM
> Subject: Re: Cuis 4.0 crash on startup
> 
> 
> Wow P?hi! This is great! Please send all this to the vm-dev mail list, and ask for it to be included (or directions to do it yourself).
> 
> It is my bug in the end... I wrote JPEGReadWriter2Plugin about 10 years ago...
> 
> Cheers,
> 
> Juan Vuletich
> 
> 
> Quoting P?hi Andr?s <pahia at t-online.hu>:
> 
>   Hi,
> 
>   Replace the winbuild/src/JPEGReadWriter2Plugin/JPEGReadWriter2Plugin.c file with the attached file,
>   recompile the Squeak VM and Cuis 4.0 rocks. In the plugin there was a 1 off access in the formBits array.
>   Because the code is generated, it needs to be corrected in the VM sources.
> 
>   How can I got there ?
> 
>   Executing several times World buildMagnifiedBackgroundImage in Cuis 3.3 crashed the Squeak VM.
>   I exported the background image data, converted from JPEG to other format (BMP/PNG) and set the new
>   format image to World backgroundImageData:. Everything worked as expected.
>   I replaced the formBits array accesses in JPEGReadWriter2Plugin2.c with my routines, checking the indices
>   and reporting off-array access attempts. Then I fixed the places in the C source.
> 
>   Regards,
>   Andras Pahi
>   ----- Original Message ----- 
>     From: Juan Vuletich (mail lists) 
>     To: P?hi Andr?s 
>     Sent: Thursday, April 26, 2012 12:47 PM
>     Subject: Re: Cuis 4.0 crash on startup
> 
> 
>     Great. Thanks. You might ask for help or directions in the vm-dev mail list. I'm sure that several people are at least aware of the bug.
> 
>     Cheers,
> 
>     Juan Vuletich
> 
> 
>     Quoting P?hi Andr?s <pahia at t-online.hu>:
> 
>       Hi,
>       Thank you for your reply. I will dig into the issues with the Windows VM.
>       Regards,
>       Andras Pahi
>         ----- Original Message ----- 
>         From: Juan Vuletich (mail lists) 
>         To: P?hi Andr?s 
>         Sent: Thursday, April 26, 2012 11:59 AM
>         Subject: Re: Cuis 4.0 crash on startup
> 
> 
>         Hi P?hi,
> 
>         There is some problem with that Windows VM, not limited to Cuis. If you google "Squeak VM 4.1.1 windows crash" you'll find many results, for example http://bugs.squeak.org/view.php?id=7585 http://bugs.squeak.org/bug_view_advanced_page.php?bug_id=7686 and http://bugs.squeak.org/view.php?id=7523 .
> 
>         I'm not involved in the development of the Windows VM, all I can do is recommend using Cog until this is fixed somehow.
> 
>         Regards,
> 
>         Juan Vuletich
> 
> 
>         Quoting P?hi Andr?s <pahia at t-online.hu>:
> 
>           Hi,
>           I have just gave a shoot on Cuis 4.0 using SqueakVM 4.1.1 on Windows.
>           It even does not start, the crash dump is attached, but with the latest 
>           Cog VM it runs fine.
>           I checked the older Cuis 3.3 image with the same SqueakVM, it runs fine.
>           Regards,
>           Andras Pahi
> 
> Cheers,
> Juan Vuletich