fixup! OstreeMutableTree: Refactor: Add parent pointer

ostreedev · Jun 27, 2018 · 462d7c5 · 462d7c5
1 parent 3c091e4
commit 462d7c5
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/libostree/ostree-mutable-tree.c b/src/libostree/ostree-mutable-tree.c
@@ -115,6 +115,7 @@ insert_child_mtree (OstreeMutableTree *self, const gchar* name,
   g_assert_null (child->parent);
   g_hash_table_insert (self->subdirs, g_strdup (name), child);
   child->parent = self;
+  invalidate_contents_checksum (self);
 }
 
 static void
@@ -146,7 +147,7 @@ invalidate_contents_checksum (OstreeMutableTree *self)
       break;
 
     g_free (self->contents_checksum);
-    self->contents_checksum = NULL;
+    g_clear_pointer (self->contents_checksum, g_free);
     self = self->parent;
   }
 }
@@ -265,7 +266,6 @@ ostree_mutable_tree_ensure_dir (OstreeMutableTree *self,
   if (!ret_dir)
     {
       ret_dir = ostree_mutable_tree_new ();
-      invalidate_contents_checksum (self);
       insert_child_mtree (self, name, g_object_ref (ret_dir));
     }
 
@@ -347,7 +347,6 @@ ostree_mutable_tree_ensure_parent_dirs (OstreeMutableTree  *self,
       next = g_hash_table_lookup (subdir->subdirs, name);
       if (!next) 
         {
-          invalidate_contents_checksum (subdir);
           next = ostree_mutable_tree_new ();
           ostree_mutable_tree_set_metadata_checksum (next, metadata_checksum);
           insert_child_mtree (subdir, g_strdup (name), next);