From bd3dad01b0c2d934462d70eeabb31abcd0310b3f Mon Sep 17 00:00:00 2001 From: Hung-Te Lin Date: Mon, 26 Sep 2011 10:59:46 +0800 Subject: [PATCH] sign_official_build: allow repacking firmware when executed by sudo To prevent execution permissions lost after being copied to /tmp, force adding a+rx to the staging file. BUG=chromium-os:20797 TEST=sudo sign_official_build.sh ssd \ x86-zgb-0.16.1089.0.bin ../../tests/devkeys ssd_image.bin Change-Id: Ibee12dbb3faea9f6b05600d1343620e0af8633fb Reviewed-on: http://gerrit.chromium.org/gerrit/8263 Tested-by: Hung-Te Lin Reviewed-by: Gaurav Shah Commit-Ready: Gaurav Shah --- scripts/image_signing/sign_official_build.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/scripts/image_signing/sign_official_build.sh b/scripts/image_signing/sign_official_build.sh index 59e49dfbb6..2ab0579604 100755 --- a/scripts/image_signing/sign_official_build.sh +++ b/scripts/image_signing/sign_official_build.sh @@ -354,6 +354,7 @@ resign_firmware_payload() { new_shellball=$(make_temp_file) cp -f "${firmware_bundle}" "${new_shellball}" + chmod a+rx "${new_shellball}" repack_firmware_bundle "${shellball_dir}" "${new_shellball}" sudo cp -f "${new_shellball}" "${firmware_bundle}" sudo chmod a+rx "${firmware_bundle}"