diff options
| author | Konstantin Porotchkin <kostap@marvell.com> | 2018-06-12 13:56:01 +0300 |
|---|---|---|
| committer | Kostya Porotchkin <kostap@marvell.com> | 2018-06-12 16:25:39 +0300 |
| commit | 81f04dcfa59af07d7a5f71a6671fc6205e870e87 (patch) | |
| tree | f821479f9aeca27b1bf4e5db33310fba90257101 /tools | |
| parent | fb05212f87fcbe28a88511806596c30cb9b6858b (diff) | |
Leave space for trailing \0 after MAX_FILENAME
Detected by gcc 8.0.1 warnings
Change-Id: Ia12d13194fa1671899ccea0c1bb9fa945fb8effe
Signed-off-by: Bernhard Rosenkränzer <bero@lindev.ch>
Signed-off-by: Konstantin Porotchkin <kostap@marvell.com>
Reviewed-on: http://vgitil04.il.marvell.com:8080/56635
Reviewed-by: Igal Liberman <igall@marvell.com>
Tested-by: iSoC Platform CI <ykjenk@marvell.com>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/doimage/doimage.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/tools/doimage/doimage.c b/tools/doimage/doimage.c index 7d3ae827..299f40fa 100644 --- a/tools/doimage/doimage.c +++ b/tools/doimage/doimage.c @@ -130,9 +130,9 @@ typedef struct _sec_entry { typedef struct _sec_options { #ifdef CONFIG_MVEBU_SECURE_BOOT - char aes_key_file[MAX_FILENAME]; - char kak_key_file[MAX_FILENAME]; - char csk_key_file[CSK_ARR_SZ][MAX_FILENAME]; + char aes_key_file[MAX_FILENAME+1]; + char kak_key_file[MAX_FILENAME+1]; + char csk_key_file[CSK_ARR_SZ][MAX_FILENAME+1]; uint32_t box_id; uint32_t flash_id; uint32_t jtag_delay; @@ -150,8 +150,8 @@ typedef struct _sec_options { } sec_options; typedef struct _options { - char bin_ext_file[MAX_FILENAME]; - char sec_cfg_file[MAX_FILENAME]; + char bin_ext_file[MAX_FILENAME+1]; + char sec_cfg_file[MAX_FILENAME+1]; sec_options *sec_opts; uint32_t load_addr; uint32_t exec_addr; @@ -1422,9 +1422,9 @@ error: int main(int argc, char *argv[]) { - char in_file[MAX_FILENAME]; - char out_file[MAX_FILENAME]; - char ext_file[MAX_FILENAME]; + char in_file[MAX_FILENAME+1]; + char out_file[MAX_FILENAME+1]; + char ext_file[MAX_FILENAME+1]; FILE *in_fd = NULL; FILE *out_fd = NULL; int parse = 0; |