Without that fix we have compilation errors:
    nv_data-imei.c: In function ‘get_imei’:
    nv_data-imei.c:181:12: error: format ‘%d’ expects argument of type
    ‘int’, but argument 2 has type ‘size_t’ {aka ‘long unsigned int’}
    [-Werror=format=]
      181 |     printf("In addition it is also invalid"
          |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    ......
      184 |            len, IMEI_LENGTH);
          |            ~~~
          |            |
          |            size_t {aka long unsigned int}
    nv_data-imei.c:183:14: note: format string is defined here
      183 |            "%d digits instead of %d.\n",
          |             ~^
          |              |
          |              int
          |             %ld
    nv_data-imei.c:193:12: error: format ‘%d’ expects argument of type
    ‘int’, but argument 3 has type ‘size_t’ {aka ‘long unsigned int’}
    [-Werror=format=]
      193 |     printf("The '%s' "
          |            ^~~~~~~~~~~
    ......
      196 |            imei->optarg, len, IMEI_LENGTH);
          |                          ~~~
          |                          |
          |                          size_t {aka long unsigned int}
    nv_data-imei.c:195:14: note: format string is defined here
      195 |            "%d digits instead of %d.\n",
          |             ~^
          |              |
          |              int
          |             %ld

This issue was introduced in commit
1cd1a569bef21ed3bfaeda15fb210ef6fd218152 (tools: Add a new
nv_data-imei tool based on rfs-imei)

Signed-off-by: Denis 'GNUtoo' Carikli <gnu...@cyberdimension.org>
---
 tools/nv_data-imei.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/nv_data-imei.c b/tools/nv_data-imei.c
index 2c04079..c5301b7 100644
--- a/tools/nv_data-imei.c
+++ b/tools/nv_data-imei.c
@@ -180,7 +180,7 @@ static int get_imei(struct command *command, void *arg)
                                       imei->optarg);
                                printf("In addition it is also invalid"
                                       " as it is composed of "
-                                      "%d digits instead of %d.\n",
+                                      "%zd digits instead of %d.\n",
                                       len, IMEI_LENGTH);
                                return -EINVAL;
                        } else if (!str_is_digit) {
@@ -192,7 +192,7 @@ static int get_imei(struct command *command, void *arg)
                        } else if (!str_len_valid) {
                                printf("The '%s' "
                                       "IMEI is invalid as it is composed of "
-                                      "%d digits instead of %d.\n",
+                                      "%zd digits instead of %d.\n",
                                       imei->optarg, len, IMEI_LENGTH);
                                return -EINVAL;
                        }
-- 
2.35.1

_______________________________________________
Replicant mailing list
Replicant@osuosl.org
https://lists.osuosl.org/mailman/listinfo/replicant

Reply via email to