[syslinux:pathbased] sha256/512: fix signedness errors

syslinux-bot for H. Peter Anvin hpa at linux.intel.com
Wed May 12 16:12:14 PDT 2010


Commit-ID:  9b9087c159bc9eccbb55c1cc4e71228fbcb86cbf
Gitweb:     http://syslinux.zytor.com/commit/9b9087c159bc9eccbb55c1cc4e71228fbcb86cbf
Author:     H. Peter Anvin <hpa at linux.intel.com>
AuthorDate: Wed, 12 May 2010 15:40:46 -0700
Committer:  H. Peter Anvin <hpa at linux.intel.com>
CommitDate: Wed, 12 May 2010 15:40:46 -0700

sha256/512: fix signedness errors

Fix signedness errors as part of -Werror cleanup.

Signed-off-by: H. Peter Anvin <hpa at linux.intel.com>


---
 com32/libutil/sha256crypt.c |   12 ++++++------
 com32/libutil/sha512crypt.c |   10 +++++-----
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/com32/libutil/sha256crypt.c b/com32/libutil/sha256crypt.c
index 2e94535..adc7b09 100644
--- a/com32/libutil/sha256crypt.c
+++ b/com32/libutil/sha256crypt.c
@@ -276,13 +276,13 @@ static const char sha256_salt_prefix[] = "$5$";
 static const char sha256_rounds_prefix[] = "rounds=";
 
 /* Maximum salt string length.  */
-#define SALT_LEN_MAX 16
+#define SALT_LEN_MAX 16U
 /* Default number of rounds if not explicitly specified.  */
-#define ROUNDS_DEFAULT 5000
+#define ROUNDS_DEFAULT 5000UL
 /* Minimum number of rounds.  */
-#define ROUNDS_MIN 1000
+#define ROUNDS_MIN 1000UL
 /* Maximum number of rounds.  */
-#define ROUNDS_MAX 999999999
+#define ROUNDS_MAX 999999999UL
 
 /* Table with characters for base64 transformation.  */
 static const char b64t[64] =
@@ -379,7 +379,7 @@ static char *sha256_crypt_r(const char *key, const char *salt, char *buffer,
 
     /* Take the binary representation of the length of the key and for every
        1 add the alternate sum, for every 0 the key.  */
-    for (cnt = key_len; cnt > 0; cnt >>= 1)
+    for (cnt = key_len; cnt; cnt >>= 1)
 	if ((cnt & 1) != 0)
 	    sha256_process_bytes(alt_result, 32, &ctx);
 	else
@@ -408,7 +408,7 @@ static char *sha256_crypt_r(const char *key, const char *salt, char *buffer,
     sha256_init_ctx(&alt_ctx);
 
     /* For every character in the password add the entire password.  */
-    for (cnt = 0; cnt < 16 + alt_result[0]; ++cnt)
+    for (cnt = 0; cnt < (size_t)16 + alt_result[0]; ++cnt)
 	sha256_process_bytes(salt, salt_len, &alt_ctx);
 
     /* Finish the digest.  */
diff --git a/com32/libutil/sha512crypt.c b/com32/libutil/sha512crypt.c
index df839af..9db9c0c 100644
--- a/com32/libutil/sha512crypt.c
+++ b/com32/libutil/sha512crypt.c
@@ -311,13 +311,13 @@ static const char sha512_salt_prefix[] = "$6$";
 static const char sha512_rounds_prefix[] = "rounds=";
 
 /* Maximum salt string length.  */
-#define SALT_LEN_MAX 16
+#define SALT_LEN_MAX 16U
 /* Default number of rounds if not explicitly specified.  */
-#define ROUNDS_DEFAULT 5000
+#define ROUNDS_DEFAULT 5000UL
 /* Minimum number of rounds.  */
-#define ROUNDS_MIN 1000
+#define ROUNDS_MIN 1000UL
 /* Maximum number of rounds.  */
-#define ROUNDS_MAX 999999999
+#define ROUNDS_MAX 999999999UL
 
 /* Table with characters for base64 transformation.  */
 static const char b64t[64] =
@@ -443,7 +443,7 @@ static char *sha512_crypt_r(const char *key, const char *salt, char *buffer,
     sha512_init_ctx(&alt_ctx);
 
     /* For every character in the password add the entire password.  */
-    for (cnt = 0; cnt < 16 + alt_result[0]; ++cnt)
+    for (cnt = 0; cnt < (size_t)16 + alt_result[0]; ++cnt)
 	sha512_process_bytes(salt, salt_len, &alt_ctx);
 
     /* Finish the digest.  */



More information about the Syslinux-commits mailing list