Merge lp:~mitya57/nautilus/ubuntu-fix-lp-795708 into lp:~ubuntu-desktop/nautilus/ubuntu

Proposed by Dmitry Shachnev
Status: Merged
Merged at revision: 214
Proposed branch: lp:~mitya57/nautilus/ubuntu-fix-lp-795708
Merge into: lp:~ubuntu-desktop/nautilus/ubuntu
Diff against target: 64 lines (+42/-0)
3 files modified
debian/changelog (+8/-0)
debian/patches/18_fix_crash_in_get_current_uri.patch (+33/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~mitya57/nautilus/ubuntu-fix-lp-795708
Reviewer Review Type Date Requested Status
Martin Pitt Approve
Review via email: mp+84370@code.launchpad.net

Description of the change

This branch contains ported patch from https://bugzilla.gnome.org/show_bug.cgi?id=652320.
This patch fixes the crash for me.

Now that patch is in Git, so you can add a "git_" prefix if you want.

To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

Thanks! I'll also upload this as SRU.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2011-12-02 14:58:13 +0000
3+++ debian/changelog 2011-12-03 14:51:59 +0000
4@@ -1,3 +1,11 @@
5+nautilus (1:3.2.1-0ubuntu6) precise; urgency=low
6+
7+ * debian/patches/18_fix_crash_in_get_current_uri.patch:
8+ - Upstream patch to fix crash in nautilus_window_slot_get_current_uri()
9+ (LP: #795708)
10+
11+ -- Dmitry Shachnev <mitya57@gmail.com> Fri, 02 Dec 2011 20:37:11 +0400
12+
13 nautilus (1:3.2.1-0ubuntu5) precise; urgency=low
14
15 [ Didier Roche ]
16
17=== added file 'debian/patches/18_fix_crash_in_get_current_uri.patch'
18--- debian/patches/18_fix_crash_in_get_current_uri.patch 1970-01-01 00:00:00 +0000
19+++ debian/patches/18_fix_crash_in_get_current_uri.patch 2011-12-03 14:51:59 +0000
20@@ -0,0 +1,33 @@
21+Description: Fix crash in nautilus_window_slot_get_current_uri()
22+Author: Cosimo Cecchi <cosimoc@gnome.org>
23+Bug: https://bugzilla.gnome.org/show_bug.cgi?id=652320
24+Bug-Ubuntu: https://bugs.launchpad.net/bugs/795708
25+
26+Index: ubuntu/src/nautilus-window.c
27+===================================================================
28+--- ubuntu.orig/src/nautilus-window.c 2011-12-02 20:37:06.156934437 +0400
29++++ ubuntu/src/nautilus-window.c 2011-12-02 20:36:28.325575521 +0400
30+@@ -474,8 +474,10 @@
31+ {
32+ DEBUG ("Destroying sidebar");
33+
34+- gtk_widget_destroy (GTK_WIDGET (window->details->sidebar));
35+- window->details->sidebar = NULL;
36++ if (window->details->sidebar != NULL) {
37++ gtk_widget_destroy (GTK_WIDGET (window->details->sidebar));
38++ window->details->sidebar = NULL;
39++ }
40+ }
41+
42+ void
43+@@ -721,8 +723,8 @@
44+
45+ DEBUG ("Destroying window");
46+
47+- window->details->content_paned = NULL;
48+- window->details->split_view_hpane = NULL;
49++ /* close the sidebar first */
50++ nautilus_window_tear_down_sidebar (window);
51+
52+ /* close all panes safely */
53+ panes_copy = g_list_copy (window->details->panes);
54
55=== modified file 'debian/patches/series'
56--- debian/patches/series 2011-12-02 14:52:42 +0000
57+++ debian/patches/series 2011-12-03 14:51:59 +0000
58@@ -14,6 +14,7 @@
59 15_use-ubuntu-help.patch
60 16_run_in_gnome_unity.patch
61 17_dont_allow_new_tab_on_desktop.patch
62+18_fix_crash_in_get_current_uri.patch
63 git-0001-Workaround-focus-issues-on-typeahead.patch
64 git_sideplace_sorting.patch
65

Subscribers

People subscribed via source and target branches

to all changes: