[Sugar-devel] [PATCH] Infoslicer: Avoid crash when a article is downloaded - SL #2856

[Walter Bender]

On Fri, May 20, 2011 at 1:15 PM, Gonzalo Odiard <godiard at sugarlabs.org>wrote:

>
> I am not very familiar with treeview, but the "path" in the set_cursor call
>> seems to be OK (just ran some tests) but maybe it is a race condition of
>> some sort: pointing to the new last entry in the list before the list knows
>> it has a new last entry? In any case, simply eliminating the call to
>> set_cursor seems to be enough to address the crash problem. What is the
>> first part of the patch trying to accomplish?
>>
>>
> I have seen a crash in Fedora 14 .
> The error is the second in the ticket:
>
> 1305794514.172503 DEBUG infoslicer: load article b5ae3528-81f3-11e0-873b-0017c4a748fe
>
>
> Gdk-ERROR **: The program 'sugar-activity' received an X Window System error.
> This probably reflects a bug in the program.
> The error was 'RenderBadPicture (invalid Picture parameter)'.
>   (Details: serial 8423 error_code 159 request_code 147 minor_code 6)
>
>   (Note to programmers: normally, X errors are reported asynchronously;
>    that is, you will receive the error a while after causing it.
>    To debug your program, run it with the --sync command line
>    option to change this behavior. You can then get a meaningful
>
>    backtrace from your debugger if you break on the gdk_x_error() function.)
> aborting...
> Terminated by signal 6, pid 3283 data (None, <open file '<fdopen>', mode 'w' at 0x8cd8f98>, dbus.ByteArray('76b6586b426d3aaa9a710551c59d77fae4d574e6', variant_level=1))
>
> I could reproduce it several times.
>
> If you want, I can send you two different patches
>

No need to send two separate patches... I was just curious what the
relationship between the two was. I take it that the X bug had to do with
timing, which the call to idle_add fixes?

-walter


> Gonzalo
>
>
>
>> -walter
>>
>> --
>> Walter Bender
>> Sugar Labs
>> http://www.sugarlabs.org
>>
>>
>
>
> --
> Gonzalo Odiard
> SugarLabs Argentina
>
>


-- 
Walter Bender
Sugar Labs
http://www.sugarlabs.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20110520/0a5eb013/attachment.html>