Re: [reSIProcate] Rejecting media negotiation in case of SDP-less reInvite

[Francis Joanis <francis.joanis@xxxxxxxxx>]

On Mon, Sep 19, 2011 at 5:05 AM, Robert Szokovacs
<rszokovacs@xxxxxxxxxxxxxxx> wrote:
> On 2011 September 16, Friday 17:04:24 you wrote:
>> Hi, the following patch seems to work:
>>
>> svn diff resip/dum/InviteSession.cxx
>> Index: resip/dum/InviteSession.cxx
>> ===================================================================
>> --- resip/dum/InviteSession.cxx (revision 9266)
>> +++ resip/dum/InviteSession.cxx (working copy)
>> @@ -757,7 +757,15 @@
>>           send(response);
>>           break;
>>        }
>> -
>> +      // Sent a reINVITE no offer and received a 200-offer.
>> +      // Simply send an ACK without an answer and stay in Connected.
>> +      case SentReinviteAnswered:
>> +      {
>> +         InfoLog (<< "Not sending " << statusCode << " error since
>> transaction already completed, sending answer-less ACK");
>> +         transition(Connected);
>> +         sendAck();
>> +         break;
>> +      }
>>        default:
>>           assert(0);
>>           break;
>>
> [...]
>
>> Robert, could you test it and let me know if it works?
>
> I confirm, the patch is working!
>
> Thank you!
>
> br
>
> Szo
>
> Szo
>

Hi,

I've applied to patch to the mainline, rev 9267. Let me know if you
have any question.

Thanks,
Francis