[Asterisk-code-review] Check for ao2 alloc failure in ast channel internal alloc. (asterisk[11])
Corey Farrell
asteriskteam at digium.com
Tue Apr 21 15:35:33 CDT 2015
Corey Farrell has uploaded a new change for review.
https://gerrit.asterisk.org/188
Change subject: Check for ao2_alloc failure in __ast_channel_internal_alloc.
......................................................................
Check for ao2_alloc failure in __ast_channel_internal_alloc.
Fix a crash that could occur in __ast_channel_internal_alloc if
ao2_alloc fails.
ASTERISK-24991 #close
Change-Id: I4ca89189eb22f907408cb87d0a1645cfe1314a90
---
M main/channel_internal_api.c
1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/88/188/1
diff --git a/main/channel_internal_api.c b/main/channel_internal_api.c
index 3f892dd..3cfa681 100644
--- a/main/channel_internal_api.c
+++ b/main/channel_internal_api.c
@@ -1344,6 +1344,10 @@
tmp = ao2_alloc(sizeof(*tmp), destructor);
#endif
+ if (!tmp) {
+ return NULL;
+ }
+
if ((ast_string_field_init(tmp, 128))) {
return ast_channel_unref(tmp);
}
--
To view, visit https://gerrit.asterisk.org/188
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4ca89189eb22f907408cb87d0a1645cfe1314a90
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Corey Farrell <git at cfware.com>
More information about the asterisk-code-review
mailing list