aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--Makefile3
-rw-r--r--common.h6
-rw-r--r--libhashsum.h14
-rw-r--r--libhashsum_init_md4_hasher.c8
-rw-r--r--libhashsum_init_md5_hasher.c5
-rw-r--r--libhashsum_init_ripemd_128_hasher.c8
-rw-r--r--libhashsum_init_ripemd_160_hasher.c8
-rw-r--r--libhashsum_init_ripemd_256_hasher.c8
-rw-r--r--libhashsum_init_ripemd_320_hasher.c8
-rw-r--r--libhashsum_reverse_byte__.c12
10 files changed, 68 insertions, 12 deletions
diff --git a/Makefile b/Makefile
index 09db4c4..f1cfb34 100644
--- a/Makefile
+++ b/Makefile
@@ -33,7 +33,8 @@ OBJ =\
libhashsum_init_sha_512_hasher.o\
libhashsum_init_sha_512_224_hasher.o\
libhashsum_init_sha_512_256_hasher.o\
- libhashsum_init_sha2_hasher.o
+ libhashsum_init_sha2_hasher.o\
+ libhashsum_reverse_byte__.o
HDR =\
libhashsum.h\
diff --git a/common.h b/common.h
index 117972b..e70d922 100644
--- a/common.h
+++ b/common.h
@@ -11,6 +11,12 @@
#include <string.h>
+#if defined(__GNUC__)
+__attribute__((__const__))
+#endif
+uint8_t libhashsum_reverse_byte__(uint8_t);
+
+
#ifdef TEST
# include <stdlib.h>
# include <stdio.h>
diff --git a/libhashsum.h b/libhashsum.h
index 5fb71cf..0bfb48a 100644
--- a/libhashsum.h
+++ b/libhashsum.h
@@ -315,6 +315,13 @@ struct libhashsum_hasher {
* Update the hash state given it's final
* input data
*
+ * Regardless of the algorithm's standard, the function
+ * will takes the lower bits from `data[bytes]`, if
+ * `extra_bits > 0` and use the for the additional bits;
+ * the least significant bit will be used as the first
+ * bit and the most significant bit will be used as the
+ * last bit
+ *
* @param this The object containing this function pointer
* @param data The new input data, the function may rewrite it's content
* @param bytes The number of bytes available in `data` for reading
@@ -333,6 +340,13 @@ struct libhashsum_hasher {
* Update the hash state given it's final
* input data
*
+ * Regardless of the algorithm's standard, the function
+ * will takes the lower bits from `data[bytes]`, if
+ * `extra_bits > 0` and use the for the additional bits;
+ * the least significant bit will be used as the first
+ * bit and the most significant bit will be used as the
+ * last bit
+ *
* @param this The object containing this function pointer
* @param data The new input data, the function may rewrite it's content
* @param bytes The number of bytes available in `data` for reading
diff --git a/libhashsum_init_md4_hasher.c b/libhashsum_init_md4_hasher.c
index c51ebf8..f3c26ac 100644
--- a/libhashsum_init_md4_hasher.c
+++ b/libhashsum_init_md4_hasher.c
@@ -98,10 +98,12 @@ finalise_common(struct libhashsum_hasher *this, uint8_t *m, size_t bytes, unsign
this->state.md4.count *= 8U;
this->state.md4.count += (size_t)extra_bits;
- memset(&m[bytes], 0, 64U - bytes);
+ if (extra_bits)
+ m[bytes] = libhashsum_reverse_byte__(m[bytes]);
+ memset(&m[bytes + 1U], 0, 63U - bytes);
mask = (uint8_t)(1U << (7U - extra_bits));
m[bytes] |= mask;
- m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits */
+ m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits (original value was reversed) */
for (i = 0; i < 14; i++)
this->state.md4.m.m32[i] = LETO32(&m[i * 4U]);
@@ -143,6 +145,7 @@ finalise_const(struct libhashsum_hasher *this, const void *data, size_t bytes, u
m = &m[r];
bytes -= r;
+ this->state.md4.m.m8[bytes] = 0;
memcpy(this->state.md4.m.m8, m, bytes + (size_t)(extra_bits > 0U));
return finalise_common(this, this->state.md4.m.m8, bytes, extra_bits);
}
@@ -161,6 +164,7 @@ finalise(struct libhashsum_hasher *this, void *data, size_t bytes, unsigned extr
size -= r;
if (size < 64U) {
+ this->state.md4.m.m8[bytes] = 0;
memcpy(this->state.md4.m.m8, m, bytes + (size_t)(extra_bits > 0U));
m = this->state.md4.m.m8;
}
diff --git a/libhashsum_init_md5_hasher.c b/libhashsum_init_md5_hasher.c
index 292bd1e..7bf19af 100644
--- a/libhashsum_init_md5_hasher.c
+++ b/libhashsum_init_md5_hasher.c
@@ -109,9 +109,11 @@ finalise_common(struct libhashsum_hasher *this, uint8_t *m, size_t bytes, unsign
this->state.md5.count *= 8U;
this->state.md5.count += (size_t)extra_bits;
+ if (extra_bits)
+ m[bytes] = libhashsum_reverse_byte__(m[bytes]);
mask = (uint8_t)(1U << (7U - extra_bits));
m[bytes] |= mask;
- m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits */
+ m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits (original value was reversed) */
bytes++;
if (bytes > 56U) {
@@ -159,6 +161,7 @@ finalise_const(struct libhashsum_hasher *this, const void *data, size_t bytes, u
m = &m[r];
bytes -= r;
+ this->state.md5.m[bytes] = 0;
memcpy(this->state.md5.m, m, bytes + (size_t)(extra_bits > 0U));
return finalise_common(this, this->state.md5.m, bytes, extra_bits);
}
diff --git a/libhashsum_init_ripemd_128_hasher.c b/libhashsum_init_ripemd_128_hasher.c
index d4e818f..ef9f215 100644
--- a/libhashsum_init_ripemd_128_hasher.c
+++ b/libhashsum_init_ripemd_128_hasher.c
@@ -134,10 +134,12 @@ finalise_common(struct libhashsum_hasher *this, uint8_t *m, size_t bytes, unsign
this->state.ripemd_128.count *= 8U;
this->state.ripemd_128.count += (size_t)extra_bits;
- memset(&m[bytes], 0, 64U - bytes);
+ if (extra_bits)
+ m[bytes] = libhashsum_reverse_byte__(m[bytes]);
+ memset(&m[bytes + 1U], 0, 63U - bytes);
mask = (uint8_t)(1U << (7U - extra_bits));
m[bytes] |= mask;
- m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits */
+ m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits (original value was reversed) */
for (i = 0; i < 14; i++)
this->state.ripemd_128.m.m32[i] = LETO32(&m[i * 4U]);
@@ -179,6 +181,7 @@ finalise_const(struct libhashsum_hasher *this, const void *data, size_t bytes, u
m = &m[r];
bytes -= r;
+ this->state.ripemd_128.m.m8[bytes] = 0;
memcpy(this->state.ripemd_128.m.m8, m, bytes + (size_t)(extra_bits > 0U));
return finalise_common(this, this->state.ripemd_128.m.m8, bytes, extra_bits);
}
@@ -197,6 +200,7 @@ finalise(struct libhashsum_hasher *this, void *data, size_t bytes, unsigned extr
size -= r;
if (size < 64U) {
+ this->state.ripemd_128.m.m8[bytes] = 0;
memcpy(this->state.ripemd_128.m.m8, m, bytes + (size_t)(extra_bits > 0U));
m = this->state.ripemd_128.m.m8;
}
diff --git a/libhashsum_init_ripemd_160_hasher.c b/libhashsum_init_ripemd_160_hasher.c
index 476fa97..001cbb8 100644
--- a/libhashsum_init_ripemd_160_hasher.c
+++ b/libhashsum_init_ripemd_160_hasher.c
@@ -133,10 +133,12 @@ finalise_common(struct libhashsum_hasher *this, uint8_t *m, size_t bytes, unsign
this->state.ripemd_160.count *= 8U;
this->state.ripemd_160.count += (size_t)extra_bits;
- memset(&m[bytes], 0, 64U - bytes);
+ if (extra_bits)
+ m[bytes] = libhashsum_reverse_byte__(m[bytes]);
+ memset(&m[bytes + 1U], 0, 63U - bytes);
mask = (uint8_t)(1U << (7U - extra_bits));
m[bytes] |= mask;
- m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits */
+ m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits (original value was reversed) */
for (i = 0; i < 14; i++)
this->state.ripemd_160.m.m32[i] = LETO32(&m[i * 4U]);
@@ -182,6 +184,7 @@ finalise_const(struct libhashsum_hasher *this, const void *data, size_t bytes, u
m = &m[r];
bytes -= r;
+ this->state.ripemd_160.m.m8[bytes] = 0;
memcpy(this->state.ripemd_160.m.m8, m, bytes + (size_t)(extra_bits > 0U));
return finalise_common(this, this->state.ripemd_160.m.m8, bytes, extra_bits);
}
@@ -200,6 +203,7 @@ finalise(struct libhashsum_hasher *this, void *data, size_t bytes, unsigned extr
size -= r;
if (size < 64U) {
+ this->state.ripemd_160.m.m8[bytes] = 0;
memcpy(this->state.ripemd_160.m.m8, m, bytes + (size_t)(extra_bits > 0U));
m = this->state.ripemd_160.m.m8;
}
diff --git a/libhashsum_init_ripemd_256_hasher.c b/libhashsum_init_ripemd_256_hasher.c
index b1e9241..f08acf7 100644
--- a/libhashsum_init_ripemd_256_hasher.c
+++ b/libhashsum_init_ripemd_256_hasher.c
@@ -145,10 +145,12 @@ finalise_common(struct libhashsum_hasher *this, uint8_t *m, size_t bytes, unsign
this->state.ripemd_256.count *= 8U;
this->state.ripemd_256.count += (size_t)extra_bits;
- memset(&m[bytes], 0, 64U - bytes);
+ if (extra_bits)
+ m[bytes] = libhashsum_reverse_byte__(m[bytes]);
+ memset(&m[bytes + 1U], 0, 63U - bytes);
mask = (uint8_t)(1U << (7U - extra_bits));
m[bytes] |= mask;
- m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits */
+ m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits (original value was reversed) */
for (i = 0; i < 14; i++)
this->state.ripemd_256.m.m32[i] = LETO32(&m[i * 4U]);
@@ -190,6 +192,7 @@ finalise_const(struct libhashsum_hasher *this, const void *data, size_t bytes, u
m = &m[r];
bytes -= r;
+ this->state.ripemd_256.m.m8[bytes] = 0;
memcpy(this->state.ripemd_256.m.m8, m, bytes + (size_t)(extra_bits > 0U));
return finalise_common(this, this->state.ripemd_256.m.m8, bytes, extra_bits);
}
@@ -208,6 +211,7 @@ finalise(struct libhashsum_hasher *this, void *data, size_t bytes, unsigned extr
size -= r;
if (size < 64U) {
+ this->state.ripemd_256.m.m8[bytes] = 0;
memcpy(this->state.ripemd_256.m.m8, m, bytes + (size_t)(extra_bits > 0U));
m = this->state.ripemd_256.m.m8;
}
diff --git a/libhashsum_init_ripemd_320_hasher.c b/libhashsum_init_ripemd_320_hasher.c
index d9ee4aa..2f09ce7 100644
--- a/libhashsum_init_ripemd_320_hasher.c
+++ b/libhashsum_init_ripemd_320_hasher.c
@@ -147,10 +147,12 @@ finalise_common(struct libhashsum_hasher *this, uint8_t *m, size_t bytes, unsign
this->state.ripemd_320.count *= 8U;
this->state.ripemd_320.count += (size_t)extra_bits;
- memset(&m[bytes], 0, 64U - bytes);
+ if (extra_bits)
+ m[bytes] = libhashsum_reverse_byte__(m[bytes]);
+ memset(&m[bytes + 1U], 0, 63U - bytes);
mask = (uint8_t)(1U << (7U - extra_bits));
m[bytes] |= mask;
- m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits */
+ m[bytes] &= (uint8_t)~(mask - 1U); /* keep high bits (original value was reversed) */
for (i = 0; i < 14; i++)
this->state.ripemd_320.m.m32[i] = LETO32(&m[i * 4U]);
@@ -196,6 +198,7 @@ finalise_const(struct libhashsum_hasher *this, const void *data, size_t bytes, u
m = &m[r];
bytes -= r;
+ this->state.ripemd_320.m.m8[bytes] = 0;
memcpy(this->state.ripemd_320.m.m8, m, bytes + (size_t)(extra_bits > 0U));
return finalise_common(this, this->state.ripemd_320.m.m8, bytes, extra_bits);
}
@@ -214,6 +217,7 @@ finalise(struct libhashsum_hasher *this, void *data, size_t bytes, unsigned extr
size -= r;
if (size < 64U) {
+ this->state.ripemd_320.m.m8[bytes] = 0;
memcpy(this->state.ripemd_320.m.m8, m, bytes + (size_t)(extra_bits > 0U));
m = this->state.ripemd_320.m.m8;
}
diff --git a/libhashsum_reverse_byte__.c b/libhashsum_reverse_byte__.c
new file mode 100644
index 0000000..c608eb0
--- /dev/null
+++ b/libhashsum_reverse_byte__.c
@@ -0,0 +1,12 @@
+/* See LICENSE file for copyright and license details. */
+#include "common.h"
+
+
+uint8_t
+libhashsum_reverse_byte__(uint8_t x)
+{
+ x = (uint8_t)(((x & 0xAAU) >> 1) | ((x << 1) & 0xAAU));
+ x = (uint8_t)(((x & 0xCCU) >> 2) | ((x << 2) & 0xCCU));
+ x = (uint8_t)(((x & 0xF0U) >> 4) | ((x << 4) & 0xF0U));
+ return x;
+}