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-0600 | Date Closed: | 2008-01-15 16:14:27.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | 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 |