download.pl: properly cleanup intermediate .hash file
authorPetr Štetiar <ynezz@true.cz>
Fri, 27 Nov 2020 20:56:30 +0000 (21:56 +0100)
committerPetr Štetiar <ynezz@true.cz>
Fri, 27 Nov 2020 21:27:08 +0000 (22:27 +0100)
It seems like after a build the /dl dir seems to now contain a .hash
file for each source file due to inproper cleanup so fix it by removing
those intermediate files before leaving the download action.

Fixes: 4e19cbc55335 ("download: handle possibly invalid local tarballs")
Reported-by: Hannu Nyman <hannu.nyman@iki.fi>
Signed-off-by: Petr Štetiar <ynezz@true.cz>
scripts/download.pl

index 2d87f47f842bc7d2b9ccc70f43f1f504e1c8c798..84d10dbe5f9daea410c33677ef8cb3b90160f3ae 100755 (executable)
@@ -272,11 +272,11 @@ if (-f "$target/$filename") {
                $sum =~ /^(\w+)\s*/ or die "Could not generate file hash\n";
                $sum = $1;
 
+               cleanup();
                exit 0 if $sum eq $file_hash;
 
                die "Hash of the local file $filename does not match (file: $sum, requested: $file_hash) - deleting download.\n";
                unlink "$target/$filename";
-               cleanup();
        };
 }