[alsa-devel] snd_usb_caiaq bug?

[Takashi Iwai]

At Sat, 10 Jan 2009 13:53:24 +0100,
Marco Santamaria wrote:
> 
> During a Rosegarden session a qsampler crash caused the following output in
> dmesg:
> 
> [11530.642970] qsampler[22794]: segfault at 00000008 eip b73f52c5 esp bfe77e20
> error 4
> [13278.161113] ALSA /home/marco/build/alsa-1.0.17/linux-2.6.24-23-rt/driver/
> alsa-driver-1.0.17/usb/caiaq/../../alsa-kernel/usb/caiaq/caiaq-midi.c:90:
> snd-usb-caiaq log: snd_usb_caiaq_midi_send(f6d52b40): usb_submit_urb()
> failed,ret=-22, len=61
> [13720.859888] qjackctl.bin[22273]: segfault at 00000314 eip b7b07a7f esp
> bfbe8f54 error 4
> 
> There are segmentation fault errors from qsampler and qjackctl, but perhaps
> there's also a problem in caiaq-midi.c .

Might be, but difficult to judge as now since the error seems
happening after qsampler, and the urb error itself seems
non-critical.


Takashi

>  
> Regards,
> Marco
> 
> 2009/1/9 Takashi Iwai <tiwai at suse.de>
> 
>     At Thu, 8 Jan 2009 23:11:53 +0100,
>     Daniel Mack wrote:
>     >
>     > Hi Takashi,
>     >
>     > On Thu, Jan 08, 2009 at 03:37:30PM +0100, Takashi Iwai wrote:
>     > > > every thing is Ok for me now. No more crash, no more messages in
>     dmesg.
>     > > > I just had a warning during compilation, but I think this is not
>     really
>     > > > important :
>     > > >
>     > > > sound/usb/caiaq/caiaq-midi.c: In function \
>     u2018snd_usb_caiaq_midi_output_done
>     > > > \u2019:
>     > > > sound/usb/caiaq/caiaq-midi.c:170: attention : unused variable \
>     u2018buf\u2019
>     > > >
>     > > > Perhaps this is not the same line (170) as you but, in my file, it
>     concerns
>     > > > the last function.
>     > >
>     > > OK, it's simple to fix.
>     > > The below is the latest version I committed now to sound git tree.
>     > > It'll be pushed to 2.6.29 (and stable kernels) later.
>     >
>     > Thanks a lot for your help - I was on vacation until now and had very
>     > limited ways to follow that thread. I would, however, apprectiate the
>     > driver version number would be increased for this change, as it makes
>     > tracking of user error reports better trackable.
>     >
>     > Do you want me to send in a patch for that or is it easier if you do
>     > it yourself?
>    
>     I increased the version to 1.3.10 now.
> 
>     thanks,
>    
>     Takashi
> 
>