[sr-dev] git:mariusbucur/conference: use xmlCopyNode instead of unlink because for now we want ALL states to be agregated into one conference-info document

admin at sip-router.org admin at sip-router.org
Thu Jul 22 15:09:23 CEST 2010


Module: sip-router
Branch: mariusbucur/conference
Commit: 11a720be7146daa7d38e8403b562a6fc4d8ce8d5
URL:    http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=11a720be7146daa7d38e8403b562a6fc4d8ce8d5

Author: mariusbucur <marius at marius-bucur.ro>
Committer: mariusbucur <marius at marius-bucur.ro>
Date:   Thu Jul 22 16:06:16 2010 +0300

use xmlCopyNode instead of unlink because for now we want ALL states to be agregated into one conference-info document

---

 modules_k/presence_conference/notify_body.c |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/modules_k/presence_conference/notify_body.c b/modules_k/presence_conference/notify_body.c
index 9e4e755..7992973 100644
--- a/modules_k/presence_conference/notify_body.c
+++ b/modules_k/presence_conference/notify_body.c
@@ -212,14 +212,10 @@ str* agregate_xmls(str* pres_user, str* pres_domain, str** body_array, int n)
 				  * would then be declared redundant (libxml unfortunately cannot remove 
 				  * namespaces)
 				  */
-				LM_ERR("[unlink node %p]\n", node->next);
- 				xmlUnlinkNode(node);//don't mess around with modifying an iterating list
-				LM_ERR("[after unlink %p]\n", node->next);
-				if(xmlAddChild(root_node, node) == NULL) {
+				if(xmlAddChild(root_node, xmlCopyNode(node, 1)) == NULL) {
 					LM_ERR("while adding child\n");
 					goto error;
 				}
-				LM_ERR("[after adding unlinked node %p]\n", node->next);
 			}
 		}
 		}




More information about the sr-dev mailing list