Ubuntu

[SRU]pcmanfm crashed with SIGSEGV in fm_nav_history_get_cur()

Reported by Julien Lavergne on 2011-09-18
26
This bug affects 3 people
Affects Status Importance Assigned to Milestone
PCManFM
Unknown
Unknown
pcmanfm (Ubuntu)
Medium
Unassigned
Oneiric
Low
Unassigned

Bug Description

[Impact]
The bug can only be reproduced when refreshing
[Test Case]
On Lubuntu 11.10, Open pcmanfm
Click on Show -> Refresh (F5)
Notice how pcmanfm crashes
[Development Fix]
https://launchpadlibrarian.net/107451095/pcmanfm_0.9.9-0ubuntu1.1.debdiff
[Stable Fix]
Same as Development Fix
[Regression Potential]
Low
[Original Report]
Using Lubuntu up-to-date (2011-09-18)

Crash on clicking on Show => Refresh (F5)

ProblemType: Crash
DistroRelease: Ubuntu 11.10
Package: pcmanfm 0.9.9-0ubuntu1
ProcVersionSignature: Ubuntu 3.0.0-10.16-generic 3.0.4
Uname: Linux 3.0.0-10-generic x86_64
ApportVersion: 1.23-0ubuntu1
Architecture: amd64
Date: Sun Sep 18 23:51:52 2011
ExecutablePath: /usr/bin/pcmanfm
InstallationMedia: Ubuntu 11.04 "Natty Narwhal" - Beta amd64 (20110330)
Pref_Config_System_lubuntu:
 [Preferred Applications]
 WebBrowser=firefox.desktop
 MailClient=
ProcCmdline: pcmanfm --desktop --profile lubuntu
ProcEnviron:
 LANGUAGE=fr_FR.UTF-8
 LANG=fr_FR.UTF-8
 LC_MESSAGES=fr_FR.UTF-8
 SHELL=/bin/bash
SegvAnalysis:
 Segfault happened at: 0x7ffe4c4545b0 <fm_nav_history_get_cur>: mov 0x30(%rdi),%rax
 PC (0x7ffe4c4545b0) ok
 source "0x30(%rdi)" (0x00000030) not located in a known VMA region (needed readable region)!
 destination "%rax" ok
SegvReason: reading NULL VMA
Signal: 11
SourcePackage: pcmanfm
StacktraceTop:
 fm_nav_history_get_cur (nh=0x0) at base/fm-nav-history.c:76
 on_folder_view_loaded (view=0x1c77020, path=<optimized out>, page=0x1cc12b0) at tab-page.c:253
 g_closure_invoke (closure=0x1cb3140, return_value=0x0, n_param_values=2, param_values=0x1cbeb60, invocation_hint=<optimized out>) at /build/buildd/glib2.0-2.29.90/./gobject/gclosure.c:774
 signal_emit_unlocked_R (node=<optimized out>, detail=0, instance=0x1c77020, emission_return=0x0, instance_and_params=0x1cbeb60) at /build/buildd/glib2.0-2.29.90/./gobject/gsignal.c:3272
 g_signal_emit_valist (instance=<optimized out>, signal_id=<optimized out>, detail=<optimized out>, var_args=0x7fff50d7f5f8) at /build/buildd/glib2.0-2.29.90/./gobject/gsignal.c:3003
Title: pcmanfm crashed with SIGSEGV in fm_nav_history_get_cur()
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm admin cdrom dialout dip fax floppy fuse lpadmin plugdev sambashare tape vboxusers video

Julien Lavergne (gilir) wrote :

StacktraceTop:
 fm_nav_history_get_cur (nh=0x0) at base/fm-nav-history.c:76
 on_folder_view_loaded (view=0x1c77020, path=<optimized out>, page=0x1cc12b0) at tab-page.c:253
 g_closure_invoke (closure=0x1cb3140, return_value=0x0, n_param_values=2, param_values=0x1cbeb60, invocation_hint=<optimized out>) at /build/buildd/glib2.0-2.29.92/./gobject/gclosure.c:774
 signal_emit_unlocked_R (node=<optimized out>, detail=0, instance=0x1c77020, emission_return=0x0, instance_and_params=0x1cbeb60) at /build/buildd/glib2.0-2.29.92/./gobject/gsignal.c:3272
 g_signal_emit_valist (instance=<optimized out>, signal_id=<optimized out>, detail=<optimized out>, var_args=0x7fff50d7f5f8) at /build/buildd/glib2.0-2.29.92/./gobject/gsignal.c:3003

Changed in pcmanfm (Ubuntu):
importance: Undecided → Medium
tags: removed: need-amd64-retrace

Forwarded upstream.

visibility: private → public
Changed in pcmanfm (Ubuntu):
status: New → Triaged
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pcmanfm - 0.9.10-0ubuntu2

---------------
pcmanfm (0.9.10-0ubuntu2) precise; urgency=low

  * debian/patches:
   - Rename 91-integrate-lxshortcut.patch to 02-integrate-lxshortcut.patch.
   - 03_avoid_free.patch: Avoid a not necessary free (LP: #963605, #833987)
   - 90_french_desktop.patch: From upstream, fix the French name in the
     desktop file.
   - 101_crash_on_button_press_event.patch &
     103_drag_desktop_item.patch &
     104_fix_crash_desktop.patch :
     Add several patches from Vadim Ushakov to fix potential crashes.
   - 102_crash_on_refresh.patch: Fix crash on refresh (LP: #853507).
 -- Julien Lavergne <email address hidden> Tue, 03 Apr 2012 22:35:05 +0200

Changed in pcmanfm (Ubuntu):
status: Triaged → Fix Released
Vibhav Pant (vibhavp) on 2012-06-03
Changed in pcmanfm (Ubuntu Oneiric):
assignee: nobody → Vibhav Pant (vibhavp)
Vibhav Pant (vibhavp) wrote :
Vibhav Pant (vibhavp) on 2012-06-04
Changed in pcmanfm (Ubuntu Oneiric):
status: New → In Progress
Sebastien Bacher (seb128) wrote :

Thank you for your work, could you follow the SRU guidelines though (https://wiki.ubuntu.com/StableReleaseUpdates), the bug needs a rational, testcase and regression potential definition. Is it really worth SRUing fixed for Oneiric when there is a newer version out which is a lts and when the bug has no duplicate?

Changed in pcmanfm (Ubuntu Oneiric):
importance: Undecided → Low
Stefano Rivera (stefanor) wrote :

Unsubscribing ubuntu-sponsors. Please re-subscribe the ubuntu-sponsors team when this is ready for review again.

Vibhav Pant (vibhavp) on 2012-06-10
summary: - pcmanfm crashed with SIGSEGV in fm_nav_history_get_cur()
+ [SRU]pcmanfm crashed with SIGSEGV in fm_nav_history_get_cur()
Vibhav Pant (vibhavp) on 2012-06-10
description: updated
Julien Lavergne (gilir) wrote :

Thanks for your work on this. The initial debdiff had a typo which cause a FTBFS. I corrected the patch, and I'll upload the package in onieric-proposed.

description: updated
Changed in pcmanfm (Ubuntu Oneiric):
status: In Progress → Confirmed
assignee: Vibhav Pant (vibhavp) → nobody
Sebastien Bacher (seb128) wrote :

Hello Julien, or anyone else affected,

Accepted pcmanfm into oneiric-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/pcmanfm/0.9.9-0ubuntu1.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please change the bug tag from verification-needed to verification-done. If it does not, change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in pcmanfm (Ubuntu Oneiric):
status: Confirmed → Fix Committed
tags: added: verification-needed

The fix for the this bug has been awaiting testing feedback in the -proposed repository for oneiric for more than 90 days. Please test this fix and update the bug appropriately with the results. In the event that the fix for this bug is still not verified 15 days from now the package will be removed from the -proposed repository.

jerrylamos (jerrylamos) wrote :

Let me give a try - I've just installed pcmanfm on raring.

Phill Whiteside (phillw) wrote :

Installing lubuntu-11.10 (amd64) which has has pcmanfm 0.9.9.0-ubuntu1 into KVM, I cannot get the original bug reproduced.

tags: added: removal-candidate
tags: added: verification-done
removed: verification-needed
Scott Kitterman (kitterman) wrote :

Adolfo: Does that mean you were able to recreate the original problem and verify the new package fixes it?

Brian Murray (brian-murray) wrote :

I've switched the tag back to verification-needed as the verification of this is incomplete and inconclusive.

tags: added: verification-needed
removed: verification-done
Bryan Quigley (bryanquigley) wrote :

Was just going to close it, getting ready for 11.10 EOL, is anyone still looking for this update? (< 2 months of support left).

Changed in pcmanfm (Ubuntu Oneiric):
status: Fix Committed → Incomplete
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

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