git: 51f120a81042 - stable/12 - mkimg: Add support for offset if the source is an image

From: Li-Wen Hsu <lwhsu_at_FreeBSD.org>
Date: Wed, 02 Feb 2022 14:24:12 UTC
The branch stable/12 has been updated by lwhsu:

URL: https://cgit.FreeBSD.org/src/commit/?id=51f120a810427b802c46686350fff0f30a145633

commit 51f120a810427b802c46686350fff0f30a145633
Author:     Emmanuel Vadot <manu@FreeBSD.org>
AuthorDate: 2021-02-25 17:10:19 +0000
Commit:     Li-Wen Hsu <lwhsu@FreeBSD.org>
CommitDate: 2022-02-02 14:19:01 +0000

    mkimg: Add support for offset if the source is an image
    
    This allow us to create image with the following format:
    mkimg -v -o sdcard -s gpt -p efi:=esp_aarch64.img:1M -p freebsd-ufs::1G
    
    Which will add a efi partition at a 1M offset on the image with its content
    coming from the esp_aarch64.img file.
    
    MFC after:      3 days
    
    (cherry picked from commit 8f3c71c85e5f6a4d2bddbfead225d33b96dbd7d7)
---
 usr.bin/mkimg/mkimg.c | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/usr.bin/mkimg/mkimg.c b/usr.bin/mkimg/mkimg.c
index 8bfbeb753170..118bef5de2f2 100644
--- a/usr.bin/mkimg/mkimg.c
+++ b/usr.bin/mkimg/mkimg.c
@@ -174,8 +174,10 @@ usage(const char *why)
 	fprintf(stderr, "\t<t>[/<l>]::<size>[:[+]<offset>]\t-  "
 	    "empty partition of given size and\n\t\t\t\t\t"
 	    "   optional relative or absolute offset\n");
-	fprintf(stderr, "\t<t>[/<l>]:=<file>\t\t-  partition content and size "
-	    "are\n\t\t\t\t\t   determined by the named file\n");
+	fprintf(stderr, "\t<t>[/<l>]:=<file>[:[+]offset]\t-  partition "
+	    "content and size are\n\t\t\t\t\t"
+	    "   determined by the named file and\n"
+	    "\t\t\t\t\t   optional relative or absolute offset\n");
 	fprintf(stderr, "\t<t>[/<l>]:-<cmd>\t\t-  partition content and size "
 	    "are taken\n\t\t\t\t\t   from the output of the command to run\n");
 	fprintf(stderr, "\t-\t\t\t\t-  unused partition entry\n");
@@ -459,9 +461,11 @@ mkimg(void)
 		/* Look for an offset. Set size too if we can. */
 		switch (part->kind) {
 		case PART_KIND_SIZE:
+		case PART_KIND_FILE:
 			offset = part->contents;
 			size = strsep(&offset, ":");
-			if (expand_number(size, &bytesize) == -1)
+			if (part->kind == PART_KIND_SIZE &&
+			    expand_number(size, &bytesize) == -1)
 				error = errno;
 			if (offset != NULL) {
 				if (*offset != '+')
@@ -476,14 +480,13 @@ mkimg(void)
 
 		/* Work out exactly where the partition starts. */
 		blkoffset = (byteoffset + secsz - 1) / secsz;
-		if (abs_offset) {
-			part->block = scheme_metadata(SCHEME_META_PART_ABSOLUTE,
+		if (abs_offset)
+			block = scheme_metadata(SCHEME_META_PART_ABSOLUTE,
 			    blkoffset);
-		} else {
+		else
 			block = scheme_metadata(SCHEME_META_PART_BEFORE,
 			    block + blkoffset);
-			part->block = block;
-		}
+		part->block = block;
 
 		if (verbose)
 			fprintf(stderr, "partition %d: starting block %llu "