[Nice] patch for TURN nonce expired

[Madaro Livio]

Hello,
I think this patch is still pending. Has it been rejected or is it under review?

Regards,
Livio


-----Original Message-----
From: nice-bounces+livio.madaro=telecomitalia.it at lists.freedesktop.org [mailto:nice-bounces+livio.madaro=telecomitalia.it at lists.freedesktop.org] On Behalf Of Madaro Livio
Sent: Thursday, March 21, 2013 11:02 AM
To: nice at lists.freedesktop.org
Subject: Re: [Nice] patch for TURN nonce expired

Hi,
I think the error is not specific to TURN because it is defined in RFC5389. I think the nice agent can already handle the  error 438, but it is "filtered" by the stun agent.

Regards,
Livio

-----Original Message-----
From: nice-bounces+livio.madaro=telecomitalia.it at lists.freedesktop.org [mailto:nice-bounces+livio.madaro=telecomitalia.it at lists.freedesktop.org] On Behalf Of Youness Alaoui
Sent: Wednesday, March 20, 2013 8:50 PM
To: nice at lists.freedesktop.org
Subject: Re: [Nice] patch for TURN nonce expired

Humm.. I'm not sure this is done the right way. Shouldn't the error be validated in the nice agent rather than stun agent, since that error is specific to turn, right ?
I think libnice should detect that error and clear the nonce and retry, instead of considering that the message is not authenticated.
I'll have to think about it, but I'm a bit busy now.

Thanks,
Youness.

On 02/28/2013 05:48 AM, Madaro Livio wrote:
> Hi,
>
> I think there is a problem when  TURN nonce expires.
>
> If the nonce expires the TURN server replies with error code 438 when 
> libnice sends a TURN allocation refresh.  libnice should re-send the 
> allocation refresh with the new nonce but libnice fails to validate 
> the error code 438 (STUN_VALIDATION_UNAUTHORIZED).
>
> Using turnserver.org the nonce expires after 1 hour.
>
>
>
> I send a patch for review. I just added a check for error code 438 in 
> stun validation (stun_agent_validate). Do you think the check for 
> error 438 is missing elsewhere?
>
>
>
> Regards,
>
> Livio
>
>
>
> Questo messaggio e i suoi allegati sono indirizzati esclusivamente 
> alle persone indicate. La diffusione, copia o qualsiasi altra azione 
> derivante dalla conoscenza di queste informazioni sono rigorosamente 
> vietate. Qualora abbiate ricevuto questo documento per errore siete 
> cortesemente pregati di darne immediata comunicazione al mittente e di provvedere alla sua distruzione, Grazie.
>
> /This e-mail and any attachments// is //confidential and may contain 
> privileged information intended for the addressee(s) only.
> Dissemination, copying, printing or use by anybody else is 
> unauthorised. If you are not the intended recipient, please delete 
> this message and any attachments and advise the sender by return 
> e-mail, Thanks./
>
> *rispetta l'ambienteRispetta l'ambiente. Non stampare questa mail se 
> non è
> necessario.*
>
>
>
> _______________________________________________
> nice mailing list
> nice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/nice
>



Questo messaggio e i suoi allegati sono indirizzati esclusivamente alle persone indicate. La diffusione, copia o qualsiasi altra azione derivante dalla conoscenza di queste informazioni sono rigorosamente vietate. Qualora abbiate ricevuto questo documento per errore siete cortesemente pregati di darne immediata comunicazione al mittente e di provvedere alla sua distruzione, Grazie.

This e-mail and any attachments is confidential and may contain privileged information intended for the addressee(s) only. Dissemination, copying, printing or use by anybody else is unauthorised. If you are not the intended recipient, please delete this message and any attachments and advise the sender by return e-mail, Thanks.

_______________________________________________
nice mailing list
nice at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nice