Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Merge pull request #443 from yuusuke/fix_memory_leak

Fix: memory leak found by valgrind was corrected.
  • Loading branch information...
commit 9812e5b81fc3b8cff056593c65a67bb42d223b8a 2 parents c30f974 + 137b4aa
@beekhof beekhof authored
Showing with 7 additions and 1 deletion.
  1. +2 −0  cib/messages.c
  2. +5 −1 lib/common/xml.c
View
2  cib/messages.c
@@ -223,6 +223,8 @@ cib_process_ping(const char *op, int options, const char *section, xmlNode * req
crm_info("Reporting our current digest to %s: %s (%d)", host, digest, cs && cs->targets);
+ free(digest);
+
return pcmk_ok;
#endif
}
View
6 lib/common/xml.c
@@ -800,9 +800,10 @@ xml_create_patchset(int format, xmlNode *source, xmlNode *target, bool *config_c
}
if(patch && with_digest) {
- const char *digest = calculate_xml_versioned_digest(target, FALSE, TRUE, version);
+ char *digest = calculate_xml_versioned_digest(target, FALSE, TRUE, version);
crm_xml_add(patch, XML_ATTR_DIGEST, digest);
+ free(digest);
}
return patch;
}
@@ -1326,7 +1327,9 @@ __xml_find_path(xmlNode *top, const char *key)
free(remainder);
free(current);
+ free(section);
free(tag);
+ free(id);
return target;
}
@@ -1533,6 +1536,7 @@ xml_apply_patchset(xmlNode *xml, xmlNode *patchset, bool check_version)
crm_trace("Digest matched: expected %s, calculated %s", digest, new_digest);
}
free(new_digest);
+ free(version);
}
free_xml(old);
return rc;
Please sign in to comment.
Something went wrong with that request. Please try again.