[cairo-bugs] [Bug 91967] Assertion "(_cairo_atomic_int_get (&(&surface->ref_count)->ref_count) > 0)"

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=91967

--- Comment #39 from Alberts Muktupāvels <alberts.muktupavels at gmail.com> ---
(In reply to Wolfgang Ulbrich from comment #38)
> (In reply to Alberts Muktupāvels from comment #37)
> > This bug is/was about assert that is caused by calling cairo_surface_destroy
> > on already destroyed image. Setting image to NULL after destroying it is
> > enough to fix this.
> > 
> > This is already tested by several users/developers. Can someone of cairo
> > developers apply this fix? Please!
> 
> Sorry Albert,
> did you ever try the reproducer?

No, but just did... Without patch I lost whole session, crashed both
gnome-panel and metacity.

With patch only gnome-panel crashed with different error.

> Cairo crashed at several places with your patch.
> Also libwnck3-3.18.0 is in stacktraces.
> Simply applying your patch is not enough.

It is enough to fix bug that I reported. This bug is about assert not about
possible x errors. Lets not mix separate bugs in one report.

X errors can be handled in applications, if you create xlib cairo surface you
can add error traps to fix problem - handle error or simply ignore. This might
be or might not be problem in cairo, but probably can be fixed in applications.
Anyway this is already different problem and probably should be reported as
separate bug.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.cairographics.org/archives/cairo-bugs/attachments/20160617/8412685c/attachment.html>