Crash when pressing LMB+RMB

Bug #232522 reported by Homersp
8
Affects Status Importance Assigned to Milestone
Nautilus
Fix Released
Critical
nautilus (Ubuntu)
Fix Released
Medium
Ubuntu Desktop Bugs
Hardy
Fix Released
Medium
Ubuntu Desktop Bugs

Bug Description

Binary package hint: nautilus

Using Ubuntu Hardy 8.04 with Nautilus 1:2.22.2-0ubuntu6 pressing LMB (Left mouse button) and quickly afterwards RMB (Right mouse button) on a directory causes a Nautilus Segmentation fault and a system hang, only CTRL+ALT+Backspace lets me get out of it. The mouse pointer changes to the hand with a plus sign (indicating that Nautilus wants to copy something) when the hang occurs. The expected behavior would be that the directory is opened.

Since the system hangs I was not able to retrieve a proper backtrace, but I attached the parts I was able to get.

It may take a few retries before it crashes, but it certainly is a bug.

Revision history for this message
Homersp (maa-tillman) wrote :
Revision history for this message
Pedro Villavicencio (pedro) wrote :

Thanks for your bug report. Can you send the whole stacktrace as documented at http://wiki.ubuntu.com/DebuggingProgramCrash. This will greatly help us in tracking down your problem.

Changed in nautilus:
assignee: nobody → desktop-bugs
importance: Undecided → Medium
status: New → Incomplete
Revision history for this message
Homersp (maa-tillman) wrote :
Revision history for this message
Homersp (maa-tillman) wrote :

I was able to get a full backtrace too, it's attached.

Revision history for this message
A. Walton (awalton) wrote :

Absolutely bizarre crash. Are you using compositing (Compiz/Metacity with the Compositor enabled/some other compositing WM)? Did you perhaps turn it off/on at some point while Nautilus was running? Maybe it crashed and caused us to crash too?

The fix is simple enough and I'll try to get it in Bugzilla upstream in just a minute, we're not handling the null pixmap case sensibly there (can't create a cairo context for a null pixbuf, setting an operator on a null cairo context crashes). Thanks for taking the time to get the better stacktrace.

A. Walton (awalton)
Changed in nautilus:
status: Incomplete → New
Revision history for this message
Homersp (maa-tillman) wrote :

I do use Compiz, however I have not turned it off since I installed it.

I first noticed the crash when I was going to enter a directory and accidentally pressed LMB+RMB, I do that quite alot actually (I guess my mouse is too sensitive). It can get quite annoying, even though it doesn't happen very often. So thanks for taking the time to read through my bug report.

Changed in nautilus:
status: Unknown → In Progress
Revision history for this message
Homersp (maa-tillman) wrote :

Your patch seems to have fixed it. Guess it just was a small coding error (you should always check for NULL pointers).

This could probably be closed now.

Changed in nautilus:
status: New → Triaged
Changed in nautilus:
status: In Progress → Fix Released
Revision history for this message
Pedro Villavicencio (pedro) wrote :

fixed upstream, thanks for reporting.

Changed in nautilus:
status: Triaged → Fix Committed
Changed in nautilus:
assignee: nobody → desktop-bugs
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted into -proposed, please test and give feedback here

Changed in nautilus:
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nautilus - 1:2.23.2-0ubuntu1

---------------
nautilus (1:2.23.2-0ubuntu1) intrepid; urgency=low

  * New upstream versions:
    - Increase default thumbnailing size limit
    - Various fixes to keyboard handling in icon and list view (lp: #64289)
    - Use description from gio as window title
    - Fix double click behaviour in list view (lp: #120746)
    - Remember windows size when closed while maximized (lp: #137479)
    - Always use single click policy in the places sidebar
    - Fix selection handling after canceling delete operation
    - Improve autorun behaviour
    - Improve internal error handling
    - Fix adding emblems (lp: #224162)
    - Fix gconf preferences description
    - Improve list column page indentation
    - Prevent recursive move/copy into itself (lp: #232190)
    - Show mount dialog for already mounted shares too
    - Prevent autogenerated file names from overflowing the max filename length
    - Use ellipsis as per HIG
    - Fix leaks
    - Fix crashes (lp: #232522)
    - Build fixes
    - Translation updates
    - Support new gnome-session autostart semantics.
    - Unify session save/restore routines with "--restart" and "--load-session"
    - Allow to invert selection (lp: #18636)
    - Do not automount mount points inside hidden hierarchies
    - Add NautilusFileInfo APIs: can_write
    - Add initial Gtk-doc support.
    - Remove some dead code.
  * Sync with Debian
  * debian/control.in:
    - Build-Depends on liblaunchpad-integration-dev
    - don't build using beagle and tracker
    - don't require desktop-base and libesd
    - nautilus Recommends gnome-app-install
    - nautilus-data Replaces nautilus (<< 2.16) for dapper upgrades
    - nautilus Suggests gamin before fam
    - updated maintainer information
  * debian/mount-archive.desktop,
    debian/mount-archive.desktop.in,
    debian/nautilus.install:
    - new nautilus context menu item to allow mounting zip, iso, etc
  * debian/nautilus.1:
    - use correct command
  * debian/nautilus-data.gconf-schemas:
    - change the default desktop configuration.
  * debian/patches/01_lpi.patch:
    - launchpad changes.
  * debian/patches/02_autoconf.patch:
    - configure update.
  * debian/patches/03_menu_entry.patch:
    - change the menu item category.
  * debian/patches/05_places-sidebar_single-click.patch:
    - dropped, the new version has a similar change
  * debian/patches/08_display_mimetype_warning.patch:
    - new "preferences/display_mimetype_warning" gconf key which allow to use
      or not the mismatching mimetype warning dialog (Ubuntu: #19101)
  * debian/patches/10_rename_desktop.patch:
    - make renaming of a .desktop work with the language packs too
  * debian/patches/15_translations_list_update.patch:
    - list the new desktop as translatable
  * debian/patches/80_suppress_umount_in_ltsp.patch:
    - don't list unmount and eject actions on ltsp clients
  * debian/patches/81_gnome-app-install.patch:
    - Invoke gnome-app-install when no application known for file.
  * debian/rules:
    - updated shlibs version

 -- Sebastien Bacher <email address hidden> Tue, 03 Jun 2008 11:13:32 +0200

Changed in nautilus:
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

With the new version I tried the LMB-RMB combination several dozen times, and it never crashed. Also, the package is generally behaving nicely, no apparent regressions.

Revision history for this message
Martin Pitt (pitti) wrote :

Copied to hardy-updates.

Changed in nautilus:
status: Fix Committed → Fix Released
Changed in nautilus:
importance: Unknown → Critical
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.