This bug was fixed in the package openbox - 3.6.1-10ubuntu1
---
openbox (3.6.1-10ubuntu1) lunar; urgency=medium
* Cherry-pick patch from
http://git.openbox.org/?p=mikachu/openbox.git;a=commit;h=d41128e5a1002af41c976c8860f8299cfcd3cd72
to avoid crashing when switching from
Thanks, uploaded!
** Changed in: openbox (Ubuntu)
Status: Confirmed => Fix Committed
--
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to glib2.0 in Ubuntu.
https://bugs.launchpad.net/bugs/2011751
Title:
openbox crashed with
Latest patch has requested changes. Bug has been submitted to upstream
Debian here: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1033385
** Bug watch added: Debian Bug tracker #1033385
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1033385
** Patch added: "glib_crash_bugfix_v5.patch"
Thanks for the patch, Aaron. I'm able to reproduce this bug using the
firefox f11 -> open link in new window, when using the live session of
the daily-live lubuntu iso. And the attached patch does seem to help.
Overall the patch is what I was hoping to see, but I will request some
changes.
*
Gah, ok, one more try. I failed to update the debian/copyright file last
time. This version has the updated copyright file.
** Patch added: "glib_crash_bugfix.patch"
https://bugs.launchpad.net/ubuntu/+source/openbox/+bug/2011751/+attachment/5657032/+files/glib_crash_bugfix.patch
--
You
After a discussion with dbungert, we're using a different patch instead.
This one is possibly upstream already, which makes it superior to the
original patch I was suggesting since it means less maintenance overhead
in the future.
** Patch added: "glib_crash_bugfix.patch"
New patch, same approach, but with a fallback added if itPrev turns out
to be null.
** Patch added: "glib_crash_bugfix.patch"
https://bugs.launchpad.net/ubuntu/+source/openbox/+bug/2011751/+attachment/5656971/+files/glib_crash_bugfix.patch
--
You received this bug notification because you
One worry I am having with my patch now is, what happens if the first
element is the one being modified? Then itPrev is a null pointer. I
*think* from looking at the GLib code that g_list_next will then return
null when it it handed itPrev, which means "it" will be set to null,
which I believe
** Tags added: patch
--
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to glib2.0 in Ubuntu.
https://bugs.launchpad.net/bugs/2011751
Title:
openbox crashed with SIGABRT
Status in glib2.0 package in Ubuntu:
Invalid
Status in openbox
(One extra bit of info that I realized was left out - according to the
comment on the Arch Linux forums here:
https://bbs.archlinux.org/viewtopic.php?pid=2090270#p2090270 the first
loop iteration succeeds. It's not until the second iteration (after the
list has been modified) that the crash
Aaron asked me to test his version via PPA
As noted in the original bug report; this bug was annoying... as was
hitting me often enough I stopped using Lubuntu & used Xfce & GNOME
instead while, before returning to LXQt though I replaced `openbox` as
WM with `xfwm4` (to avoid issues).
(note:
** Description changed:
Lubuntu lunar (primary box) on
- dell [optiplex] 7050 (i5-6500, 16gb, intel hd530/i915)
I experienced a crash yesterday (openbox) but tend to ignore the first
crashes... Whilst using the machine again today a crash occurred.
This is my primary box, so my
OK, so I'm not entirely sure testing the one item case is even possible
since the crash happens when switching from a fullscreen window (which
implies that there are at least two windows). However, this is what I
did, and everything was successful (no crashes).
1. Boot a Lubuntu Lunar ISO, and
@dbungert: While the patch that simply copies the list is a possible way
of doing things, a very experienced user on the Arch Linux forums had
concerns about doing things that way, see
https://bbs.archlinux.org/viewtopic.php?pid=2090570#p2090570 "But
creating a copy of the list is inefficient and
Partial review of the patch so far. I haven't read much openbox code so
take this with a grain of salt.
https://bugzilla.icculus.org/show_bug.cgi?id=6669#c9 mentions a concern
that itPrev may still be pointing to invalid data. I share this
concern. I think some interesting testcases are what
I don't have upload access to this package yet (it's not in the Lubuntu
packageset apparently) so I'm uploading the patch here so that it can be
sponsored.
** Patch added: "glib_crash_bugfix.patch"
** Changed in: openbox (Ubuntu)
Assignee: (unassigned) => Aaron Rainbolt (arraybolt3)
** Changed in: glib2.0 (Ubuntu)
Status: Confirmed => Invalid
--
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to glib2.0 in Ubuntu.
I believe we have a working patch for the problem. A quick summary of
what it looks like is happening:
Openbox maintains a list of windows organized by stacking order from
highest to lowest. This is a doubly-linked list.
There is a function in Openbox called client_calc_layer that uses a
pointer
This bug has been reported on the Ubuntu ISO testing tracker.
A list of all reports related to this bug can be found here:
https://iso.qa.ubuntu.com/qatracker/reports/bugs/2011751
** Tags added: iso-testing
--
You received this bug notification because you are a member of Desktop
Packages,
After discussion with the Security Team, we do not believe this is a
Security bug and that there currently is no need to keep it private as
there is other public discourse via other related bugs on other trackers
and here in Launchpad with other bugs (marked as dupes of this or such).
**
20 matches
Mail list logo
