[PATCH] image.h: make image_sign_info.fit point to const

Hannu Lounento hannu.lounento at vaisala.com
Mon Oct 18 07:49:03 CEST 2021


The data blob apparently does not need to be modified through the fit
field of the image_sign_info struct so make it point to const to avoid
the need to cast away constness in functions that assign a pointer to
const data to the field.

fit_image_setup_verify already had to cast away constness as it assigned
a const void * argument to the field. The cast can now be removed.

Signed-off-by: Hannu Lounento <hannu.lounento at vaisala.com>
---
 common/image-fit-sig.c | 2 +-
 include/image.h        | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/common/image-fit-sig.c b/common/image-fit-sig.c
index 4edebbf2d32..63e5423c925 100644
--- a/common/image-fit-sig.c
+++ b/common/image-fit-sig.c
@@ -85,7 +85,7 @@ static int fit_image_setup_verify(struct image_sign_info *info,
 
 	memset(info, '\0', sizeof(*info));
 	info->keyname = fdt_getprop(fit, noffset, FIT_KEY_HINT, NULL);
-	info->fit = (void *)fit;
+	info->fit = fit;
 	info->node_offset = noffset;
 	info->name = algo_name;
 	info->checksum = image_get_checksum_algo(algo_name);
diff --git a/include/image.h b/include/image.h
index 34d13ada84b..fd662e74b41 100644
--- a/include/image.h
+++ b/include/image.h
@@ -1159,7 +1159,7 @@ struct image_sign_info {
 	const char *keydir;		/* Directory conaining keys */
 	const char *keyname;		/* Name of key to use */
 	const char *keyfile;		/* Filename of private or public key */
-	void *fit;			/* Pointer to FIT blob */
+	const void *fit;		/* Pointer to FIT blob */
 	int node_offset;		/* Offset of signature node */
 	const char *name;		/* Algorithm name */
 	struct checksum_algo *checksum;	/* Checksum algorithm information */
-- 
2.31.1



More information about the U-Boot mailing list