#162 Crash on Changing Default Search Style

development
closed-fixed
Program (402)
5
2002-05-16
2002-05-08
No

I can reliably produce cores by trying to change the default search style.

#0 searchRegexCB (w=0x83c4880, window=0x83677d0, callData=0xbfffd2a0 "\002") at menu.c:542
#1 0x400567bf in XtCallCallbackList () from /usr/X11R6/lib/libXt.so.6
#2 0x0811e9d9 in ToggleButtonCallback ()
#3 0x08122066 in XmToggleButtonSetState ()
#4 0x081612f5 in AllOffExcept ()
#5 0x08161566 in RadioBehaviorAndMenuHistory ()
#6 0x08161733 in EntryFired ()
#7 0x081616f8 in ChildsActivateCallback ()
#8 0x0811f70e in BtnUp ()
#9 0x4008644d in HandleActions () from /usr/X11R6/lib/libXt.so.6
#10 0x4008690f in HandleSimpleState () from /usr/X11R6/lib/libXt.so.6
#11 0x40086e64 in _XtTranslateEvent () from /usr/X11R6/lib/libXt.so.6
#12 0x40061f77 in XtDispatchEventToWidget () from /usr/X11R6/lib/libXt.so.6
#13 0x40062928 in _XtDefaultDispatcher () from /usr/X11R6/lib/libXt.so.6
#14 0x40062b89 in XtDispatchEvent () from /usr/X11R6/lib/libXt.so.6
#15 0x08097c14 in ServerMainLoop (context=0x83220b8) at server.c:158
#16 0x08051338 in main (argc=2, argv=0xbffff784) at nedit.c:536
#17 0x401dc7ee in __libc_start_main () from /lib/libc.so.6

NEdit release of Mar 11, 2002

Built on: Linux, x86, GNU C
Built at: Apr 24 2002, 20:16:11
With Motif: 2001 [@(#)Motif Version 2.1.30]
Running Motif: 2001
Server: The XFree86 Project, Inc 40100001

This one works fine on the same display:
NEdit release of Mar 11, 2002

Built on: Linux, x86, GNU C
Built at: Apr 18 2002, 14:14:31
With Motif: 2001 [@(#)Motif Version 2.1.30]
Running Motif: 2001
Server: The XFree86 Project, Inc 40100001

Discussion

  • Eddy De Greef

    Eddy De Greef - 2002-05-08
    • milestone: --> development
     
  • Eddy De Greef

    Eddy De Greef - 2002-05-08

    Logged In: YES
    user_id=73597

    This looks like another instance of the
    HidePointerOnKeyedEvent() problem: not all callbacks receive
    mouse/key events as argument (in this case, it's a
    XmNvalueChangedCallback callback).
    I think that there are several other cases where it can
    go wrong (for instance, the same callback, such as
    doActionCB, can be called under various conditions,
    which not always include an event).

     
  • Steve LoBasso

    Steve LoBasso - 2002-05-08

    Logged In: YES
    user_id=140805

    I've added a check in HidePointerOnKeyedEvent() to check if
    the event is NULL. This may fix the problem, but I can't
    duplicate it to verify.

     
  • Eddy De Greef

    Eddy De Greef - 2002-05-08

    Logged In: YES
    user_id=73597

    That won't be sufficient. According to Thorsten's stack
    trace, the crash happens before that function is called,
    probably at line 2161:

    HidePointerOnKeyedEvent(WidgetToWindow(MENU_WIDGET(w))->lastFocus,
    ((XmAnyCallbackStruct *)callData)->event);

    The callData argument may not even carry an event.

    It doesn't look like it is easy to fix. There are so many
    callbacks that are called under various conditions.

     
  • Steve LoBasso

    Steve LoBasso - 2002-05-08

    Logged In: YES
    user_id=140805

    after looking at the places where I added
    HidePointerOnKeyedEvent(), many of them shouldn't have been
    there. It was an accident that I added them. I was only
    trying to add it in callbacks realated to menu items, that
    could be driven by key events. For this reason several of
    them were just wrong. I have removed them.

     
  • Eddy De Greef

    Eddy De Greef - 2002-05-16
    • labels: --> Program
    • assigned_to: nobody --> slobasso
    • status: open --> closed-fixed
     
  • Eddy De Greef

    Eddy De Greef - 2002-05-16

    Logged In: YES
    user_id=73597

    I suppose everything is fine now. Closing.

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks