From f605777d9739d2f8873b9ab65929e8aee1ebb64e Mon Sep 17 00:00:00 2001 From: Martin Szulecki Date: Fri, 10 Jul 2009 19:02:47 +0200 Subject: Make sure to also copy string/data buffers when cloning the node data struct --- src/plist.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/src/plist.c b/src/plist.c index c205d46..8c326af 100644 --- a/src/plist.c +++ b/src/plist.c @@ -148,6 +148,19 @@ static void plist_copy_node(GNode * node, gpointer parent_node_ptr) assert(data); // plist should always have data memcpy(newdata, data, sizeof(struct plist_data_s)); + + plist_type node_type = plist_get_node_type(node); + if (node_type == PLIST_DATA || node_type == PLIST_STRING) { + switch (node_type) { + case PLIST_DATA: + newdata->buff = (uint8_t *) malloc(data->length); + memcpy(newdata->buff, data->buff, data->length); + case PLIST_STRING: + newdata->strval = strdup((char *) data->strval); + default: + break; + } + } newnode = plist_new_node(newdata); if (*(plist_t*)parent_node_ptr) { -- cgit v1.1-32-gdbae