refactor my_strupr

Refactor my_strupr such that it will not check and possibly alter bytes
after the string's null terminator for strings of length 0.

Signed-off-by: Guido Vranken <guidovranken@gmail.com>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <CAO5O-ELXwXgGtRFou=FY1TAhW4W4BHPZF4=rqSY877njtXNMJg@mail.gmail.com>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg14756.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>
(cherry picked from commit 69162924de3600bfe8ae9708a1d6e3f4515ef995)
(cherry picked from commit 8dd598e89f25074f2d4d23f77cb601cf48a810e3)

diff --git a/ntlm.c b/ntlm.c
index 0453358cc22a8aade5e93f50109513b68f38cb3d..5c3d6d6ad97696e3cb2b572834982cda52426a93 100644 (file)
--- a/ntlm.c
+++ b/ntlm.c
@@ -127,13 +127,16 @@ gen_nonce (unsigned char *nonce)
        }
 }
 
-unsigned char *my_strupr(unsigned char *str)
-{ 
-       /* converts string to uppercase in place */
-       unsigned char *tmp = str;;
+void
+my_strupr(unsigned char *str)
+{
+    /* converts string to uppercase in place */
 
-       do *str = toupper(*str); while (*(++str));
-       return tmp;
+    while (*str)
+    {
+        *str = toupper(*str);
+        str++;
+    }
 }
 
 static int