[Asterisk-code-review] main/config options: Check for existance of internal object ... (asterisk[13])

Matt Jordan asteriskteam at digium.com
Tue Sep 8 20:13:36 CDT 2015


Matt Jordan has uploaded a new patch set (#2).

Change subject: main/config_options: Check for existance of internal object before derefing
......................................................................

main/config_options: Check for existance of internal object before derefing

A can load and register an object type while still having an invalid sorcery
mapping. This can cause an issue when a creation call is invoked. For example,
mis-configuring PJSIP's endpoint identifier by IP address mapping in
sorcery.conf will cause the sorcery mechanism to be invalidated; however, a
subsequent ARI invocation to create the object will cause a crash, as the
internal type may not be registered as sorcery expects.

Merely checking for a NULL pointer here solves the issue.

Change-Id: I54079fb94a1440992f4735a9a1bbf1abb1c601ac
---
M main/config_options.c
1 file changed, 4 insertions(+), 0 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I54079fb94a1440992f4735a9a1bbf1abb1c601ac
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list