'content' is declared as char content[] so if char is signed, all characters with the high bit set will be negative so they will be < 0x20. This means the code ...

The following test case unexpectedly stops with an exception:

include

include <plist/plist++.h>

int main()

{

PList::Dictionary a;

PList::String b("Hello");

P...

libxml2 can be told to use malloc/free/... using xmlMemSetup, but using that wouldn't be so nice if the app wants to use libxml2 on its own.

Sorry for the spam, I really meant this patch :)

Here is an alternate patch that keeps the g_node_unlink. It won't solve the issue you describe since if one keep pointers "inside" the tree and frees the memory...

I realized this morning that ssl_certificate should be set to NULL in lockdownd_client_new.

First patch fixes invalid read from valgrind
Second patch fixes memory leaks