Hi Verbal,
I have made some change to the patch. So please use this one instead. Just
some error checking, and no change in functionality.
Can auth_db maintainer incorporate this into cvs?
Richard
-----Original Message-----
From: Francesco Bottà [mailto:francesco.botta@eutelia.it]
Sent: Wednesday, January 12, 2005 4:22 AM
To: Richard; Daniel-Constantin.Mierla(a)fokus.fraunhofer.de
Cc: serusers(a)lists.iptel.org; 'Maxim Sobolev'
Subject: Re: [Serusers] RPID problem in 0.9.0 release
Hi all,
I've tested the patch in the 0.9.0 release and now the function
append_rpid_hd works great.
Many thanks,
Verbal
----- Original Message -----
From: "Richard" <richard(a)o-matrix.org>
To: <Daniel-Constantin.Mierla(a)fokus.fraunhofer.de>de>; "'Francesco
Bottà'"
<francesco.botta(a)eutelia.it>
Cc: <serusers(a)lists.iptel.org>rg>; "'Maxim Sobolev'"
<sobomax(a)portaone.com>
Sent: Tuesday, December 28, 2004 3:51 AM
Subject: RE: [Serusers] RPID problem in 0.9.0 release
-----Original Message-----
From: serusers-bounces(a)iptel.org [mailto:serusers-bounces@lists.iptel.org] On
Behalf Of Daniel-Constantin.Mierla(a)fokus.fraunhofer.de
Sent: Monday, December 27, 2004 8:01 AM
To: Francesco Bottà
Cc: serusers(a)lists.iptel.org; Maxim Sobolev
Subject: Re: [Serusers] RPID problem in 0.9.0 release
auth_db is one of the module which wasn't reviewed yet - reason to have
a
prelease (or testing release) by now. As far as I
understood, with these
two
new parameters you load the values of the appropriate columns from
database in
AVP list, while append_rpid_hf() uses a static buffer which is filled
when
use_rpid=1. So you have to keep the value to 1
for now.
It appears to be a bug in the auth_db module. It saves rpid in the static
buffer as well as the avp rpid. The problem is the static buffer refers to
a
local variable which probably no longer exists when append_rpid_hf() is
called.
Apply the attached patch should solve the problem.
Otherwise, you can push a value from AVP list to
a header using avpops
module.
The current avp functions can't manipulate the avp. So prefix and suffix
can't be added to the header field. Of course one can change the data in
the
database, but it will not be backward compatible to the existing system.
Richard