- add libbb function str_tolower to convert a string to lowercase.

[oweals/busybox.git] / coreutils / md5_sha1_sum.c

diff --git a/coreutils/md5_sha1_sum.c b/coreutils/md5_sha1_sum.c
index 93d894655764887ae63ee7bc20100fa9ad13d4e0..417e90b00d96c36a07fa7ca2d0dd493a5c1d3486 100644 (file)
--- a/coreutils/md5_sha1_sum.c
+++ b/coreutils/md5_sha1_sum.c
@@ -16,17 +16,12 @@ typedef enum { HASH_SHA1, HASH_MD5 } hash_algo_t;
 
 /* This might be useful elsewhere */
 static unsigned char *hash_bin_to_hex(unsigned char *hash_value,
-                               unsigned char hash_length)
+                               unsigned hash_length)
 {
-       int x, len, max;
-       unsigned char *hex_value;
-
-       max = (hash_length * 2) + 2;
-       hex_value = xmalloc(max);
-       for (x = len = 0; x < hash_length; x++) {
-               len += snprintf((char*)(hex_value + len), max - len, "%02x", hash_value[x]);
-       }
-       return (hex_value);
+       /* xzalloc zero-terminates */
+       char *hex_value = xzalloc((hash_length * 2) + 1);
+       bin2hex(hex_value, (char*)hash_value, hash_length);
+       return hex_value;
 }
 
 static uint8_t *hash_file(const char *filename, hash_algo_t hash_algo)
@@ -41,11 +36,13 @@ static uint8_t *hash_file(const char *filename, hash_algo_t hash_algo)
        void (*update)(const void*, size_t, void*);
        void (*final)(void*, void*);
 
-       if (strcmp(filename, "-") == 0) {
-               src_fd = STDIN_FILENO;
-       } else if(0 > (src_fd = open(filename, O_RDONLY))) {
-               bb_perror_msg("%s", filename);
-               return NULL;
+       src_fd = STDIN_FILENO;
+       if (NOT_LONE_DASH(filename)) {
+               src_fd = open(filename, O_RDONLY);
+               if (src_fd < 0) {
+                       bb_perror_msg("%s", filename);
+                       return NULL;
+               }
        }
 
        /* figure specific hash algorithims */
@@ -63,7 +60,7 @@ static uint8_t *hash_file(const char *filename, hash_algo_t hash_algo)
                bb_error_msg_and_die("algorithm not supported");
        }
 
-       while (0 < (count = read(src_fd, in_buf, 4096))) {
+       while (0 < (count = safe_read(src_fd, in_buf, 4096))) {
                update(in_buf, count, &context);
        }
 
@@ -81,11 +78,12 @@ static uint8_t *hash_file(const char *filename, hash_algo_t hash_algo)
        return hash_value;
 }
 
+int md5_sha1_sum_main(int argc, char **argv);
 int md5_sha1_sum_main(int argc, char **argv)
 {
        int return_value = EXIT_SUCCESS;
        uint8_t *hash_value;
-       unsigned int flags;
+       unsigned flags;
        hash_algo_t hash_algo = ENABLE_MD5SUM
                ? (ENABLE_SHA1SUM ? (**argv=='m' ? HASH_MD5 : HASH_SHA1) : HASH_MD5)
                : HASH_SHA1;
@@ -105,10 +103,10 @@ int md5_sha1_sum_main(int argc, char **argv)
        }
 
        if (argc == optind) {
-               argv[argc++] = "-";
+               argv[argc++] = (char*)"-";
        }
 
-       if (ENABLE_FEATURE_MD5_SHA1_SUM_CHECK && flags & FLAG_CHECK) {
+       if (ENABLE_FEATURE_MD5_SHA1_SUM_CHECK && (flags & FLAG_CHECK)) {
                FILE *pre_computed_stream;
                int count_total = 0;
                int count_failed = 0;
@@ -120,9 +118,8 @@ int md5_sha1_sum_main(int argc, char **argv)
                                ("only one argument may be specified when using -c");
                }
 
-               if (strcmp(file_ptr, "-") == 0) {
-                       pre_computed_stream = stdin;
-               } else {
+               pre_computed_stream = stdin;
+               if (NOT_LONE_DASH(file_ptr)) {
                        pre_computed_stream = xfopen(file_ptr, "r");
                }
 
@@ -131,6 +128,10 @@ int md5_sha1_sum_main(int argc, char **argv)
 
                        count_total++;
                        filename_ptr = strstr(line, "  ");
+                       /* handle format for binary checksums */
+                       if (filename_ptr == NULL) {
+                               filename_ptr = strstr(line, " *");
+                       }
                        if (filename_ptr == NULL) {
                                if (flags & FLAG_WARN) {
                                        bb_error_msg("invalid format");
@@ -162,9 +163,11 @@ int md5_sha1_sum_main(int argc, char **argv)
                        bb_error_msg("WARNING: %d of %d computed checksums did NOT match",
                                                 count_failed, count_total);
                }
+               /*
                if (fclose_if_not_stdin(pre_computed_stream) == EOF) {
                        bb_perror_msg_and_die("cannot close file %s", file_ptr);
                }
+               */
        } else {
                while (optind < argc) {
                        char *file_ptr = argv[optind++];