[Devel] FL_REQ_UPSTREAM introduction in module acc

Bogdan-Andrei Iancu bogdan at voice-system.ro
Mon Jun 26 10:37:23 CEST 2006


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
>




More information about the Devel mailing list