Re: [osip-dev] Ignored T2 value in oSIP

[David Ramirez]

Hi Aymeric,

You forgot osip.c source file:

--- a/src/osip2/osip.c    2013-02-28 13:25:06.000000000 +0100
+++ b/src/osip2/osip.c    2013-02-28 13:25:30.000000000 +0100
@@ -251,8 +251,8 @@
 {
   if (osip_timercmp (current, &ixt->start, >)) {
     ixt->interval = ixt->interval * 2;
-    if (ixt->interval > 4000)
-      ixt->interval = 4000;
+    if (ixt->interval > DEFAULT_T2)
+      ixt->interval = DEFAULT_T2;
     add_gettimeofday (&ixt->start, ixt->interval);
     if (ixt->ack != NULL)
       osip->cb_send_message (NULL, ixt->ack, ixt->dest, ixt->port, ixt->sock);


Thanks for the clarification.

Regards,
David

On 28/02/13 11:51, Aymeric Moizard wrote:

Hi David,

Tks for finding this tiny mistake! I have made the patch and pushed it into the git.

http://git.savannah.gnu.org/cgit/osip.git/commit/?id=e07abc21252f0b8be603a96fe305ab5466364dd0

Regards

Aymeric

2013/2/27 David Ramirez <address@hidden>

Hi everyone,

I've notice that the DEFAULT_T2 macro definition is ignored in latest osip version, instead value 4000 is hardcoded.
There is a comment in src/osip2/nict_fsm.c that states: "in PROCEEDING STATE, TIMER is always 4000".
Why isn't T2 the value for this timers?

Regards,
David

--
David Ramírez
Ingeniería

Tel.: +34 914 904 103
Móv.: +34 663 82 21 38
Fax: +34 914 903 601
address@hidden
http://www.mantica-solutions.com
Antes de imprimir este mensaje, asegúrese de que es necesario. El medio ambiente está en nuestra mano.
 
AVISO LEGAL
Este mensaje y, en su caso, los ficheros adjuntos, se dirigen exclusivamente a su/s destinatario/s y pueden contener información privilegiada o confidencial. Si no es Vd. el destinatario indicado, queda notificado de que la utilización, divulgación y/o copia sin autorización está prohibida en virtud de la legislación vigente. Si ha recibido este mensaje por error, le rogamos que nos lo comunique inmediatamente por esta misma vía y proceda a su destrucción.
 
CONFIDENTIAL NOTICE
This email and any files transmitted with it are confidential and intended solely for the use of the individual or entity to whom they are addressed. If you are not the intended recipient you are notified that disclosing, copying, distributing or taking any action in reliance on the contents of this information is strictly prohibited. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system.

_______________________________________________
osip-dev mailing list
address@hidden
https://lists.gnu.org/mailman/listinfo/osip-dev

--
Antisip - http://www.antisip.com

--
David Ramírez
Ingeniería

Tel.: +34 914 904 103
Móv.: +34 663 82 21 38
Fax: +34 914 903 601
address@hidden
http://www.mantica-solutions.com
Antes de imprimir este mensaje, asegúrese de que es necesario. El medio ambiente está en nuestra mano.
 
AVISO LEGAL
Este mensaje y, en su caso, los ficheros adjuntos, se dirigen exclusivamente a su/s destinatario/s y pueden contener información privilegiada o confidencial. Si no es Vd. el destinatario indicado, queda notificado de que la utilización, divulgación y/o copia sin autorización está prohibida en virtud de la legislación vigente. Si ha recibido este mensaje por error, le rogamos que nos lo comunique inmediatamente por esta misma vía y proceda a su destrucción.
 
CONFIDENTIAL NOTICE
This email and any files transmitted with it are confidential and intended solely for the use of the individual or entity to whom they are addressed. If you are not the intended recipient you are notified that disclosing, copying, distributing or taking any action in reliance on the contents of this information is strictly prohibited. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system.