Of course! You are right!
-Raphael.
On 17.04.2008, at 15:24, Stefan Sayer <[EMAIL PROTECTED]> wrote:
>
> shouldn't that be
> if(!t->retr_buf || !t->retr_len){
> // there is nothing to re-transmit yet!!!
> ?
>
> Stefan
> [EMAIL PROTECTED] wrote:
>> Author: rco
>> Date: 2008-04-17 15:19:56 +0200 (Thu, 17 Apr 2008)
>> New Revision: 895
>>
>> Modified:
>> trunk/core/plug-in/sipctrl/trans_layer.cpp
>> Log:
>> - suppress re-transmission attempt if there is nothing to re-
>> transmit.
>>
>>
>> Modified: trunk/core/plug-in/sipctrl/trans_layer.cpp
>> ===================================================================
>> --- trunk/core/plug-in/sipctrl/trans_layer.cpp 2008-04-16
>> 15:04:28 UTC (rev 894)
>> +++ trunk/core/plug-in/sipctrl/trans_layer.cpp 2008-04-17
>> 13:19:56 UTC (rev 895)
>> @@ -1170,6 +1170,11 @@
>> void trans_layer::retransmit(sip_trans* t)
>> {
>> assert(transport);
>> + if(!t->retr_buf || t->retr_len){
>> + // there is nothing to re-transmit yet!!!
>> + return;
>> + }
>> +
>> int send_err = transport->send(&t->retr_addr,t->retr_buf,t-
>> >retr_len);
>> if(send_err < 0){
>> ERROR("Error from transport layer\n");
>>
>> _______________________________________________
>> Semsdev mailing list
>> [email protected]
>> http://lists.iptel.org/mailman/listinfo/semsdev
>
> --
> Stefan Sayer
> VoIP Services
>
> [EMAIL PROTECTED]
> www.iptego.com
>
> iptego GmbH
> Am Borsigturm 40
> 13507 Berlin
> Germany
>
> Amtsgericht Charlottenburg, HRB 101010
> Geschaeftsfuehrer: Alexander Hoffmann
> _______________________________________________
> Semsdev mailing list
> [email protected]
> http://lists.iptel.org/mailman/listinfo/semsdev
_______________________________________________
Semsdev mailing list
[email protected]
http://lists.iptel.org/mailman/listinfo/semsdev
