[Home]

Summary:ASTERISK-14264: [patch] memory leak in asterisk some bug fixing and removing Redundant condition
Reporter:Mayank Jain Nawal (contactmayankjain)Labels:
Date Opened:2009-06-04 07:31:06Date Closed:2009-06-25 16:47:48
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Addons/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) memory_leak_stuff.trunk.diff
( 1) patch.txt
Description:Used cppcheck to find out memory leak in asterisk some bug fixing and removing Redundant condition

****** ADDITIONAL INFORMATION ******

Also adding patch
Comments:By: Mayank Jain Nawal (contactmayankjain) 2009-06-04 07:36:07

Attaching the patch file which contains fixes for the code. Will correct more bugs as the time permits. Hope this will be of help.
By: Leif Madsen (lmadsen) 2009-06-16 14:02:54

Setting this issue to be targeted for 1.6.2.0. Thanks for the submission!
By: David Vossel (dvossel) 2009-06-18 11:33:27

patch looked good for the most part, fixed a few things I thought might cause problems.
By: Digium Subversion (svnbot) 2009-06-18 11:37:44

Repository: asterisk
Revision: 201678

U   trunk/apps/app_rpt.c
U   trunk/channels/h323/ast_h323.cxx
U   trunk/channels/misdn/isdn_lib.c
U   trunk/channels/xpmr/xpmr.c
U   trunk/codecs/gsm/src/gsm_destroy.c
U   trunk/main/ast_expr2.c
U   trunk/main/ast_expr2f.c
U   trunk/main/asterisk.c
U   trunk/pbx/pbx_config.c
U   trunk/res/ael/ael_lex.c
U   trunk/res/res_config_ldap.c
U   trunk/utils/ael_main.c
U   trunk/utils/conf2ael.c
U   trunk/utils/extconf.c
U   trunk/utils/stereorize.c

------------------------------------------------------------------------
r201678 | dvossel | 2009-06-18 11:37:43 -0500 (Thu, 18 Jun 2009) | 11 lines

fixes some memory leaks and redundant conditions

(closes issue ASTERISK-14264)
Reported by: contactmayankjain
Patches:
     patch.txt uploaded by contactmayankjain (license 740)
     memory_leak_stuff.trunk.diff uploaded by dvossel (license 671)
Tested by: contactmayankjain, dvossel



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

http://svn.digium.com/view/asterisk?view=rev&revision=201678
By: Digium Subversion (svnbot) 2009-06-18 11:44:39

Repository: asterisk
Revision: 201679

_U  branches/1.6.2/
U   branches/1.6.2/apps/app_rpt.c
U   branches/1.6.2/channels/h323/ast_h323.cxx
U   branches/1.6.2/channels/misdn/isdn_lib.c
U   branches/1.6.2/channels/xpmr/xpmr.c
U   branches/1.6.2/codecs/gsm/src/gsm_destroy.c
U   branches/1.6.2/main/ast_expr2.c
U   branches/1.6.2/main/ast_expr2f.c
U   branches/1.6.2/main/asterisk.c
U   branches/1.6.2/pbx/pbx_config.c
U   branches/1.6.2/res/ael/ael_lex.c
U   branches/1.6.2/res/res_config_ldap.c
U   branches/1.6.2/utils/ael_main.c
U   branches/1.6.2/utils/conf2ael.c
U   branches/1.6.2/utils/extconf.c
U   branches/1.6.2/utils/stereorize.c

------------------------------------------------------------------------
r201679 | dvossel | 2009-06-18 11:44:38 -0500 (Thu, 18 Jun 2009) | 16 lines

Merged revisions 201678 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r201678 | dvossel | 2009-06-18 11:37:42 -0500 (Thu, 18 Jun 2009) | 11 lines
 
 fixes some memory leaks and redundant conditions
 
 (closes issue ASTERISK-14264)
 Reported by: contactmayankjain
 Patches:
       patch.txt uploaded by contactmayankjain (license 740)
       memory_leak_stuff.trunk.diff uploaded by dvossel (license 671)
 Tested by: contactmayankjain, dvossel
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=201679
By: Digium Subversion (svnbot) 2009-06-18 11:51:55

Repository: asterisk
Revision: 201680

_U  branches/1.6.1/
U   branches/1.6.1/apps/app_rpt.c
U   branches/1.6.1/channels/h323/ast_h323.cxx
U   branches/1.6.1/channels/misdn/isdn_lib.c
U   branches/1.6.1/channels/xpmr/xpmr.c
U   branches/1.6.1/codecs/gsm/src/gsm_destroy.c
U   branches/1.6.1/main/ast_expr2.c
U   branches/1.6.1/main/ast_expr2f.c
U   branches/1.6.1/main/asterisk.c
U   branches/1.6.1/pbx/pbx_config.c
U   branches/1.6.1/res/ael/ael_lex.c
U   branches/1.6.1/res/res_config_ldap.c
U   branches/1.6.1/utils/ael_main.c
U   branches/1.6.1/utils/conf2ael.c
U   branches/1.6.1/utils/extconf.c
U   branches/1.6.1/utils/stereorize.c

------------------------------------------------------------------------
r201680 | dvossel | 2009-06-18 11:51:55 -0500 (Thu, 18 Jun 2009) | 16 lines

Merged revisions 201678 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r201678 | dvossel | 2009-06-18 11:37:42 -0500 (Thu, 18 Jun 2009) | 11 lines
 
 fixes some memory leaks and redundant conditions
 
 (closes issue ASTERISK-14264)
 Reported by: contactmayankjain
 Patches:
       patch.txt uploaded by contactmayankjain (license 740)
       memory_leak_stuff.trunk.diff uploaded by dvossel (license 671)
 Tested by: contactmayankjain, dvossel
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=201680
By: Digium Subversion (svnbot) 2009-06-18 11:58:04

Repository: asterisk
Revision: 201682

_U  branches/1.6.0/
U   branches/1.6.0/channels/h323/ast_h323.cxx
U   branches/1.6.0/channels/misdn/isdn_lib.c
U   branches/1.6.0/codecs/gsm/src/gsm_destroy.c
U   branches/1.6.0/main/ast_expr2.c
U   branches/1.6.0/main/ast_expr2f.c
U   branches/1.6.0/main/asterisk.c
U   branches/1.6.0/pbx/pbx_config.c
U   branches/1.6.0/res/ael/ael_lex.c
U   branches/1.6.0/res/res_config_ldap.c
U   branches/1.6.0/utils/ael_main.c
U   branches/1.6.0/utils/conf2ael.c
U   branches/1.6.0/utils/extconf.c
U   branches/1.6.0/utils/stereorize.c

------------------------------------------------------------------------
r201682 | dvossel | 2009-06-18 11:58:03 -0500 (Thu, 18 Jun 2009) | 16 lines

Merged revisions 201678 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r201678 | dvossel | 2009-06-18 11:37:42 -0500 (Thu, 18 Jun 2009) | 11 lines
 
 fixes some memory leaks and redundant conditions
 
 (closes issue ASTERISK-14264)
 Reported by: contactmayankjain
 Patches:
       patch.txt uploaded by contactmayankjain (license 740)
       memory_leak_stuff.trunk.diff uploaded by dvossel (license 671)
 Tested by: contactmayankjain, dvossel
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=201682
By: Digium Subversion (svnbot) 2009-06-25 16:45:33

Repository: asterisk
Revision: 203444

U   trunk/main/ast_expr2.c
U   trunk/main/ast_expr2.fl

------------------------------------------------------------------------
r203444 | dvossel | 2009-06-25 16:45:33 -0500 (Thu, 25 Jun 2009) | 4 lines

fixes a few redundant conditions

(issue ASTERISK-14264)

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

http://svn.digium.com/view/asterisk?view=rev&revision=203444
By: Digium Subversion (svnbot) 2009-06-25 16:46:14

Repository: asterisk
Revision: 203445

_U  branches/1.6.2/
U   branches/1.6.2/main/ast_expr2.c
U   branches/1.6.2/main/ast_expr2.fl

------------------------------------------------------------------------
r203445 | dvossel | 2009-06-25 16:46:14 -0500 (Thu, 25 Jun 2009) | 11 lines

Merged revisions 203444 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r203444 | dvossel | 2009-06-25 16:45:32 -0500 (Thu, 25 Jun 2009) | 4 lines
 
 fixes a few redundant conditions
 
 (issue ASTERISK-14264)
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=203445
By: Digium Subversion (svnbot) 2009-06-25 16:46:47

Repository: asterisk
Revision: 203446

_U  branches/1.6.1/
U   branches/1.6.1/main/ast_expr2.c
U   branches/1.6.1/main/ast_expr2.fl

------------------------------------------------------------------------
r203446 | dvossel | 2009-06-25 16:46:47 -0500 (Thu, 25 Jun 2009) | 11 lines

Merged revisions 203444 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r203444 | dvossel | 2009-06-25 16:45:32 -0500 (Thu, 25 Jun 2009) | 4 lines
 
 fixes a few redundant conditions
 
 (issue ASTERISK-14264)
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=203446
By: Digium Subversion (svnbot) 2009-06-25 16:47:48

Repository: asterisk
Revision: 203447

_U  branches/1.6.0/
U   branches/1.6.0/main/ast_expr2.c
U   branches/1.6.0/main/ast_expr2.fl

------------------------------------------------------------------------
r203447 | dvossel | 2009-06-25 16:47:48 -0500 (Thu, 25 Jun 2009) | 11 lines

Merged revisions 203444 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r203444 | dvossel | 2009-06-25 16:45:32 -0500 (Thu, 25 Jun 2009) | 4 lines
 
 fixes a few redundant conditions
 
 (issue ASTERISK-14264)
........

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

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