[Asterisk-code-review] cdr: Missing NULL check and unlock. (asterisk[13])

Corey Farrell asteriskteam at digium.com
Mon Dec 25 20:53:05 CST 2017


Corey Farrell has uploaded a new patch set (#2). ( https://gerrit.asterisk.org/7740 )

Change subject: cdr: Missing NULL check and unlock.
......................................................................

cdr: Missing NULL check and unlock.

* handle_dial_message: Missing a check for NULL peer.
* ast_cdr_register: Missing unlock on allocation failure.

ast_cdr_register is fixed by reordering so the new structure is
allocated and initialized before locking the list.

Change-Id: I5799b99270d1a7a716a555c31ac85f4b00ce8686
---
M main/cdr.c
1 file changed, 25 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/40/7740/2
-- 
To view, visit https://gerrit.asterisk.org/7740
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5799b99270d1a7a716a555c31ac85f4b00ce8686
Gerrit-Change-Number: 7740
Gerrit-PatchSet: 2
Gerrit-Owner: Corey Farrell <git at cfware.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171225/ab12565d/attachment.html>


More information about the asterisk-code-review mailing list