[U-Boot] [BUG] build warnings for tools/mtk_image.c

Hello Ryder,
commit 3b975a147c3c ("tools: MediaTek: add MTK boot header generation to mkimage") leads to build warnings:
In function ‘mtk_image_set_gen_header’, inlined from ‘mtk_image_set_header’ at tools/mtk_image.c:733:3: tools/mtk_image.c:659:2: warning: ‘strncpy’ specified bound 12 equals destination size [-Wstringop-truncation] strncpy(hdr->boot.name, bootname, sizeof(hdr->boot.name)); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ In function ‘mtk_brom_parse_imagename’, inlined from ‘mtk_image_check_params’ at tools/mtk_image.c:388:9: tools/mtk_image.c:325:5: warning: ‘strncpy’ specified bound 32 equals destination size [-Wstringop-truncation] strncpy(lk_name, val, sizeof(lk_name)); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
with gcc 8.2.0.
Please, provide a fix that avoids build warnings as these will be regarded as errors on Travis.
If lg_name and htr->boot shall be \0 terminated you could make a change like:
- strncpy(lk_name, val, sizeof(lk_name)); + strncpy(lk_name, val, sizeof(lk_name) - 1);
- strncpy(hdr->boot.name, bootname, sizeof(hdr->boot.name)); + strncpy(hdr->boot.name, bootname, sizeof(hdr->boot.name) - 1);
Best regards
Heinrich
participants (1)
-
Heinrich Schuchardt