unity8 crashed with SIGSEGV in UnityMenuModelPrivate::itemState()

Bug #1236249 reported by Jean-Baptiste Lallement
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Bluetooth Menu
Fix Released
High
Charles Kerr
indicator-bluetooth (Ubuntu)
Fix Released
Undecided
Unassigned
Saucy
Fix Released
Undecided
Unassigned
ubuntu-system-settings (Ubuntu)
Fix Released
High
Charles Kerr
Saucy
Fix Released
High
Charles Kerr
unity8 (Ubuntu)
Fix Released
High
Nick Dedekind
Saucy
Fix Released
High
Nick Dedekind

Bug Description

TEST CASE:
1. Open BT indicator and disable BT
2. Open system-settings
3. Select BT
4. Enable/Disable BT several times (max 2 or 3)
=> indicator doesn’t appear and eventually u-s-s and unity8 crashed

ProblemType: Crash
DistroRelease: Ubuntu 13.10
Package: unity8 7.82+13.10.20131005-0ubuntu1
Uname: Linux 3.4.0-3-mako armv7l
ApportVersion: 2.12.5-0ubuntu1
Architecture: armhf
CrashCounter: 1
Date: Mon Oct 7 09:15:09 2013
ExecutablePath: /usr/bin/unity8
ExecutableTimestamp: 1380974656
InstallationDate: Installed on 2013-10-05 (1 days ago)
InstallationMedia: Ubuntu Saucy Salamander (development branch) - armhf (20131005)
MarkForUpload: True
ProcCmdline: unity8
ProcCwd: /home/phablet
Signal: 6
SourcePackage: unity8
StacktraceTop:
 ?? ()
 UnityMenuModelPrivate::itemState(_GtkMenuTrackerItem*) () from /usr/lib/arm-linux-gnueabihf/libqmenumodel.so.0
 UnityMenuModel::data(QModelIndex const&, int) const () from /usr/lib/arm-linux-gnueabihf/libqmenumodel.so.0
 UnityMenuModel::get(int, QByteArray const&) () from /usr/lib/arm-linux-gnueabihf/libqmenumodel.so.0
 RootActionState::updateActionState() () from /usr/lib/arm-linux-gnueabihf/unity8/qml/Unity/Indicators/libIndicatorsQml.so
Title: unity8 crashed with SIGSEGV in UnityMenuModelPrivate::itemState()
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm autopilot cdrom dialout dip nopasswdlogin plugdev sudo tty video

Related branches

Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :
Revision history for this message
Apport retracing service (apport) wrote : This bug is a duplicate

Thank you for taking the time to report this crash and helping to make this software better. This particular crash has already been reported and is a duplicate of bug #1234737, so is being marked as such. Please look at the other bug report to see if there is any missing information that you can provide, or to see if there is a workaround for the bug. Additionally, any further discussion regarding the bug should occur in the other report. Please continue to report any other bugs you may find.

tags: removed: need-armhf-retrace
Changed in unity8 (Ubuntu):
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Michal Hruby (mhr3) wrote :

I got a slightly better stacktrace than what's available in the linked bug, see attachment.

Revision history for this message
Charles Kerr (charlesk) wrote :

assigning to lars as per discussion in #ubuntu-touch

Changed in unity8 (Ubuntu):
assignee: nobody → Lars Uebernickel (larsu)
Revision history for this message
Lars Karlitski (larsu) wrote :

Turns out this is a bug in unity8 after all. The UnityMenuModel is given a pointer to an ActionStateParser that is invalidated without telling the model.

The attached branch fixes the problem half-way: unity8 doesn't crash anymore when enabling and disabling bluetooth successively. However, the bluetooth indicator doesn't show up after the first disable / enable cylce. Nick, could you please have a look at that?

Changed in unity8 (Ubuntu):
assignee: Lars Uebernickel (larsu) → Nick Dedekind (nick-dedekind)
Revision history for this message
Nick Dedekind (nick-dedekind) wrote :

@larsu.
It seems as though turning bluetooth on after it's been turned off doesn't actually work.

phablet@ubuntu-phablet:~$ hciconfig
hci0: Type: BR/EDR Bus: UART
 BD Address: 38:0A:94:2F:17:5D ACL MTU: 1021:8 SCO MTU: 64:1
 UP RUNNING PSCAN
 RX bytes:893 acl:0 sco:0 events:37 errors:0
 TX bytes:936 acl:0 sco:0 commands:37 errors:0

* DISABLED IN SETTINGS APP.

phablet@ubuntu-phablet:~$ hciconfig
hci0: Type: BR/EDR Bus: UART
 BD Address: 38:0A:94:2F:17:5D ACL MTU: 1021:8 SCO MTU: 64:1
 DOWN
 RX bytes:900 acl:0 sco:0 events:38 errors:0
 TX bytes:949 acl:0 sco:0 commands:39 errors:0

* ENABLED IN SETTINGS APP.

phablet@ubuntu-phablet:~$ hciconfig
hci0: Type: BR/EDR Bus: UART
 BD Address: 38:0A:94:2F:17:5D ACL MTU: 1021:8 SCO MTU: 64:1
 DOWN INIT RUNNING
 RX bytes:900 acl:0 sco:0 events:38 errors:0
 TX bytes:961 acl:0 sco:0 commands:42 errors:0

* A FEW SECONDS LATER.

phablet@ubuntu-phablet:~$ hciconfig
hci0: Type: BR/EDR Bus: UART
 BD Address: 38:0A:94:2F:17:5D ACL MTU: 1021:8 SCO MTU: 64:1
 DOWN
 RX bytes:907 acl:0 sco:0 events:38 errors:0
 TX bytes:999 acl:0 sco:0 commands:49 errors:0

Revision history for this message
Nick Dedekind (nick-dedekind) wrote :

Running "rfkill unblock bluetooth" has same effect. Looks like it tries to enable, then fails.

Charles Kerr (charlesk)
Changed in ubuntu-system-settings (Ubuntu):
assignee: nobody → Charles Kerr (charlesk)
Changed in indicator-bluetooth:
assignee: nobody → Charles Kerr (charlesk)
Revision history for this message
Charles Kerr (charlesk) wrote :

The ubuntu-system-settings and indicator-bluetooth branches I've linked here attempt to handle things more gracefully whenever indicator-bluetooth can't work with /dev/rfkill -- it attempts to fall back to just calling org.bluez.Adapter::SetProperties("Powered", on/off)

This is an improvement; unfortunately, it doesn't seem to be a complete fix yet as powering on/off on the desktop is unreliable in the latest nightlies, even when using bluez' command-line utilities to power off & on again. I've filed that separate issue as bug #1238672

Michael Terry (mterry)
Changed in unity8 (Ubuntu):
status: Confirmed → Fix Committed
Charles Kerr (charlesk)
Changed in indicator-bluetooth:
status: New → In Progress
Changed in ubuntu-system-settings (Ubuntu):
status: New → In Progress
importance: Undecided → High
Changed in indicator-bluetooth:
importance: Undecided → High
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-system-settings - 0.1+13.10.20131015-0ubuntu1

---------------
ubuntu-system-settings (0.1+13.10.20131015-0ubuntu1) saucy; urgency=low

  [ Charles Kerr ]
  * In the battery plugin, use indicator-bluetooth's "bluetooth-
    supported" action state to know whether or not to show the bluetooth
    toggle. (LP: #1233628)
  * In the bluetooth plugin, handle bluetooth adapters
    appearing/disappearing on the bus. (LP: #1236249)

  [ Sebastien Bacher ]
  * update: get the current build number, use it in the os version
    string. (LP: #1233682)
  * update: tweak a bit the UI, we don't have a description for the
    updates so there is no need to keep the stacked UI, we can use one
    line. Copy the icon in the source as well since it's not installed
    on touch. (LP: #1233682)
  * sound: use the new sounds sets from ubuntu-touch-sounds.

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 456
 -- Ubuntu daily release <email address hidden> Tue, 15 Oct 2013 04:19:09 +0000

Changed in ubuntu-system-settings (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:indicator-bluetooth at revision 70, scheduled for release in indicator-bluetooth, milestone Unknown

Changed in indicator-bluetooth:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in indicator-bluetooth (Ubuntu):
status: New → Confirmed
Changed in unity8 (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package indicator-bluetooth - 0.0.6+13.10.20131016-0ubuntu1

---------------
indicator-bluetooth (0.0.6+13.10.20131016-0ubuntu1) saucy; urgency=low

  [ Charles Kerr ]
  * When the user chooses to toggle bluetooth on or off, if /dev/rfkill
    isn't available, fall back to toggling org.bluez.Adapter's Powered
    property. (LP: #1236249, #1230275)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 70
 -- Ubuntu daily release <email address hidden> Wed, 16 Oct 2013 02:34:31 +0000

Changed in indicator-bluetooth (Ubuntu Saucy):
status: Confirmed → Fix Released
Charles Kerr (charlesk)
Changed in indicator-bluetooth:
status: Fix Committed → Fix Released
tags: added: has-test
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.