From 132ff4320dba21b167aee9ec5888eca265e32897 Mon Sep 17 00:00:00 2001 From: Don Garrett Date: Fri, 12 Nov 2010 15:29:30 -0800 Subject: [PATCH] cros_generate_upload_payload --patch_kernel should mount from the source image read only to avoid unintended changes. Change-Id: Id3a83786f2c1240a4401978c8cacffc0dcb9d66f BUG=9073 TEST=Ran script with --patch_kernel, verified source image unmodified. Ran ./image_to_live.sh to verify valid update was generated. Review URL: http://codereview.chromium.org/4892002 --- cros_generate_update_payload | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cros_generate_update_payload b/cros_generate_update_payload index dd0a67c06e..3b5b9f9d6d 100755 --- a/cros_generate_update_payload +++ b/cros_generate_update_payload @@ -87,7 +87,7 @@ patch_kernel() { offset=$(($offset * 512)) sudo losetup -o "$offset" "$STATE_LOOP_DEV" "$IMAGE" STATE_MNT=$(mktemp -d /tmp/state.XXXXXX) - sudo mount "$STATE_LOOP_DEV" "$STATE_MNT" + sudo mount --read-only "$STATE_LOOP_DEV" "$STATE_MNT" dd if="$STATE_MNT"/vmlinuz_hd.vblock of="$KERN_FILE" conv=notrunc sudo umount "$STATE_MNT" STATE_MNT=""