[Bug 967416] Re: inkscape crashed with SIGSEGV in file_save()

2015-01-11 Thread Mark Harmer
@jazzynico Thanks, I think that would solve the crash, however some issues come to mind with that solution: 1. It would introduce some global state, which would be tricky to reason about for anyone not familiar with the specifics. This is particularly complicated since it appears from the

[Bug 967416] Re: inkscape crashed with SIGSEGV in file_save()

2015-01-11 Thread Mark Harmer
I've attached a proposed patch. One thing of note is that this actually keeps the Glib::MainLoop, after reading more into it, it appears that a recursive call to running a main loop is supported. The actual problem is that this was reusing the default MainContext which had all of the original

[Bug 967416] Re: inkscape crashed with SIGSEGV in file_save()

2015-01-10 Thread Mark Harmer
I thought I might take a crack at this since it looked interesting. I apologize for the length, I've added debugging details so hopefully it's clear. I'm new to the codebase and GTK, so I've added a lot so it can be double-checked. I don't think this is a threading problem. Rather, it looks like