The fix is incorrect since it creates memory leak. I've just committed a proper fix. Please test and let me know if you are still experiencing this problem.
-Maxim
Richard wrote:
-----Original Message----- From: serusers-bounces@iptel.org [mailto:serusers-bounces@lists.iptel.org] On Behalf Of Daniel-Constantin.Mierla@fokus.fraunhofer.de Sent: Monday, December 27, 2004 8:01 AM To: Francesco Bottà Cc: serusers@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