kicad-developers team mailing list archive

[Dick Hollenbeck <dick@xxxxxxxxxxx>]

On 08/15/2014 11:04 PM, Andrew Zonenberg wrote:
> I filed a ticket with upstream
> http://trac.wxwidgets.org/ticket/16423#ticket so we'll see what they
> say.
> 
> On Fri, 2014-08-15 at 23:53 -0400, Andrew Zonenberg wrote:
>> The invalid read seems to be a bug in wxWidgets:
>>
>> src/gtk/window.cpp:1543
>>
>> static void SendSetCursorEvent(wxWindowGTK* win, int x, int y)
>> {
>>     wxSetCursorEvent event(x, y);
>>     wxWindowGTK* w = win;
>>     do {
>>         if (w->GTKProcessEvent(event))
>>         {
>>             gs_overrideCursor = &event.GetCursor();

That's a good catch.  3.0.1 has this bug.  3.0.0 does not.

And SVN HEAD does not.  Sheesh.

Here's subversion head as of today:

static void SendSetCursorEvent(wxWindowGTK* win, int x, int y)
{
    wxSetCursorEvent event(x, y);
    wxWindowGTK* w = win;
    do {
        if (w->GTKProcessEvent(event))
        {
            win->GTKUpdateCursor(false, false, &event.GetCursor());
            win->m_needCursorReset = true;
            return;
        }
        // this is how wxMSW works...
        if (w->GetCursor().IsOk())
            break;
        w = w->GetParent();
    } while (w);
    if (win->m_needCursorReset)
        win->GTKUpdateCursor();
}


For a long while I was simply using SVN head and updating it every couple of months.  It
does seem better than 3.0.1 at least in this regard.

It is a shame that this blows the confidence in 3.0.1.  Although if a person was going to
build from source, I suppose they could patch this bug before building.


Anyways, nothing wrong with compatibility of that package you installed, it simply has a
crashing bug in it.


Dick




>>             win->GTKUpdateCursor();
>>             gs_needCursorResetMap[win] = true;
>>             return;
>>         }
>>         // this is how wxMSW works...
>>         if (w->GetCursor().IsOk())
>>             break;
>>         w = w->GetParent();
>>     } while (w);
>>     if (gs_needCursorResetMap[win])
>>         win->GTKUpdateCursor();
>> }
>>
>> event is a local variable on the stack and as a result, as soon as this
>> function returns gs_overrideCursor is invalid.
>>
>> I'm not yet sure if this has any relationship whatsoever to the crash
>> I'm hunting.
>>
>> On Fri, 2014-08-15 at 23:31 -0400, Andrew Zonenberg wrote:
>>> I'm not using codelight itself, just their precompiled wx binaries, so I
>>> can easily remove the binary package, it was just less work than
>>> compiling from source.
>>>
>>> I'm investigating further now, I fixed an unrelated missing variable
>>> initializer in the process and will be submitting a patch to that
>>> shortly.
>>>
>>> On Fri, 2014-08-15 at 22:14 -0500, Dick Hollenbeck wrote:
>>>> On 08/15/2014 08:32 PM, Andrew Zonenberg wrote:
>>>>> Does not crash when run under Valgrind, instead gives this error:
>>>>>
>>>>> ==32723== Invalid read of size 8
>>>>
>>>> ^ this error?  I've seen that before and never could figure it out, nor was it ever the
>>>> cause of any problem running valgrind.
>>>>
>>>> I would say build your own wxwidgets from the 3.0.1 source.  We're dealing with an unknown
>>>> problem, so one has to revert to probabilities at that point.
>>>>
>>>> I think the highest probability is that there is some incompatibility in your software
>>>> stack.  Building wx from source is an easy experiment.  You do not have to remove the wx
>>>> package if it makes codelight happy.
>>>>
>>>> Follow the instructions I gave in an earlier email today.  Then after configuring the
>>>> kicad build, I run
>>>>
>>>> (ccmake is in the package cmake-curses-gui I think.)
>>>>
>>>> $ ccmake .
>>>>
>>>> in the build directory.  Then I paste in
>>>>
>>>>   /opt/wx3.0-stl/bin/wx-config
>>>>
>>>> into the field named:  wxWidgets_CONFIG_EXECUTABLE and reconfigure in ccmake.
>>>>
>>>> Likewise for the debug build, which over time will be very helpful for you.
>>>>
>>>>
>>>> Obviously the first part of the /opt/wx3.0-stl/bin/wx-config string came from the --prefix
>>>> argument to the wx configure command.
>>>>
>>>> Don't forget to run $ sudo ldconfig
>>>> after installing the home made wx libraries.
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>> ==32723==    at 0x5D00EC0: wxCursor::GetCursor() const (cursor.cpp:287)
>>>>> ==32723==    by 0x5D20A8D: wxWindow::GTKUpdateCursor(bool, bool)
>>>>> (window.cpp:3752)
>>>>> ==32723==    by 0x5D002A0: UpdateCursors(wxWindow*, bool)
>>>>> (cursor.cpp:331)
>>>>> ==32723==    by 0x5D00F42: SetGlobalCursor(wxCursor const&)
>>>>> (cursor.cpp:350)
>>>>> ==32723==    by 0x5D0109C: wxEndBusyCursor() (cursor.cpp:376)
>>>>> ==32723==    by 0x1D3FCCDB:
>>>>> DIALOG_DRC_CONTROL::OnStartdrcClick(wxCommandEvent&)
>>>>> (dialog_drc.cpp:172)
>>>>> ==32723==    by 0x65F92DE:
>>>>> wxAppConsoleBase::CallEventHandler(wxEvtHandler*, wxEventFunctor&,
>>>>> wxEvent&) const (appbase.cpp:623)
>>>>> ==32723==    by 0x6750FF1:
>>>>> wxEvtHandler::ProcessEventIfMatchesId(wxEventTableEntryBase const&,
>>>>> wxEvtHandler*, wxEvent&) (event.cpp:1384)
>>>>> ==32723==    by 0x67513A5:
>>>>> wxEvtHandler::SearchDynamicEventTable(wxEvent&) (event.cpp:1743)
>>>>> ==32723==    by 0x6751445: wxEvtHandler::TryHereOnly(wxEvent&)
>>>>> (event.cpp:1577)
>>>>> ==32723==    by 0x6751502: wxEvtHandler::ProcessEventLocally(wxEvent&)
>>>>> (event.h:3671)
>>>>> ==32723==    by 0x6751564: wxEvtHandler::ProcessEvent(wxEvent&)
>>>>> (event.cpp:1487)
>>>>> ==32723==  Address 0x7feffdee8 is not stack'd, malloc'd or (recently)
>>>>> free'd
>>>>>
>>>>>
>>>>> On Fri, 2014-08-15 at 21:19 -0400, Andrew Zonenberg wrote:
>>>>>> Happens every time I run DRC on this board. I don't want to change the
>>>>>> design for fear of not being able to reproduce it.
>>>>>>
>>>>>> This is with the Codelite packages of wx3.0.1 and BZR 5073 kicad on
>>>>>> Debian 7.
>>>>>>
>>>>>> Program received signal SIGSEGV, Segmentation fault.
>>>>>> IA__gdk_cursor_ref (cursor=cursor@entry=0xf2e66c318c48348)
>>>>>> at /tmp/buildd/gtk+2.0-2.24.10/gdk/gdkcursor.c:57
>>>>>> 57	/tmp/buildd/gtk+2.0-2.24.10/gdk/gdkcursor.c: No such file or
>>>>>> directory.
>>>>>> (gdb) bt
>>>>>> #0  IA__gdk_cursor_ref (cursor=cursor@entry=0xf2e66c318c48348)
>>>>>> at /tmp/buildd/gtk+2.0-2.24.10/gdk/gdkcursor.c:57
>>>>>> #1  0x00007ffff47cc691 in IA__gdk_window_set_cursor (window=0xbe3120,
>>>>>> cursor=0xf2e66c318c48348) at /tmp/buildd/gtk
>>>>>> +2.0-2.24.10/gdk/gdkwindow.c:8199
>>>>>> #2  0x00007ffff6b8897f in wxWindow::GTKUpdateCursor (this=0x8273f0,
>>>>>> isBusyOrGlobalCursor=<optimized out>, isRealize=false)
>>>>>> at ../src/gtk/window.cpp:3761
>>>>>> #3  0x00007ffff6b682a1 in UpdateCursors (win=win@entry=0x8273f0,
>>>>>> isBusyOrGlobalCursor=<optimized out>) at ../src/gtk/cursor.cpp:331
>>>>>> #4  0x00007ffff6b68f43 in SetGlobalCursor (cursor=...)
>>>>>> at ../src/gtk/cursor.cpp:350
>>>>>> #5  0x00007ffff6b6909d in wxEndBusyCursor ()
>>>>>> at ../src/gtk/cursor.cpp:376
>>>>>> #6  0x00007fffe26f4cdc in DIALOG_DRC_CONTROL::OnStartdrcClick
>>>>>> (this=0x3645700, event=...)
>>>>>> at /nfs/home/azonenberg/Documents/local/programming/3rdparty/kicad/pcbnew/dialogs/dialog_drc.cpp:172
>>>>>> #7  0x00007ffff627c2df in wxAppConsoleBase::CallEventHandler
>>>>>> (this=0x7b29e0, handler=0x3645700, functor=..., event=...)
>>>>>> at ../src/common/appbase.cpp:623
>>>>>>
>>>>>>
>>>>>> _______________________________________________
>>>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>>>> More help   : https://help.launchpad.net/ListHelp
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>>> More help   : https://help.launchpad.net/ListHelp
>>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>> More help   : https://help.launchpad.net/ListHelp
>>>
>>> _______________________________________________
>>> Mailing list: https://launchpad.net/~kicad-developers
>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>> More help   : https://help.launchpad.net/ListHelp
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help   : https://help.launchpad.net/ListHelp
> 
> 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>