← Back to team overview

kicad-developers team mailing list archive

Re: Segfault when pcbnew starts with an invalid file

 

On 08/05/2015 04:39 PM, Wayne Stambaugh wrote:
> On 8/5/2015 10:16 AM, Maciej Sumiński wrote:
>> Chris has pointed out that it was already discussed [1] and the change
>> was not entirely clear. Apparently it had not been wrapped before
>> revision 5834, so maybe OnPgmExit() should be still called for both
>> Python and non-Python versions of pcbnew?
> 
> This was done because the problem did not occur when wxPython scripting
> was disabled and Dick is planning on add wxPython on top of Kicad rather
> than a bolted onto the side like our current implementation.
> Compiling OnPgmExit() out of non wxPython builds was at his request.
> The original crash had to do with wxPython clean up not being called
> properly.  I just tried to duplicate this bug on a 32 bit build on
> windows by running 'pcbnew foo` with no scripting enabled, and I cannot
> get it to crash.  Are you testing this on Linux?

Yes, but please try with a file with invalid contents, so there is an
IO_ERROR exception thrown.

Regards,
Orson

>>
>> Regards,
>> Orson
>>
>> 1.
>> https://www.mail-archive.com/kicad-developers@xxxxxxxxxxxxxxxxxxx/msg14014.html
>>
>> On 08/05/2015 11:22 AM, Maciej Sumiński wrote:
>>> Pcbnew without scripting support crashes when run with an invalid file
>>> given as a parameter (stack overflow in ~wxSingleInstanceChecker()).
>>>
>>> There is a fix for that, but wrapped with #ifdef
>>> KICAD_SCRIPTING_WXPYTHON .. #endif. I removed the mentioned directives
>>> and it seems fine here, but I am not confident enough to commit it
>>> immediately. Any thoughts?
>>>
>>> Regards,
>>> Orson
>>>
>>>
>>>
>>> _______________________________________________
>>> 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
> 


Attachment: signature.asc
Description: OpenPGP digital signature


Follow ups

References