Changeset 2271 for pjproject/trunk/pjsip/src/pjsip/sip_dialog.c

Timestamp:

Sep 11, 2008 8:00:47 AM (16 years ago)

Author:

bennylp

Message:

Ticket #620: PRACK is sent to the wrong UAS when 100rel is used and the dialog forks (thanks Ruud Klaver for the report)

File:

• pjproject/trunk/pjsip/src/pjsip/sip_dialog.c (modified) (2 diffs)

pjproject/trunk/pjsip/src/pjsip/sip_dialog.c

@@ -1697 +1697 @@
      * The workaround for this is to take the To tag received in the
      * 2xx response and set it as remote tag.
+     *
+     * New update:
+     * We also need to update the dialog for 1xx responses, to handle the
+     * case when 100rel is used, otherwise PRACK will be sent to the 
+     * wrong target.
      */
     if ((dlg->state == PJSIP_DIALOG_STATE_NULL && 
@@ -1705 +1710 @@
         (dlg->role==PJSIP_ROLE_UAC &&
          !dlg->uac_has_2xx &&
-         res_code/100 == 2 &&
+         res_code/100 <= 2 &&
          pjsip_method_creates_dialog(&rdata->msg_info.cseq->method) &&
          pj_strcmp(&dlg->remote.info->tag, &rdata->msg_info.to->tag)))