crosutils: change mk_memento_images.sh to support arbitrary input source

When we want to separate kernel and rootfs, there must be a more flexible syntax
to assign input source.

The partition info parsing code is also improved to detect errors.

BUG=chromium-os:15050
TEST=./make_factory_package --release RECOVERY --factory FACTORY

Change-Id: Ie74b3e23117480a7f503488b39dedceadbfb41e3
Reviewed-on: http://gerrit.chromium.org/gerrit/5962
Reviewed-by: Nick Sanders <nsanders@chromium.org>
Tested-by: Hung-Te Lin <hungte@chromium.org>

lib/cros_image_common.sh

@@ -55,6 +55,8 @@ image_part_offset() {
   if image_has_command cgpt; then
     cgpt show -b -i "$partno" "$file"
   elif image_has_command parted; then
+    # First trial-run to make sure image is valid (because awk always return 0)
+    parted -m "$file" unit s print | grep -qs "^$partno:" || exit 1
     parted -m "$file" unit s print | awk -F ':' "/^$partno:/ { print int(\$2) }"
   elif [ -f "$unpack_file" ]; then
     awk "/ $partno  *Label:/ { print \$2 }" "$unpack_file"
@@ -73,6 +75,8 @@ image_part_size() {
   if image_has_command cgpt; then
     cgpt show -s -i "$partno" "$file"
   elif image_has_command parted; then
+    # First trial-run to make sure image is valid (because awk always return 0)
+    parted -m "$file" unit s print | grep -qs "^$partno:" || exit 1
     parted -m "$file" unit s print | awk -F ':' "/^$partno:/ { print int(\$4) }"
   elif [ -s "$unpack_file" ]; then
     awk "/ $partno  *Label:/ { print \$3 }" "$unpack_file"

make_factory_package.sh

@@ -230,7 +230,7 @@ compress_and_hash_memento_image() {
       grep hash |
       awk '{print $4}'
   else
-    sudo "${SCRIPTS_DIR}/mk_memento_images.sh" "$input_file" 2 3 |
+    sudo "${SCRIPTS_DIR}/mk_memento_images.sh" "$input_file:2" "$input_file:3" |
       grep hash |
       awk '{print $4}'
   fi

mk_memento_images.sh

@@ -37,9 +37,22 @@ if ! . "${SCRIPT_ROOT}/lib/cros_image_common.sh"; then
   exit 1
 fi
 
-if [ -z "$2" -o -z "$1" ] || [ "${#@}" -ne 2 -a "${#@}" -ne 3 ]; then
+# We need 2-3 non-zero parameters.
-  echo "usage: $0 path/to/kernel_partition_img path/to/rootfs_partition_img"
+if [ "$#" -lt 2 ] || [ "$#" -gt 3 ] || [ -z "$1" ] || [ -z "$2" ]; then
-  echo "    or $0 path/to/chromiumos_img kern_part_no rootfs_part_no"
+  echo "
+Usage: $0 kernel_partition_img[:index] rootfs_partition_img[:index] [output_dir]
+
+          Input parameters may be either a simple partition image file, or a
+          disk image file name followed by ':' and target partition index number
+
+          If output_dir is omitted, the folder of kernel_partition_img will be
+          use.
+
+Examples:
+       $0 part_2 part_3
+       $0 chromiumos_image.bin:2 part3
+       $0 chromiumos_image.bin:2 otherimage.bin:3 /tmp/myoutput
+  "
   exit 1
 fi
 
@@ -64,31 +77,46 @@ if [ $(whoami) = "root" ]; then
   echo "running $0 as root which is unneccessary"
 fi
 
-# Determine the offset size, and file name of parameters
+# Usage: load_partition_file VARIABLE_NAME_PREFIX partition_file
-if [ -z "$3" ]; then
+# Writes VARIABLE_NAME_PREFIX[, _OFFSE, _SIZE, _SECTORS] by parsing
-  # kernnel_img rootfs_img
+# partition_file, which can be a simple file or image:partno.
-  KPART="$1"
+load_partition_file() {
-  ROOT_PART="$2"
+  local var_prefix="$1"
-  KPART_SIZE=$(stat -c%s "$KPART")
+  local var="$2"
-  ROOT_PART_SIZE=$(stat -c%s "$ROOT_PART")
+  local var_offset=""
-  KPART_OFFSET=0
+  local var_size=""
-  KPART_SECTORS=$((KPART_SIZE / 512))
+  local var_sectors=""
-  ROOT_OFFSET=0
+  local part_no="${var##*:}"
-  ROOT_SECTORS=$((ROOT_PART_SIZE / 512))
+
-else
+  # test if var is in image:partno format.
-  # chromiumos_img kern_part_no rootfs_part_no
+  if [ "$part_no" != "$var" ]; then
-  KPART="$1"
+    var="${var%:*}"
-  ROOT_PART="$1"
+  else
-  KPART_OFFSET="$(image_part_offset "$KPART" "$2")" ||
+    part_no=""
-    image_die "cannot retieve kernel partition offset"
+  fi
-  KPART_SECTORS="$(image_part_size "$KPART" "$2")" ||
+
-    image_die "cannot retieve kernel partition size"
+  if [ -z "$part_no" ]; then
-  ROOT_OFFSET="$(image_part_offset "$ROOT_PART" "$3")" ||
+    var_offset=0
-    image_die "cannot retieve root partition offset"
+    var_size="$(stat -c"%s" "$var")" ||
-  ROOT_SECTORS="$(image_part_size "$ROOT_PART" "$3")" ||
+      image_die "Invalid file: $var"
-    image_die "cannot retieve root partition size"
+    var_sectors="$((var_size / 512))"
-  KPART_SIZE=$((KPART_SECTORS * 512))
+  else
-fi
+    var_offset="$(image_part_offset "$var" "$part_no")" ||
+      image_die "Cannot retieve offset for partition $var:$part_no"
+    var_sectors="$(image_part_size "$var" "$part_no")" ||
+      image_die "Cannot retieve size for partition $var:$part_no"
+    var_size=$((var_sectors * 512))
+  fi
+
+  # publish the values
+  eval "${var_prefix}"="$var"
+  eval "${var_prefix}_OFFSET"="$var_offset"
+  eval "${var_prefix}_SIZE"="$var_size"
+  eval "${var_prefix}_SECTORS"="$var_sectors"
+}
+
+load_partition_file KPART "$1"
+load_partition_file ROOTFS "$2"
 
 # Sanity check size.
 if [ "$KPART_SIZE" -gt $((16 * 1024 * 1024)) ]; then
@@ -97,7 +125,13 @@ if [ "$KPART_SIZE" -gt $((16 * 1024 * 1024)) ]; then
   exit 1
 fi
 
-FINAL_OUT_FILE=$(dirname "$1")/update.gz
+if [ "$#" = "3" ]; then
+  FINAL_OUT_DIR="$(readlink -f $3)"
+else
+  FINAL_OUT_DIR="$(dirname "$(readlink -f $1)")"
+fi
+FINAL_OUT_FILE="$FINAL_OUT_DIR/update.gz"
+echo "Output: $FINAL_OUT_FILE"
 
 # Update payload format:
 #  [kernel_size: big-endian uint64][kernel_blob][rootfs_blob]
@@ -114,7 +148,7 @@ CS_AND_RET_CODES="$(
     echo "Compressing kernel..." >&2
     image_dump_partial_file "$KPART" "$KPART_OFFSET" "$KPART_SECTORS"
     echo "Compressing rootfs..." >&2
-    image_dump_partial_file "$ROOT_PART" "$ROOT_OFFSET" "$ROOT_SECTORS") |
+    image_dump_partial_file "$ROOTFS" "$ROOTFS_OFFSET" "$ROOTFS_SECTORS") |
   image_gzip_compress -c -9 |
   tee "$FINAL_OUT_FILE" |
   openssl sha1 -binary |