commit c08feedcfdc14076b89f039ddc6d15f6067d773d
author anjiahao <anjiahao@xiaomi.com> Fri Jul 28 20:26:39 2023 +0800
committer Xiang Xiao <xiaoxiang781216@gmail.com> Sun Jul 30 01:20:36 2023 -0700
tree 2a46f3387555b469b4636806a80ee0ccafb04da0
parent 55b0d4a110b45cd609c2f0c405f627e74f7ba222

ymodem:fix compile warnings

rb_main.c:219:42: warning: 'snprintf' output may be truncated before the last format character [-Wformat-truncation=]
  219 |           snprintf(temp, PATH_MAX, "%s/%s", priv->foldname,

Signed-off-by: anjiahao <anjiahao@xiaomi.com>

system/ymodem/rb_main.c

diff --git a/system/ymodem/rb_main.c b/system/ymodem/rb_main.c
index 41878ea..1beb8c2 100644
--- a/system/ymodem/rb_main.c
+++ b/system/ymodem/rb_main.c
@@ -171,7 +171,7 @@
 
   if (ctx->packet_type == YMODEM_FILENAME_PACKET)
     {
-      char temp[PATH_MAX];
+      char temp[PATH_MAX + 1];
       FAR char *filename;
 
       if (priv->fd > 0)
@@ -216,7 +216,7 @@
 
       if (priv->foldname != NULL)
         {
-          snprintf(temp, PATH_MAX, "%s/%s", priv->foldname,
+          snprintf(temp, sizeof(temp), "%s/%s", priv->foldname,
                    filename);
           filename = temp;
         }

system/ymodem/ymodem.c

diff --git a/system/ymodem/ymodem.c b/system/ymodem/ymodem.c
index ca7e30e..1f8cf6f 100644
--- a/system/ymodem/ymodem.c
+++ b/system/ymodem/ymodem.c
@@ -231,7 +231,7 @@
   if ((total_seq & 0xff) - 1 == ctx->header[1])
     {
       ymodem_debug("recv_file: Received the previous packet that has"
-                   "been received, continue %lu %u\n", total_seq,
+                   "been received, continue %" PRIu32 " %u\n", total_seq,
                    ctx->header[1]);
 
       ctx->header[0] = ACK;
@@ -239,7 +239,7 @@
     }
   else if ((total_seq & 0xff) != ctx->header[1])
     {
-      ymodem_debug("recv_file: total seq error:%lu %u\n", total_seq,
+      ymodem_debug("recv_file: total seq error:%" PRIu32 " %u\n", total_seq,
                    ctx->header[1]);
       ctx->header[0] = CRC;
       goto recv_packet;