Hi,

I've found a bug in the latest trunk of sems.

Sems saves To tag on early dialog state (reply code between 101 and 199)
and dosn't update To tag on connected state (reply code between 200 and 299).

See code in AmSipDialog.cpp
==================================================
void AmSipDialog::updateStatus(const AmSipReply& reply)
{
....
  if ((reply.code >= 101) && (reply.code < 300) && 
      (remote_tag.empty() && !reply.remote_tag.empty()))
    remote_tag = reply.remote_tag;
....
==================================================
I think the fix is to add
==================================================
  if ((reply.code >= 200) && (reply.code < 300) &&
      (status != Connected && !reply.remote_tag.empty()))
    remote_tag = reply.remote_tag;
==================================================

See my patсh in the attachment.

Best regards,
Alex Gradinar

Attachment: to_tag.patch
Description: Binary data

_______________________________________________
Semsdev mailing list
[email protected]
http://lists.iptel.org/mailman/listinfo/semsdev

Reply via email to