[Devel] FL_REQ_UPSTREAM introduction in module acc

[openser-devel at list.coretech.ro]

thanks,
razvan radu

Bogdan-Andrei Iancu wrote:
> Salut Razvan,
>
> yes that's right, the comments in the sip_msg structure are misleading 
> and the FL_REQ_UPSTREAM flag no not set by ACC module in the proper 
> flags set.
>
> I fixed it on the CVS.
>
> thanks and regards,
> Bogdan
>
> openser-devel at list.coretech.ro wrote:
>
>>
>> hello,
>> please ignore my message if this is the intended functionality and if 
>> it is documented somewhere I did not find it.
>>
>> problem: if you set flag 9 in script the accounting values for "from" 
>> & "to" get switched between them.
>> in file "modules/acc/acc.c" the line "_msg->flags&FL_REQ_UPSTREAM" is 
>> used to test something but "->flags" is the one used to store script 
>> flags, I think that msg_flags should be used in file "modules/acc/acc.c"
>>
>> also I think that will be nice to switch between them the comments on 
>> fields "flags" & "msg_flags" in file "parser/msg_parser.h" for 
>> structure "struct sip_msg"
>>
>> thanks,
>> razvan radu
>>
>>
>>
>> _______________________________________________
>> Devel mailing list
>> Devel at openser.org
>> http://openser.org/cgi-bin/mailman/listinfo/devel
>>
>