[asterisk-commits] dlee: branch 11 r409083 - in /branches/11: ./ utils/astman.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Feb 27 10:24:23 CST 2014


Author: dlee
Date: Thu Feb 27 10:24:20 2014
New Revision: 409083

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=409083
Log:
Fix memory stomping bug in astman.

This memset complained in dev mod on my Ubuntu box. The memset is both
unnecessary and dangerous. At this point, m hasn't been initialized
yet, so the memset will write off to whatever address happens to be
on the stack at the time.
........

Merged revisions 409077 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/11/   (props changed)
    branches/11/utils/astman.c

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

Modified: branches/11/utils/astman.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/utils/astman.c?view=diff&rev=409083&r1=409082&r2=409083
==============================================================================
--- branches/11/utils/astman.c (original)
+++ branches/11/utils/astman.c Thu Feb 27 10:24:20 2014
@@ -737,7 +737,6 @@
 					show_message("Login Failed", get_header(m, "Message"));
 				}
 			} else {
-				memset(m, 0, sizeof(m));
 				manager_action("Login", 
 					"Username: %s\r\n"
 					"Secret: %s\r\n",




More information about the asterisk-commits mailing list