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

[EMAIL PROTECTED] 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
[email protected]
http://openser.org/cgi-bin/mailman/listinfo/devel



_______________________________________________
Devel mailing list
[email protected]
http://openser.org/cgi-bin/mailman/listinfo/devel

Reply via email to