[asterisk-dev] [Code Review] 4521: DTLS-crashes-ASTERISK-24832

[one47]

> On April 28, 2015, 10:16 a.m., one47 wrote:
> > I have applied a cleaned-up version of this patch. I do not believe you need the 1-second wait loop at-all, but the rest of the patch looks clean, and after some research I even worked out why it helps :)
> > 
> > Yes, I think it should be progressed.
> 
> Stefan Engström wrote:
>     Cool - where can I view the contents of your cleaned-up version?
> 
> one47 wrote:
>     FYI I have patched asterisk v.11, and reproduced the issue using a load-test setup.

It is hardly worth me pushing a specific version to you - It is your patch with all of the whitespace and comment changes removed, and I then removed the 1-second delay loop. If you prefer I can email you my version direct - I do not think I can add it here?


- one47


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4521/#review15200
-----------------------------------------------------------


On March 25, 2015, 4:24 p.m., Stefan Engström wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4521/
> -----------------------------------------------------------
> 
> (Updated March 25, 2015, 4:24 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> Not in final form - What's essential to review is the extra check done before calling dtls_perform_handshake.
> 
> I'm aware that this patch is polluted with comments, bad spacing and ast_log-calls... I'll align the patch with https://wiki.asterisk.org/wiki/display/AST/Coding+Guidelines only if you judge the 'actual patch-contents' might be commitable
> 
> 
> Diffs
> -----
> 
>   /branches/13/res/res_rtp_asterisk.c 433370 
> 
> Diff: https://reviewboard.asterisk.org/r/4521/diff/
> 
> 
> Testing
> -------
> 
> Yes, with this patch applied crashes has gone down to 0 in 30 days, and before patch it would crash once every 1-4 hours.
> 
> 
> Thanks,
> 
> Stefan Engström
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150428/0348cfb3/attachment.html>