[SRU]pcmanfm crashed with SIGSEGV in fm_nav_history_get_cur()

Bug #853507 reported by Julien Lavergne
26
This bug affects 3 people
Affects Status Importance Assigned to Milestone
PCManFM
Unknown
Unknown
pcmanfm (Ubuntu)
Fix Released
Medium
Unassigned
Oneiric
Won't Fix
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

Revision history for this message
Julien Lavergne (gilir) wrote :
Revision history for this message
Apport retracing service (apport) 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

Revision history for this message
Apport retracing service (apport) wrote : Stacktrace.txt
Revision history for this message
Apport retracing service (apport) wrote : ThreadStacktrace.txt
Changed in pcmanfm (Ubuntu):
importance: Undecided → Medium
tags: removed: need-amd64-retrace
Revision history for this message
Julien Lavergne (gilir) wrote : Re: pcmanfm crashed with SIGSEGV in fm_nav_history_get_cur()

Forwarded upstream.

visibility: private → public
Changed in pcmanfm (Ubuntu):
status: New → Triaged
Revision history for this message
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)
Changed in pcmanfm (Ubuntu Oneiric):
assignee: nobody → Vibhav Pant (vibhavp)
Revision history for this message
Vibhav Pant (vibhavp) wrote :
Vibhav Pant (vibhavp)
Changed in pcmanfm (Ubuntu Oneiric):
status: New → In Progress
Revision history for this message
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
Revision history for this message
Stefano Rivera (stefanor) wrote :

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

Vibhav Pant (vibhavp)
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)
description: updated
Revision history for this message
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
Revision history for this message
Sebastien Bacher (seb128) wrote :
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

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
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote : Verification still 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.

Revision history for this message
jerrylamos (jerrylamos) wrote :

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

Revision history for this message
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
Revision history for this message
Scott Kitterman (kitterman) wrote :

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

Revision history for this message
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
Revision history for this message
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
Revision history for this message
Rolf Leggewie (r0lf) wrote :

oneiric has seen the end of its life and is no longer receiving any updates. Marking the oneiric task for this ticket as "Won't Fix".

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

Other bug subscribers

Remote bug watches

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