[Vm-dev] Pulse audio Scratch plugin causes seg fault on 64bit Ubuntu installation

[David T. Lewis]

On Thu, Aug 02, 2012 at 05:36:42PM -0400, Amos Blanton wrote:
>  
> To reproduce:
> 
> 1. Execute Scratch using current VM and pulse plugin (on Ubuntu 12.04, 64
> bit system):
> Executing: /usr/lib/squeak/4.4.7-2357/squeakvm -encoding UTF-8
> -vm-display-x11 -xshm -plugins
> /usr/lib/scratch/plugins/:/usr/lib/squeak/4.4.7-2357/ -vm-sound-pulse
> /usr/share/scratch/Scratch.image
> (There is no pulse plugin in /usr/lib/scratch/plugins in my installation
> package, so it must be using the one in /usr/lib/squeak/4.4.7-2357)
> 
> 2. Click "Sounds" tab, click play icon next to meow. That results in:
> 
> Segmentation fault
> 
> 10273532 [] in SoundPlayer class>playLoop
> 10262332 Semaphore>critical:
> 10262240 SoundPlayer class>playLoop
> 10273228 [] in SoundPlayer class>startPlayerProcessBufferSize:rate:stereo:
> 10273320 [] in BlockContext>newProcess
> Aborted (core dumped)
> 
> Note that the same package on 32 bit systems does not produce the error
> (playback and recording both work).
> 
> The issue is also described here:
> https://bugs.launchpad.net/ubuntu/+source/squeak-vm/+bug/1009080
> 
> If someone can take a look and patch this, we'd very much appreciate it!
> This is the last major (known) issue we've got with Scratch running on
> Ubuntu.
> 
> -Amos

I have a possible fix for this problem, but need some help to verify.

The issue is tracked on Mantis http://bugs.squeak.org/view.php?id=7701

I believe the problem is just some type declaration issues in the support
code that result in pointers being stored in ints, hence VM crash. A
potential fix is attached (and also on uploaded to the Mantis issue).

I need someone to test the 64-bit compiled VM on Scratch with Ubuntu
and pulse audio and see if the VM crashes are eliminated.

Bert: Can I send you a compiled VM to test with Scratch/Etoys/Squeak
and find out if sound still crashes the VM?

Ian: Regardless of whether this fixes the Scratch problem, the type
declarations in vm-sound-pulse need to be updated, so please consider
the attached as a patch.

Thanks,
Dave

-------------- next part --------------
A non-text attachment was scrubbed...
Name: platforms-unix-sound-pulse-sqUnixSoundPulse.c.tar
Type: application/x-tar
Size: 40960 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/vm-dev/attachments/20120807/5bc83eb4/platforms-unix-sound-pulse-sqUnixSoundPulse.c-0001.tar