[Home]

Summary:ASTERISK-13901: [patch] memory leak in some part of code
Reporter:Mayank Jain Nawal (contactmayankjain)Labels:
Date Opened:2009-04-06 05:38:33Date Closed:2009-05-28 10:35:26
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Addons/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) all_changes.patch
Description:Memory leak are present in some part of the asterisk code. This have been tested by using tool cppcheck
cppcheck -a -j 3 -q -f --style asterisk/

At some of the places NULL is check before deleting memory which is  Redundant condition. It is safe to deallocate a NULL pointer.
http://www.mail-archive.com/asterisk-dev@lists.digium.com/msg36740.html
Comments:By: Mayank Jain Nawal (contactmayankjain) 2009-04-06 06:02:16

Have attached a small patch file for this. Will continue with resolving more issue as time allows.
By: Digium Subversion (svnbot) 2009-05-28 10:35:25

Repository: asterisk
Revision: 197616

U   trunk/apps/app_rpt.c
U   trunk/channels/chan_console.c
U   trunk/channels/chan_dahdi.c
U   trunk/main/astobj2.c
U   trunk/main/cli.c

------------------------------------------------------------------------
r197616 | tilghman | 2009-05-28 10:35:24 -0500 (Thu, 28 May 2009) | 7 lines

Eliminate several needless checks and fix a few memory leaks
(closes issue ASTERISK-13901)
Reported by: contactmayankjain
Patches:
      all_changes.patch uploaded by contactmayankjain (license 740)
      slightly modified by me

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=197616