Hello,

thanks for troubleshooting and fix. I would rather change the define, like:

#define IMC_MEMBER_SKIP (1<<4)

It should be safer for similar cases in future usages.

I'll look into the code soon.

Cheers, Daniel

On 8/2/13 10:40 AM, Shankar wrote:

Hello,

While testing group chat via IMC module, we found that owner of a chat room could not destroy the room or invite new users to a chat room after few message exchange.

We had modified the code at "imc_cmd.c" to fix this issue.

Fix,

member->flags &= ~IMC_MEMBER_SKIP; à member->flags &= ~(IMC_MEMBER_SKIP);

where

~IMC_MEMBER_SKIP is defined as '#define IMC_MEMBER_SKIP 1<<4'.

Here ~ takes precedence over the << operator. With this fix the owner of the chat-room is able to invite new users during a chat and destroy the room.

Regards,

Shankar

---

SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list sr-users@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users