Summary: | ASTERISK-10659: [patch] Memory leak on res_jabber.c | ||
Reporter: | Eliel Sardanons (eliel) | Labels: | |
Date Opened: | 2007-10-31 16:04:58 | Date Closed: | 2007-10-31 16:21:58 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Resources/res_jabber |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) res_jabber.c.patch | |
Description: | It is a common error to do things like this: a = ast_malloc(); if (!a) return NULL; b = ast_malloc(); if (!b) return NULL; <--- and we need to ast_free(a)!! | ||
Comments: | By: Digium Subversion (svnbot) 2007-10-31 16:21:58 Repository: asterisk Revision: 87909 _U trunk/ U trunk/res/res_jabber.c ------------------------------------------------------------------------ r87909 | qwell | 2007-10-31 16:21:58 -0500 (Wed, 31 Oct 2007) | 13 lines Merged revisions 87908 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 (closes issue ASTERISK-10659) ........ r87908 | qwell | 2007-10-31 16:23:11 -0500 (Wed, 31 Oct 2007) | 4 lines Make sure we free some allocated memory before returning. Issue 11131, patch by eliel. ........ ------------------------------------------------------------------------ |