[PATCH firmware-utils 2/4] uimage_sgehdr: use "char" type for header struct strings

Rafał Miłecki zajec5 at gmail.com
Thu Feb 22 22:18:37 PST 2024


From: Rafał Miłecki <rafal at milecki.pl>

This fixes:

src/uimage_sgehdr.c: In function ‘main’:
src/uimage_sgehdr.c:152:10: warning: pointer targets in passing argument 1 of ‘strncpy’ differ in signedness [-Wpointer-sign]
  strncpy(imgh->sgeih_p, model, sizeof(imgh->sgeih_p));
          ^~~~
src/uimage_sgehdr.c:153:10: warning: pointer targets in passing argument 1 of ‘strncpy’ differ in signedness [-Wpointer-sign]
  strncpy(imgh->sgeih_sv, sversion, sizeof(imgh->sgeih_sv));
          ^~~~
src/uimage_sgehdr.c:154:10: warning: pointer targets in passing argument 1 of ‘strncpy’ differ in signedness [-Wpointer-sign]
  strncpy(imgh->sgeih_hv, hversion, sizeof(imgh->sgeih_hv));
          ^~~~

Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
---
 src/uimage_sgehdr.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/uimage_sgehdr.c b/src/uimage_sgehdr.c
index 28143a8..6d62c04 100644
--- a/src/uimage_sgehdr.c
+++ b/src/uimage_sgehdr.c
@@ -40,9 +40,9 @@ struct image_header {
 	uint8_t		ih_type;	/* Image Type			*/
 	uint8_t		ih_comp;	/* Compression Type		*/
 	uint8_t		ih_name[IH_NMLEN];	/* Image Name		*/
-	uint8_t		sgeih_p[SGE_PRODUCTLEN];	/* sge_Product		*/
-	uint8_t		sgeih_sv[SGE_VERSIONLEN];	/* sge Software Version		*/
-	uint8_t		sgeih_hv[SGE_VERSIONLEN];	/* sge Hardware Version		*/
+	char		sgeih_p[SGE_PRODUCTLEN];	/* sge_Product		*/
+	char		sgeih_sv[SGE_VERSIONLEN];	/* sge Software Version		*/
+	char		sgeih_hv[SGE_VERSIONLEN];	/* sge Hardware Version		*/
 };
 
 
-- 
2.35.3




More information about the openwrt-devel mailing list