Ubuntu One Client

Merge lp:~dobey/ubuntuone-client/fix-570261 into lp:ubuntuone-client

  1. fix-570261
  2. Merge into trunk
Proposed by dobey
Status: Merged
Approved by: Tim Cole
Approved revision: 511
Merged at revision: not available
Proposed branch: lp:~dobey/ubuntuone-client/fix-570261
Merge into: lp:ubuntuone-client
Diff against target: 16 lines (+4/-2)
1 file modified
nautilus/ubuntuone-nautilus.c (+4/-2)
To merge this branch: bzr merge lp:~dobey/ubuntuone-client/fix-570261
Reviewer Review Type Date Requested Status
Tim Cole (community) Approve
Rodrigo Moya (community) Approve
Review via email: mp+24157@code.launchpad.net

Commit message

As path can be NULL, guard against calling hash_remove with it

To post a comment you must log in.
Revision history for this message
Rodrigo Moya (rodrigo-moya) wrote :

looks good

review: Approve
Revision history for this message
Tim Cole (tcole) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'nautilus/ubuntuone-nautilus.c'
--- nautilus/ubuntuone-nautilus.c 2010-04-16 16:23:19 +0000
+++ nautilus/ubuntuone-nautilus.c 2010-04-26 19:24:21 +0000
@@ -1475,8 +1475,10 @@
1475 gchar * path;1475 gchar * path;
14761476
1477 path = g_hash_table_lookup (hash, "path");1477 path = g_hash_table_lookup (hash, "path");
1478 g_hash_table_remove (uon->udfs, path);1478 if (path != NULL) {
1479 ubuntuone_nautilus_reset_emblem (uon, path);1479 g_hash_table_remove (uon->udfs, path);
1480 ubuntuone_nautilus_reset_emblem (uon, path);
1481 }
1480}1482}
14811483
1482static void ubuntuone_nautilus_udf_error (DBusGProxy * proxy,1484static void ubuntuone_nautilus_udf_error (DBusGProxy * proxy,

Subscribers

People subscribed via source and target branches