[Home]

Summary:ASTERISK-06053: [patch] Converting /apps to use *alloc() wrappers - 2
Reporter:Matthew Roth (matthew roth)Labels:
Date Opened:2006-01-11 14:54:01.000-0600Date Closed:2008-01-15 16:14:27.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_alarmreceiver_to_externalivr-alloc_wrappers-1.0.patch
Description:This patch updates the following apps to use the new *alloc() wrappers:

- apps/app_alarmreceiver.c
- apps/app_curl.c
- apps/app_dial.c
- apps/app_directory.c
- apps/app_externalivr.c

The following cleanup tasks were also performed on the updated code:

- malloc() + memset() consolidated to ast_calloc()
- allocations and null checks consolidated to 1 line
- sizeof calls changed to reflect the coding guidelines

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

Please let me know if I made any mistakes or if I could improve my patches in any way.

Thank you = )
Comments:By: Russell Bryant (russell) 2006-01-11 15:25:47.000-0600

as a heads up, if you get to app_macro.c, you can remove safe_strdup() and replace callers to it with ast_strdup().

By: Russell Bryant (russell) 2006-01-11 15:42:17.000-0600

added to the trunk, thanks!

By: Digium Subversion (svnbot) 2008-01-15 16:14:27.000-0600

Repository: asterisk
Revision: 7991

U   trunk/apps/app_alarmreceiver.c
U   trunk/apps/app_curl.c
U   trunk/apps/app_dial.c
U   trunk/apps/app_directory.c
U   trunk/apps/app_externalivr.c

------------------------------------------------------------------------
r7991 | russell | 2008-01-15 16:14:26 -0600 (Tue, 15 Jan 2008) | 2 lines

conversions to memory allocation wrappers (issue ASTERISK-6053)

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

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