#139 segfault in virtKeyBindingsAreInvalid()

development
closed-fixed
Eddy De Greef
Program (402)
7
2002-03-08
2002-03-08
ariese
No

The latest cvs version crashes in
virtKeyBindingsAreInvalid() on an SGI. The reason is
that the bindings string comes with a leading and
trailing \n, therefore the number of bindings is
computed in a wrong way.

Axel

Discussion

  • Eddy De Greef
    Eddy De Greef
    2002-03-08

    • priority: 5 --> 7
    • assigned_to: nobody --> edg
     
  • Eddy De Greef
    Eddy De Greef
    2002-03-08

    Logged In: YES
    user_id=73597

    Damned. Can you provide me with the output of
    xprop -root | grep _MOTIF_DEFAULT_BINDINGS ?
    Thanks.

    You can temporarily avoid the crash by setting
    "nedit*overrideDefaultVirtualKeyBindings" to Always or Never.

     
  • Eddy De Greef
    Eddy De Greef
    2002-03-08

    Logged In: YES
    user_id=73597

    The xprop output won't be necessary any more.

    I've uploaded a fix to CVS. It should be much more
    robust now (it was a bad idea to assume a certain format).

    Can you verify that it works ?

    (Now it doesn't even crash when I attach the README file
    to the root window :-)

     
  • Eddy De Greef
    Eddy De Greef
    2002-03-08

    • status: open --> open-fixed
     
  • Eddy De Greef
    Eddy De Greef
    2002-03-08

    Logged In: YES
    user_id=73597

    The xprop output won't be necessary any more.

    I've uploaded a fix to CVS. It should be much more
    robust now (it was a bad idea to assume a certain format).

    Can you verify that it works ?

    (Now it doesn't even crash when I attach the README file
    to the root window :-)

     
  • ariese
    ariese
    2002-03-08

    Logged In: YES
    user_id=248896

    Yes, it works. Thanks.

     
  • Eddy De Greef
    Eddy De Greef
    2002-03-08

    • status: open-fixed --> closed-fixed
     
  • Eddy De Greef
    Eddy De Greef
    2002-03-08

    Logged In: YES
    user_id=73597

    Thank you!
    Closing this.