[sr-dev] git:3.1: textops(k): free only pkg body lumps for set_body()

Daniel-Constantin Mierla miconda at gmail.com
Tue Nov 29 11:03:16 CET 2011


Module: sip-router
Branch: 3.1
Commit: 2ce5ff9c1718cc14e34cc6e19a85a30edf3d3e49
URL:    http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=2ce5ff9c1718cc14e34cc6e19a85a30edf3d3e49

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: Daniel-Constantin Mierla <miconda at gmail.com>
Date:   Thu Nov 24 15:30:15 2011 +0100

textops(k): free only pkg body lumps for set_body()

- if set_body() is used in failure_route, it should not destroy
  completely the lumps cloned in tm, just ignore them and free the ones
  added in failure_route before execution of itself
- reported by Brandon Armstead, FS#181
(cherry picked from commit c22a3ec7366ad1a3cc0b5e8229fbabe2c179cca1)
(cherry picked from commit ec8535487fd06e5de7b5efa218f764a6e850b42b)

---

 modules_k/textops/textops.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/modules_k/textops/textops.c b/modules_k/textops/textops.c
index 9198f77..e02f441 100644
--- a/modules_k/textops/textops.c
+++ b/modules_k/textops/textops.c
@@ -1198,7 +1198,7 @@ static int set_body_f(struct sip_msg* msg, char* p1, char* p2)
 		return -1;
 	}
 
-	free_lump_list(msg->body_lumps);
+	del_nonshm_lump( &(msg->body_lumps) );
 	msg->body_lumps = NULL;
 
 	if (msg->content_length) 




More information about the sr-dev mailing list