Buggy speakup on Ubuntu 12.04?
Samuel Thibault
samuel.thibault at ens-lyon.org
Sat Jun 16 17:07:41 EDT 2012
Hello,
Chris Brannon, le Sat 16 Jun 2012 08:48:42 -0500, a écrit :
> "Bohdan R. Rau" <ethanak at polip.com> writes:
>
> > 1) if I press "go to" key, any next key causes immediately kernel panic
>
> Hey Bohdan,
> I think I found the source of the goto bug.
> If you look at the handle_goto function from main.c of the Speakup
> sources, you'll see this declaration:
>
> static u_char *goto_buf = "\0\0\0\0\0\0";
>
> That just creates a pointer to a string constant.
> Try to assign to *goto_buf, and boom, there is your panic.
>
> Instead, we want:
>
> static u_char goto_buf[] = "\0\0\0\0\0\0";
Indeed, and as commented, no need to explicitly initialize to zeros
since it's a static buffer.
Could you handle submitting the fix to the linux-kernel mailing list?
Samuel
More information about the Speakup
mailing list