diff options
author | Matt Johnston <matt@ucc.asn.au> | 2012-02-22 22:05:24 +0800 |
---|---|---|
committer | Matt Johnston <matt@ucc.asn.au> | 2012-02-22 22:05:24 +0800 |
commit | edf203caf44582be218c7d33797951b179cd5e48 (patch) | |
tree | d4137ebcedb099d03286a0210ed565a4703df302 | |
parent | 13fb6e7a58672abdb387abe891de5906b24ef7e7 (diff) | |
download | dropbear-edf203caf44582be218c7d33797951b179cd5e48.tar.gz |
- Fix minor leak
-rw-r--r-- | svr-authpam.c | 17 |
1 files changed, 4 insertions, 13 deletions
diff --git a/svr-authpam.c b/svr-authpam.c index f230be7..e84f076 100644 --- a/svr-authpam.c +++ b/svr-authpam.c @@ -56,11 +56,7 @@ pamConvFunc(int num_msg, struct UserDataS* userDatap = (struct UserDataS*) appdata_ptr; unsigned int msg_len = 0; unsigned int i = 0; - - const char* message = (*msg)->msg; - - /* make a copy we can strip */ - char * compare_message = m_strdup(message); + char * compare_message = NULL; TRACE(("enter pamConvFunc")) @@ -71,15 +67,10 @@ pamConvFunc(int num_msg, dropbear_log(LOG_INFO, "pamConvFunc() called with >1 messages: not supported."); return PAM_CONV_ERR; } - - TRACE(("msg_style is %d", (*msg)->msg_style)) - if (compare_message) { - TRACE(("message is '%s'", compare_message)) - } else { - TRACE(("null message")) - } - + /* make a copy we can strip */ + compare_message = m_strdup((*msg)->msg); + /* Make the string lowercase. */ msg_len = strlen(compare_message); for (i = 0; i < msg_len; i++) { |