[asterisk-commits] dlee: branch 12 r409087 - in /branches/12: ./ utils/astman.c

[SVN commits to the Asterisk project]

Author: dlee
Date: Thu Feb 27 10:25:32 2014
New Revision: 409087

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=409087
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
........

Merged revisions 409083 from http://svn.asterisk.org/svn/asterisk/branches/11

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

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

Modified: branches/12/utils/astman.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/utils/astman.c?view=diff&rev=409087&r1=409086&r2=409087
==============================================================================
--- branches/12/utils/astman.c (original)
+++ branches/12/utils/astman.c Thu Feb 27 10:25:32 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",