]> git.ipfire.org Git - thirdparty/util-linux.git/commitdiff
lib/md5: use ul_/UL_ prefix
authorKarel Zak <kzak@redhat.com>
Tue, 12 Dec 2017 10:54:08 +0000 (11:54 +0100)
committerKarel Zak <kzak@redhat.com>
Thu, 14 Dec 2017 14:48:21 +0000 (15:48 +0100)
The symbols names are too generic.

Addresses: https://github.com/karelzak/util-linux/issues/548
Signed-off-by: Karel Zak <kzak@redhat.com>
disk-utils/mkfs.cramfs.c
include/md5.h
lib/md5.c
libblkid/src/superblocks/hfs.c
libuuid/src/gen_uuid.c
misc-utils/mcookie.c
tests/helpers/test_md5.c

index a3e9aa48c72aac86e4f9c35d44deab5341d7700e..729765078b7b3e225b9cfa71f9917b25fb237b86 100644 (file)
@@ -98,7 +98,7 @@ struct entry {
        /* stats */
        unsigned char *name;
        unsigned int mode, size, uid, gid;
-       unsigned char md5sum[MD5LENGTH];
+       unsigned char md5sum[UL_MD5LENGTH];
        unsigned char flags;       /* CRAMFS_EFLAG_* */
 
        /* FS data */
@@ -194,16 +194,17 @@ do_munmap(char *start, unsigned int size, unsigned int mode){
 /* compute md5sums, so that we do not have to compare every pair of files */
 static void
 mdfile(struct entry *e) {
-       MD5_CTX ctx;
        char *start;
 
        start = do_mmap(e->path, e->size, e->mode);
        if (start == NULL) {
                e->flags |= CRAMFS_EFLAG_INVALID;
        } else {
-               MD5Init(&ctx);
-               MD5Update(&ctx, (unsigned char *) start, e->size);
-               MD5Final(e->md5sum, &ctx);
+               UL_MD5_CTX ctx;
+
+               ul_MD5Init(&ctx);
+               ul_MD5Update(&ctx, (unsigned char *) start, e->size);
+               ul_MD5Final(e->md5sum, &ctx);
 
                do_munmap(start, e->size, e->mode);
 
@@ -255,7 +256,7 @@ static int find_identical_file(struct entry *orig, struct entry *new, loff_t *fs
 
                if ((orig->flags & CRAMFS_EFLAG_MD5) &&
                    (new->flags & CRAMFS_EFLAG_MD5) &&
-                   !memcmp(orig->md5sum, new->md5sum, MD5LENGTH) &&
+                   !memcmp(orig->md5sum, new->md5sum, UL_MD5LENGTH) &&
                    identical_file(orig, new)) {
                        new->same = orig;
                        *fslen_ub -= new->size;
index d997e379d7cf0f9112fc2c8e38a404bc72555a78..d6991e1fdea38e8111e7646f6cd95aa72c4b3417 100644 (file)
@@ -1,29 +1,24 @@
-#ifndef MD5_H
-#define MD5_H
+#ifndef UTIL_LINUX_MD5_H
+#define UTIL_LINUX_MD5_H
 
-#ifdef HAVE_STDINT_H
 #include <stdint.h>
-#else
-typedef unsigned int uint32_t;
-#endif
 
-#define MD5LENGTH 16
+#define UL_MD5LENGTH 16
 
-struct MD5Context {
+struct UL_MD5Context {
        uint32_t buf[4];
        uint32_t bits[2];
        unsigned char in[64];
 };
 
-void MD5Init(struct MD5Context *context);
-void MD5Update(struct MD5Context *context, unsigned char const *buf,
-              unsigned len);
-void MD5Final(unsigned char digest[MD5LENGTH], struct MD5Context *context);
-void MD5Transform(uint32_t buf[4], uint32_t const in[16]);
+void ul_MD5Init(struct UL_MD5Context *context);
+void ul_MD5Update(struct UL_MD5Context *context, unsigned char const *buf, unsigned len);
+void ul_MD5Final(unsigned char digest[UL_MD5LENGTH], struct UL_MD5Context *context);
+void ul_MD5Transform(uint32_t buf[4], uint32_t const in[16]);
 
 /*
  * This is needed to make RSAREF happy on some MS-DOS compilers.
  */
-typedef struct MD5Context MD5_CTX;
+typedef struct UL_MD5Context UL_MD5_CTX;
 
-#endif /* !MD5_H */
+#endif /* !UTIL_LINUX_MD5_H */
index 282e2d22af25cc2645f2e37a50143f504d3faa0f..3765ab93edd6f5c313c2fa881b7dc4c7612fc08d 100644 (file)
--- a/lib/md5.c
+++ b/lib/md5.c
@@ -19,7 +19,7 @@
 #include "md5.h"
 
 #if !defined(WORDS_BIGENDIAN)
-#define byteReverse(buf, len)  /* Nothing */
+# define byteReverse(buf, len) /* Nothing */
 #else
 static void byteReverse(unsigned char *buf, unsigned longs);
 
@@ -37,14 +37,14 @@ static void byteReverse(unsigned char *buf, unsigned longs)
        buf += 4;
     } while (--longs);
 }
-#endif
-#endif
+#endif /* !ASM_MD5 */
+#endif /* !WORDS_BIGENDIAN */
 
 /*
  * Start MD5 accumulation.  Set bit count to 0 and buffer to mysterious
  * initialization constants.
  */
-void MD5Init(struct MD5Context *ctx)
+void ul_MD5Init(struct UL_MD5Context *ctx)
 {
     ctx->buf[0] = 0x67452301;
     ctx->buf[1] = 0xefcdab89;
@@ -59,7 +59,7 @@ void MD5Init(struct MD5Context *ctx)
  * Update context to reflect the concatenation of another buffer full
  * of bytes.
  */
-void MD5Update(struct MD5Context *ctx, unsigned char const *buf, unsigned len)
+void ul_MD5Update(struct UL_MD5Context *ctx, unsigned char const *buf, unsigned len)
 {
     uint32_t t;
 
@@ -84,7 +84,7 @@ void MD5Update(struct MD5Context *ctx, unsigned char const *buf, unsigned len)
        }
        memcpy(p, buf, t);
        byteReverse(ctx->in, 16);
-       MD5Transform(ctx->buf, (uint32_t *) ctx->in);
+       ul_MD5Transform(ctx->buf, (uint32_t *) ctx->in);
        buf += t;
        len -= t;
     }
@@ -93,7 +93,7 @@ void MD5Update(struct MD5Context *ctx, unsigned char const *buf, unsigned len)
     while (len >= 64) {
        memcpy(ctx->in, buf, 64);
        byteReverse(ctx->in, 16);
-       MD5Transform(ctx->buf, (uint32_t *) ctx->in);
+       ul_MD5Transform(ctx->buf, (uint32_t *) ctx->in);
        buf += 64;
        len -= 64;
     }
@@ -104,10 +104,10 @@ void MD5Update(struct MD5Context *ctx, unsigned char const *buf, unsigned len)
 }
 
 /*
- * Final wrapup - pad to 64-byte boundary with the bit pattern 
+ * Final wrapup - pad to 64-byte boundary with the bit pattern
  * 1 0* (64-bit count of bits processed, MSB-first)
  */
-void MD5Final(unsigned char digest[MD5LENGTH], struct MD5Context *ctx)
+void ul_MD5Final(unsigned char digest[UL_MD5LENGTH], struct UL_MD5Context *ctx)
 {
     unsigned count;
     unsigned char *p;
@@ -128,7 +128,7 @@ void MD5Final(unsigned char digest[MD5LENGTH], struct MD5Context *ctx)
        /* Two lots of padding:  Pad the first block to 64 bytes */
        memset(p, 0, count);
        byteReverse(ctx->in, 16);
-       MD5Transform(ctx->buf, (uint32_t *) ctx->in);
+       ul_MD5Transform(ctx->buf, (uint32_t *) ctx->in);
 
        /* Now fill the next block with 56 bytes */
        memset(ctx->in, 0, 56);
@@ -145,9 +145,9 @@ void MD5Final(unsigned char digest[MD5LENGTH], struct MD5Context *ctx)
     memcpy(&ctx->in[14 * sizeof(uint32_t)], &ctx->bits[0], 4);
     memcpy(&ctx->in[15 * sizeof(uint32_t)], &ctx->bits[1], 4);
 
-    MD5Transform(ctx->buf, (uint32_t *) ctx->in);
+    ul_MD5Transform(ctx->buf, (uint32_t *) ctx->in);
     byteReverse((unsigned char *) ctx->buf, 4);
-    memcpy(digest, ctx->buf, MD5LENGTH);
+    memcpy(digest, ctx->buf, UL_MD5LENGTH);
     memset(ctx, 0, sizeof(*ctx));      /* In case it's sensitive */
 }
 
@@ -170,7 +170,7 @@ void MD5Final(unsigned char digest[MD5LENGTH], struct MD5Context *ctx)
  * reflect the addition of 16 longwords of new data.  MD5Update blocks
  * the data and converts bytes into longwords for this routine.
  */
-void MD5Transform(uint32_t buf[4], uint32_t const in[16])
+void ul_MD5Transform(uint32_t buf[4], uint32_t const in[16])
 {
     register uint32_t a, b, c, d;
 
index c2344114a8f1a0a0e5e05ff5194ca0e95bc64b94..19f14ed0d24641e573943cb7f36e1d0abe817841 100644 (file)
@@ -130,19 +130,21 @@ struct hfsplus_vol_header {
 
 static int hfs_set_uuid(blkid_probe pr, unsigned char const *hfs_info, size_t len)
 {
-       static unsigned char const hash_init[MD5LENGTH] = {
+       static unsigned char const hash_init[UL_MD5LENGTH] = {
                0xb3, 0xe2, 0x0f, 0x39, 0xf2, 0x92, 0x11, 0xd6,
                0x97, 0xa4, 0x00, 0x30, 0x65, 0x43, 0xec, 0xac
        };
-       unsigned char uuid[MD5LENGTH];
-       struct MD5Context md5c;
+       unsigned char uuid[UL_MD5LENGTH];
+       struct UL_MD5Context md5c;
 
        if (memcmp(hfs_info, "\0\0\0\0\0\0\0\0", len) == 0)
                return -1;
-       MD5Init(&md5c);
-       MD5Update(&md5c, hash_init, MD5LENGTH);
-       MD5Update(&md5c, hfs_info, len);
-       MD5Final(uuid, &md5c);
+
+       ul_MD5Init(&md5c);
+       ul_MD5Update(&md5c, hash_init, UL_MD5LENGTH);
+       ul_MD5Update(&md5c, hfs_info, len);
+       ul_MD5Final(uuid, &md5c);
+
        uuid[6] = 0x30 | (uuid[6] & 0x0f);
        uuid[8] = 0x80 | (uuid[8] & 0x3f);
        return blkid_probe_set_uuid(pr, uuid);
index 431bf2064a4f7b1d8f2e4f2a681c01ecb46444f7..a374e75c9e3d38af0b06cf38acf4d0012ed339a4 100644 (file)
@@ -564,15 +564,15 @@ void uuid_generate(uuid_t out)
  */
 void uuid_generate_md5(uuid_t out, const uuid_t ns, const char *name, size_t len)
 {
-       MD5_CTX ctx;
-       char hash[MD5LENGTH];
+       UL_MD5_CTX ctx;
+       char hash[UL_MD5LENGTH];
 
-       MD5Init(&ctx);
+       ul_MD5Init(&ctx);
        /* hash concatenation of well-known UUID with name */
-       MD5Update(&ctx, ns, sizeof(uuid_t));
-       MD5Update(&ctx, (const unsigned char *)name, len);
+       ul_MD5Update(&ctx, ns, sizeof(uuid_t));
+       ul_MD5Update(&ctx, (const unsigned char *)name, len);
 
-       MD5Final((unsigned char *)hash, &ctx);
+       ul_MD5Final((unsigned char *)hash, &ctx);
 
        memcpy(out, hash, sizeof(uuid_t));
 
index e6c799d2473641f22f9cd4e924732cc1df712d28..fd4227a098f7d5608b111ebbb9ef02e29b45270f 100644 (file)
@@ -41,7 +41,7 @@ enum {
 };
 
 struct mcookie_control {
-       struct  MD5Context ctx;
+       struct  UL_MD5Context ctx;
        char    **files;
        size_t  nfiles;
        uint64_t maxsz;
@@ -67,12 +67,12 @@ static uint64_t hash_file(struct mcookie_control *ctl, int fd)
                r = read_all(fd, (char *) buf, rdsz);
                if (r < 0)
                        break;
-               MD5Update(&ctl->ctx, buf, r);
+               ul_MD5Update(&ctl->ctx, buf, r);
                count += r;
        }
        /* Separate files with a null byte */
        buf[0] = '\0';
-       MD5Update(&ctl->ctx, buf, 1);
+       ul_MD5Update(&ctl->ctx, buf, 1);
        return count;
 }
 
@@ -131,7 +131,7 @@ int main(int argc, char **argv)
 {
        struct mcookie_control ctl = { .verbose = 0 };
        size_t i;
-       unsigned char digest[MD5LENGTH];
+       unsigned char digest[UL_MD5LENGTH];
        unsigned char buf[RAND_BYTES];
        int c;
 
@@ -180,14 +180,14 @@ int main(int argc, char **argv)
        free(ctl.files);
 
        random_get_bytes(&buf, RAND_BYTES);
-       MD5Update(&ctl.ctx, buf, RAND_BYTES);
+       ul_MD5Update(&ctl.ctx, buf, RAND_BYTES);
        if (ctl.verbose)
                fprintf(stderr, P_("Got %d byte from %s\n",
                                   "Got %d bytes from %s\n", RAND_BYTES),
                                RAND_BYTES, random_tell_source());
 
-       MD5Final(digest, &ctl.ctx);
-       for (i = 0; i < MD5LENGTH; i++)
+       ul_MD5Final(digest, &ctl.ctx);
+       for (i = 0; i < UL_MD5LENGTH; i++)
                printf("%02x", digest[i]);
        putchar('\n');
 
index 471580e12f919d584741a6610deece8e6514912d..6f8dec4aabc7614bb5687b9fa1e95548c65142f4 100644 (file)
@@ -7,22 +7,22 @@
 int main(void)
 {
        int i, ret;
-       struct MD5Context ctx;
-       unsigned char digest[MD5LENGTH];
+       struct UL_MD5Context ctx;
+       unsigned char digest[UL_MD5LENGTH];
        unsigned char buf[BUFSIZ];
 
-       MD5Init( &ctx );
+       ul_MD5Init( &ctx );
 
        while(!feof(stdin) && !ferror(stdin)) {
                ret = fread(buf, 1, sizeof(buf), stdin);
                if (ret)
-                       MD5Update( &ctx, buf, ret );
+                       ul_MD5Update( &ctx, buf, ret );
        }
 
        fclose(stdin);
-       MD5Final( digest, &ctx );
+       ul_MD5Final( digest, &ctx );
 
-       for (i = 0; i < MD5LENGTH; i++)
+       for (i = 0; i < UL_MD5LENGTH; i++)
                printf( "%02x", digest[i] );
        printf("\n");
        return 0;