jfb2 writes in Bug 119:

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

diff --git a/coreutils/md5_sha1_sum.c b/coreutils/md5_sha1_sum.c
index f7c44c850dfa230d73c2225c382373bccfc77812..bd1c9fc29a60c6b085cc72ba0222373e8c0e73df 100644 (file)
--- a/coreutils/md5_sha1_sum.c
+++ b/coreutils/md5_sha1_sum.c
@@ -1,7 +1,7 @@
 /*
  *  Copyright (C) 2003 Glenn L. McGrath
- *  Copyright (C) 2003 Erik Andersen
- * 
+ *  Copyright (C) 2003-2004 Erik Andersen
+ *
  *  This program is free software; you can redistribute it and/or modify
  *  it under the terms of the GNU General Public License as published by
  *  the Free Software Foundation; either version 2 of the License, or
@@ -32,7 +32,7 @@
 #define FLAG_CHECK     2
 #define FLAG_WARN      4
 
-/* This might be usefull elsewhere */
+/* This might be useful elsewhere */
 static unsigned char *hash_bin_to_hex(unsigned char *hash_value,
                                                                          unsigned char hash_length)
 {
@@ -55,7 +55,7 @@ static uint8_t *hash_file(const char *filename, uint8_t hash_algo)
        int src_fd;
 
        if (strcmp(filename, "-") == 0) {
-               src_fd = fileno(stdin);
+               src_fd = STDIN_FILENO;
        } else {
                src_fd = open(filename, O_RDONLY);
        }
@@ -82,14 +82,13 @@ static uint8_t *hash_file(const char *filename, uint8_t hash_algo)
 /* This could become a common function for md5 as well, by using md5_stream */
 extern int hash_files(int argc, char **argv, const uint8_t hash_algo)
 {
-       uint8_t *hash_value;
-       unsigned int flags;
        int return_value = EXIT_SUCCESS;
+       uint8_t *hash_value;
 
 #ifdef CONFIG_FEATURE_MD5_SHA1_SUM_CHECK
+       unsigned int flags;
+
        flags = bb_getopt_ulflags(argc, argv, "scw");
-#else
-       flags = bb_getopt_ulflags(argc, argv, "s");
 #endif
 
 #ifdef CONFIG_FEATURE_MD5_SHA1_SUM_CHECK
@@ -144,16 +143,19 @@ extern int hash_files(int argc, char **argv, const uint8_t hash_algo)
                        hash_value = hash_file(filename_ptr, hash_algo);
 
                        if (hash_value && (strcmp(hash_value, line) == 0)) {
-                               printf("%s: OK\n", filename_ptr);
+                               if (!(flags & FLAG_SILENT))
+                                       printf("%s: OK\n", filename_ptr);
                        } else {
-                               printf("%s: FAILED\n", filename_ptr);
+                               if (!(flags & FLAG_SILENT))
+                                       printf("%s: FAILED\n", filename_ptr);
                                count_failed++;
+                               return_value = EXIT_FAILURE;
                        }
                        /* possible free(NULL) */
                        free(hash_value);
                        free(line);
                }
-               if (count_failed) {
+               if (count_failed && !(flags & FLAG_SILENT)) {
                        bb_error_msg("WARNING: %d of %d computed checksums did NOT match",
                                                 count_failed, count_total);
                }
@@ -162,21 +164,28 @@ extern int hash_files(int argc, char **argv, const uint8_t hash_algo)
                }
        } else
 #endif
+       {
+               uint8_t hash_length;
+
+               if (hash_algo == HASH_MD5) {
+                       hash_length = 16;
+               } else {
+                       hash_length = 20;
+               }
+               hash_value = xmalloc(hash_length);
+
                while (optind < argc) {
-                       unsigned char *file_ptr = argv[optind];
+                       unsigned char *file_ptr = argv[optind++];
 
-                       optind++;
-                       
                        hash_value = hash_file(file_ptr, hash_algo);
                        if (hash_value == NULL) {
-                               return_value++;
-                       }
-                       else if (!flags & FLAG_SILENT) {
+                               return_value = EXIT_FAILURE;
+                       } else {
                                printf("%s  %s\n", hash_value, file_ptr);
                                free(hash_value);
                        }
                }
-
+       }
        return (return_value);
 }