[sr-dev] git:kamailio_3.0: avposp: use state in searching first avp in avp_copy

Daniel-Constantin Mierla miconda at gmail.com
Thu May 6 21:14:15 CEST 2010


Module: sip-router
Branch: kamailio_3.0
Commit: 59894cc9d53b03494753a00c4057c595afc3e5c0
URL:    http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=59894cc9d53b03494753a00c4057c595afc3e5c0

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: Daniel-Constantin Mierla <miconda at gmail.com>
Date:   Thu May  6 21:09:47 2010 +0200

avposp: use state in searching first avp in avp_copy

- state parameter was missing when searching first avp, keeping it
  uninitialized when searching the next, causing a crash when flag 'g'
  was used in avp_copy
- reported by Jon Bonilla (Manwe)
(cherry picked from commit 027d68e9cc7afcdb3ba8922215a28e33a5225097)

---

 modules/avpops/avpops_impl.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/modules/avpops/avpops_impl.c b/modules/avpops/avpops_impl.c
index 3817cd8..77de8a8 100644
--- a/modules/avpops/avpops_impl.c
+++ b/modules/avpops/avpops_impl.c
@@ -842,7 +842,7 @@ int ops_copy_avp( struct sip_msg* msg, struct fis_param* src,
 		goto error;
 	}
 
-	avp = search_first_avp( name_type1, avp_name1, &avp_val, 0);
+	avp = search_first_avp( name_type1, avp_name1, &avp_val, &st);
 	while ( avp )
 	{
 		/* build a new avp with new name, but old value */




More information about the sr-dev mailing list