On Sep 16, 2009 at 21:48, Andrei Pelinescu-Onciul <[email protected]> wrote:
[...]
> 
> > This has not been modified in t_lookup.c from v0.9.7 nor v2.0 (attached)
> 
> Yes, but the problem is not in the place where it crashes, it's somewhere
> else.
> Both 0.9.7 and 0.9.4 are very old and I don't really remember all the
> fixes that went in (the cvs log is helpful, but does not tell the whole
> story). I wouldn't want to spend a lot of time debugging 0.9.4, just to
> find out in the end that the bug was fixed in 0.9.7, especially since
> there were non-trivial tm changes between the two.

I think I found the problem. It is fixed in 0.9.7.
Could you either try the attached patch, or upgrading?

If you upgrade, try the latest rel_0_9_0 branch, it has 3 additional
fixes for 0.9.7 (one being a fixed "fix" from 0.9.6). I'll add a 0.9.8
tag shortly, just to avoid confusion.

Andrei
commit 3a9745d0e610dbda4b0a1ed9dbdfbe8568797204
Author: Andrei Pelinescu-Onciul <[email protected]>
Date:   Tue Jan 17 15:30:48 2006 +0000

     - totag bmark set to NULL when no totag present, fix from  Quang Minh Phan
       <[email protected]>.

diff --git a/msg_translator.c b/msg_translator.c
index 74bc5bf..af696da 100644
--- a/msg_translator.c
+++ b/msg_translator.c
@@ -1796,7 +1796,10 @@ char * build_res_buf_from_sip_req( unsigned int code, char *text ,str *new_tag,
 					bmark->to_tag_val.s=p+(totags-hdr->name.s);
 					bmark->to_tag_val.len=
 							((struct to_body*)(hdr->parsed))->tag_value.len;
-				};
+				}else{
+					bmark->to_tag_val.s=0;
+					bmark->to_tag_val.len=0;
+				}
 			case HDR_FROM:
 			case HDR_CALLID:
 			case HDR_CSEQ:
_______________________________________________
sr-dev mailing list
[email protected]
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to