commit 5017f9b595da6e5c8f064a43fc6cd42cb62c082a
author Heinrich Schuchardt <heinrich.schuchardt@canonical.com> Tue Mar 01 08:53:56 2022 +0100
committer Tom Rini <trini@konsulko.com> Fri Mar 04 15:20:06 2022 -0500
tree bee8f82a0250f724954434b5a5eb19fc8c178ced
parent 55fd1c442e747338604ef0075a4a888a40399ddc

mkimage: error handling for FIT image

If parameter -F is given but FIT support is missing, a NULL pointer might
dereferenced (Coverity CID 350249).

If incorrect parameters are given, provide a message and show usage.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>

tools/mkimage.c

diff --git a/tools/mkimage.c b/tools/mkimage.c
index 7601451..74bd072 100644
--- a/tools/mkimage.c
+++ b/tools/mkimage.c
@@ -381,6 +381,11 @@
 	}
 
 	if (params.fflag){
+		if (!tparams) {
+			fprintf(stderr, "%s: Missing FIT support\n",
+				params.cmdname);
+			exit (EXIT_FAILURE);
+		}
 		if (tparams->fflag_handle)
 			/*
 			 * in some cases, some additional processing needs
@@ -391,7 +396,7 @@
 			retval = tparams->fflag_handle(&params);
 
 		if (retval != EXIT_SUCCESS)
-			exit (retval);
+			usage("Bad parameters for FIT image type");
 	}
 
 	if (params.lflag || params.fflag) {