scripts: mkhash fix return code handling
authorPaul Spooren <mail@aparcar.org>
Fri, 17 Jul 2020 08:19:31 +0000 (22:19 -1000)
committerDaniel Golle <daniel@makrotopia.org>
Mon, 9 Nov 2020 10:54:29 +0000 (10:54 +0000)
If hashing a file fails mkhash shouldn't just silently fail. Now check
after each call of `hash_file()` the return and exit early in case of
errors. The return value which was previously ignored and would always
return 0.

Signed-off-by: Paul Spooren <mail@aparcar.org>
scripts/mkhash.c

index e26ca3a350148aa5ec500a5dee459f7b31f749cd..ce156e979dea3761a1e2684cf046be8cf505abe8 100644 (file)
@@ -823,8 +823,11 @@ int main(int argc, char **argv)
        if (argc < 2)
                return hash_file(t, NULL, add_filename);
 
-       for (i = 0; i < argc - 1; i++)
-               hash_file(t, argv[1 + i], add_filename);
+       for (i = 0; i < argc - 1; i++) {
+               int ret = hash_file(t, argv[1 + i], add_filename);
+               if (ret)
+                       return ret;
+       }
 
        return 0;
 }