SolidHal has uploaded this change for review. ( https://review.coreboot.org/27436
Change subject: buildgcc: Actually verify checksums ......................................................................
buildgcc: Actually verify checksums
buildgcc doesn't verify the checksum of gcc downloaded using download(). This can be verified by modifying the gcc checksum in coreboot/util/crossgcc/sum, which won't cause a failure without this patch
Change-Id: Idcd583f853d0a3db5e6ccd25c27bc486c2c49691 Signed-off-by: Hal Emmerich Hal@halemmerich.com --- M util/crossgcc/buildgcc 1 file changed, 4 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/36/27436/1
diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc index 5083773..788bb65 100755 --- a/util/crossgcc/buildgcc +++ b/util/crossgcc/buildgcc @@ -320,10 +320,10 @@ cd tarballs download_showing_percentage $archive cd .. - if ! check_sum $FILE; then - printf "\n${RED}$FILE Failed Checksum Verification${NC}\n" - exit 1 - fi + if ! check_sum $FILE; then + printf "\n${RED}$FILE Failed Checksum Verification${NC}\n" + exit 1 + fi fi
if [ ! -f tarballs/$FILE ]; then