]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
Fix memory stomping bug in astman.
authorDavid M. Lee <dlee@digium.com>
Thu, 27 Feb 2014 16:24:20 +0000 (16:24 +0000)
committerDavid M. Lee <dlee@digium.com>
Thu, 27 Feb 2014 16:24:20 +0000 (16:24 +0000)
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

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/11@409083 65c4cc65-6c06-0410-ace0-fbb531ad65f3

utils/astman.c

index e61ce2388cf62871ff3907ff6c3677b3bb9c539d..3e9ca8ffb8a424ac532855d25f803bf9e533be44 100644 (file)
@@ -737,7 +737,6 @@ static int manager_login(char *hostname)
                                        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",