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

Stefan Engström reviewboard at asterisk.org
Tue Apr 28 09:47:21 CDT 2015


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

(Updated April 28, 2015, 2:47 p.m.)


Review request for Asterisk Developers.


Changes
-------

cleaned-up patch


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 (updated)
-----

  /branches/13/res/res_rtp_asterisk.c 434725 

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/cf25eda6/attachment.html>


More information about the asterisk-dev mailing list