[asterisk-commits] pitel: trunk r300215 - in /trunk: ./ res/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Jan 4 17:04:20 UTC 2011


Author: pitel
Date: Tue Jan  4 11:04:14 2011
New Revision: 300215

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=300215
Log:
Merged revisions 300214 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r300214 | pitel | 2011-01-04 18:01:52 +0100 (Út, 04 led 2011) | 7 lines
  
  Memory leaking in calendars
  
  ne_request_destroy() was missing in icalendar and exchange calendar modules, causing memory leak.
  
  (closes issue #18521)
  Review: https://reviewboard.asterisk.org/r/1068/
........

Modified:
    trunk/   (props changed)
    trunk/res/res_calendar_exchange.c
    trunk/res/res_calendar_icalendar.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: trunk/res/res_calendar_exchange.c
URL: http://svnview.digium.com/svn/asterisk/trunk/res/res_calendar_exchange.c?view=diff&rev=300215&r1=300214&r2=300215
==============================================================================
--- trunk/res/res_calendar_exchange.c (original)
+++ trunk/res/res_calendar_exchange.c Tue Jan  4 11:04:14 2011
@@ -403,6 +403,7 @@
 	ne_add_request_header(req, "Content-type", "text/xml");
 
 	ret = ne_request_dispatch(req);
+	ne_request_destroy(req);
 
 	if (ret != NE_OK || !ast_str_strlen(response)) {
 		ast_log(LOG_WARNING, "Unknown response to CalDAV calendar %s, request %s to %s: %s\n", pvt->owner->name, method, pvt->url, ne_get_error(pvt->session));

Modified: trunk/res/res_calendar_icalendar.c
URL: http://svnview.digium.com/svn/asterisk/trunk/res/res_calendar_icalendar.c?view=diff&rev=300215&r1=300214&r2=300215
==============================================================================
--- trunk/res/res_calendar_icalendar.c (original)
+++ trunk/res/res_calendar_icalendar.c Tue Jan  4 11:04:14 2011
@@ -142,6 +142,7 @@
 	ne_add_response_body_reader(req, ne_accept_2xx, fetch_response_reader, &response);
 
 	ret = ne_request_dispatch(req);
+	ne_request_destroy(req);
 	if (ret != NE_OK || !ast_str_strlen(response)) {
 		ast_log(LOG_WARNING, "Unable to retrieve iCalendar '%s' from '%s': %s\n", pvt->owner->name, pvt->url, ne_get_error(pvt->session));
 		ast_free(response);




More information about the asterisk-commits mailing list