diff options
author | Scott Griepentrog <sgriepentrog@digium.com> | 2015-03-17 22:15:09 +0000 |
---|---|---|
committer | Scott Griepentrog <sgriepentrog@digium.com> | 2015-03-17 22:15:09 +0000 |
commit | 2c7b9451499c5bd50bc6fa54a0073781b116834c (patch) | |
tree | 66390386d5b9e4200c83afa3448bd4d054254349 /main/endpoints.c | |
parent | 1fb1c81923249413e3e439a645ade8793ccc0db6 (diff) |
Various: bugfixes found via chaos
Using DEBUG_CHAOS several instances of a null
pointer crash, and one uninitialized variable
were uncovered and fixed. Also added details
on why Asterisk failed to initialize.
Review: https://reviewboard.asterisk.org/r/4468/
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/13@433064 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main/endpoints.c')
-rw-r--r-- | main/endpoints.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/main/endpoints.c b/main/endpoints.c index 3a6f494c0..66ad4618e 100644 --- a/main/endpoints.c +++ b/main/endpoints.c @@ -472,7 +472,8 @@ struct ast_endpoint_snapshot *ast_endpoint_snapshot_create( endpoint_snapshot_dtor, AO2_ALLOC_OPT_LOCK_NOLOCK); - if (ast_string_field_init(snapshot, 80) != 0) { + if (!snapshot || ast_string_field_init(snapshot, 80) != 0) { + ao2_cleanup(snapshot); return NULL; } |