Browse files

Fix: xml: Mark the parents of a partial replace as dirty

  • Loading branch information...
1 parent 7fad230 commit 007117a2e7a01654273a7f6258e62faa2c8422a6 @beekhof committed Mar 4, 2014
Showing with 3 additions and 0 deletions.
  1. +2 −0 lib/common/xml.c
  2. +1 −0 tools/regression.sh
View
2 lib/common/xml.c
@@ -4376,6 +4376,8 @@ replace_xml_child(xmlNode * parent, xmlNode * child, xmlNode * update, gboolean
xmlDoc *doc = tmp->doc;
xmlNode *old = xmlReplaceNode(child, tmp);
+ __xml_node_dirty(tmp);
+
free_xml(old);
xmlDocSetRootElement(doc, NULL);
xmlFreeDoc(doc);
View
1 tools/regression.sh
@@ -480,6 +480,7 @@ EOF
CIB_user=root CIB_file=/tmp/$$.haxor.xml CIB_shadow="" cibadmin -Ql
export CIB_user=niceguy
+ # Make sure we're rejecting things for the right reasons
export PCMK_trace_functions=__xml_acl_check,__xml_acl_post_process
desc="$CIB_user: Replace - remove acls"
cmd="cibadmin --replace --xml-file /tmp/$$.haxor.xml -V"

0 comments on commit 007117a

Please sign in to comment.