There is no need for is{upper,lower}() tests before to{lower,uppper}(),

since all other characters are mapped through transparently.
ok jsing

1 files changed, 2 insertions, 4 deletions

diff --git a/src/lib/libcrypto/x509v3/v3_utl.c b/src/lib/libcrypto/x509v3/v3_utl.c
index bf6f81a5e9..4f4dc15221 100644
--- a/src/lib/libcrypto/x509v3/v3_utl.c
+++ b/src/lib/libcrypto/x509v3/v3_utl.c
@@ -468,10 +468,8 @@ string_to_hex(const char *str, long *len)
                         free(hexbuf);
                         return NULL;
                 }
-                if (isupper(ch))
-                        ch = tolower(ch);
-                if (isupper(cl))
-                        cl = tolower(cl);
+                ch = tolower(ch);
+                cl = tolower(cl);
 
                 if ((ch >= '0') && (ch <= '9'))
                         ch -= '0';