Nautilus crash when IM preedit

Bug #625696 reported by Xhacker Liu
158
This bug affects 25 people
Affects Status Importance Assigned to Milestone
Nautilus
Fix Released
Critical
ibus (Ubuntu)
Invalid
High
Unassigned
Maverick
Invalid
High
Unassigned
nautilus (Ubuntu)
Fix Released
High
Canonical Desktop Team
Maverick
Fix Released
High
Canonical Desktop Team

Bug Description

Binary package hint: nautilus

When ibus is running...

1.open a folder.
2.input some letters. (related folder should be highlighted)
3.then nautilus crash...

I'm using maverick, nautilus 2.31.90-0ubuntu2, ibus 1.3.7-1ubuntu3.

If I quit ibus, everything will be good, the bug is not exist. Thanks.

Tags: patch

Related branches

Revision history for this message
Dsegun Ludice (xrandsegunludice) wrote :

I can confirm it.

Changed in ibus (Ubuntu):
status: New → Confirmed
Changed in nautilus (Ubuntu):
status: New → Confirmed
Changed in nautilus (Ubuntu):
importance: Undecided → Low
status: Confirmed → Invalid
Changed in ibus (Ubuntu):
importance: Undecided → High
milestone: none → ubuntu-10.10
Revision history for this message
Neo Nengrong Qu (qunengrong) wrote :

I encounter the same bug , using ubuntu 10.10 .........

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

I'm not as certain it's really a bug in ibus. The same search and typing in chinese characters (gibberish though, not going to give out any results) in List View yields no crash. On the other hand, I can reproduce this as soon as I set IBus as an input method and set it to something other than Off in Icon View or Compact View.

Revision history for this message
Fumihito YOSHIDA (hito) wrote :

backtrace of below handling with dbgsym packages.

 1) run gdb
 2) open nautilus
 3) gdb> attach nautilus's pid
 4) type "m" with ibus to nautilus's window.

Revision history for this message
IRIE Shinsuke (irie) wrote :

I found that Nautilus crashes also when using scim-bridge, so the bug might be in Nautilus.

Revision history for this message
IRIE Shinsuke (irie) wrote :

Sorry, I forgot to write the difference from the case using ibus.

When using scim-bridge, Nautilus crashes by starting preedit.

Revision history for this message
Fumihito YOSHIDA (hito) wrote :

Yet another Backtrace is here. (from dupes LP#630886, thanks Jesse)

https://bugs.launchpad.net/nautilus/+bug/630886/comments/7

Revision history for this message
IRIE Shinsuke (irie) wrote :

Umm ... even if using neither IBus nor SCIM, Nautilus crashes just type Ctrl+Shift+u.

Revision history for this message
IRIE Shinsuke (irie) wrote :

Oops, sorry, I posted a halfway comment by mistake.

Even if using neither IBus nor SCIM, Nautilus crashes by just typing Ctrl+Shift+u, which is a trigger of unicode input.

Revision history for this message
Jesse (sbjesse) wrote :

IRIE seems to be right. I am able to reproduce a backtrace following IRIE's description, as attached

Revision history for this message
Fumihito YOSHIDA (hito) wrote :

Hmm... I make diff of lucid/maverick. That includes some interested changes....

============================================================
$ diff -u nautilus-2.30.0/libnautilus-private/nautilus-icon-container.c
nautilus-2.31.92/libnautilus-private/nautilus-icon-container.c

@@ -4896,7 +4916,7 @@
  * callback.
  */
 static void
-nautilus_icon_container_search_preedit_changed (GtkIMContext *im_context,
+nautilus_icon_container_search_preedit_changed (GtkEntry *entry,
                                                NautilusIconContainer *container)
 {
        container->details->imcontext_changed = 1;
@@ -4940,7 +4960,7 @@

        nautilus_icon_container_search_dialog_hide (widget, container);

- if (event->window == GTK_LAYOUT (container)->bin_window) {
+ if (event->window == gtk_layout_get_bin_window (GTK_LAYOUT (container))) {
                button_press_event (GTK_WIDGET (container), event);
        }
============================================================

Changed in nautilus:
importance: Unknown → Critical
status: Unknown → New
Revision history for this message
Fumihito YOSHIDA (hito) wrote :

I test with attached patch, I could not reproduce problem. And, i put to my ppa for test purpose.
https://launchpad.net/~hito/+archive/ppa/+packages?field.name_filter=&field.status_filter=published&field.series_filter=maverick

Could you please test it?

This patch is simple, nautilus_icon_container_search_preedit_changed() rollbacks to lucid.
IMHO, maverick's that routine does not has im-context, but im-context is essential for pre-edit phase.

Fumihito YOSHIDA (hito)
summary: - Nautilus crash when ibus is running
+ Nautilus crash when IM preedit
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

I can confirm the patch appears to fix the issue, so I marked this bug Invalid for ibus and Triaged for nautilus. I applied the patch to a local copy of the maverick nautilus branch, which I just pushed to LP and should appear on this bug report shortly...

Changed in nautilus (Ubuntu Maverick):
status: Invalid → Triaged
importance: Low → High
Changed in ibus (Ubuntu Maverick):
status: Confirmed → Invalid
Changed in nautilus (Ubuntu Maverick):
milestone: none → ubuntu-10.10
Changed in nautilus (Ubuntu Maverick):
assignee: nobody → Canonical Desktop Team (canonical-desktop-team)
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Just pushed in the queue :)

Changed in nautilus (Ubuntu Maverick):
status: Triaged → Fix Committed
tags: added: patch
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nautilus - 1:2.31.92-0ubuntu2

---------------
nautilus (1:2.31.92-0ubuntu2) maverick; urgency=low

  * Patch typeahead search in icon/compact view to properly deal with input
    method changes (LP: #625696)
    - added debian/patches/18_preedit_use_imcontext.patch
    - updated debian/patches/series
 -- Mathieu Trudel-Lapierre <email address hidden> Wed, 22 Sep 2010 09:17:53 -0400

Changed in nautilus (Ubuntu Maverick):
status: Fix Committed → Fix Released
Changed in nautilus:
status: New → Fix Released
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.