fix: fix uart && shell string buffer bugs

Author: libdriver

project/raspberrypi4b/driver/src/raspberrypi4b_driver_at24cxx_interface.c

@@ -155,12 +155,12 @@ void at24cxx_interface_delay_ms(uint32_t ms)
 void at24cxx_interface_debug_print(const char *const fmt, ...)
 {
     char str[256];
-    uint8_t len;
+    uint16_t len;
     va_list args;
 
     memset((char *)str, 0, sizeof(char) * 256); 
     va_start(args, fmt);
-    vsnprintf((char *)str, 256, (char const *)fmt, args);
+    vsnprintf((char *)str, 255, (char const *)fmt, args);
     va_end(args);
 
     len = strlen((char *)str);

project/raspberrypi4b/src/main.c

@@ -67,7 +67,7 @@ uint8_t at24cxx(uint8_t argc, char **argv)
         {"type", required_argument, NULL, 4},
         {NULL, 0, NULL, 0},
     };
-    char type[32] = "unknow";
+    char type[33] = "unknow";
     uint8_t data = rand() % 0xFF;
     uint16_t reg = 0x0000;
     at24cxx_t chip_type = AT24C01;
@@ -96,7 +96,7 @@ uint8_t at24cxx(uint8_t argc, char **argv)
             case 'h' :
             {
                 /* set the type */
-                memset(type, 0, sizeof(char) * 32);
+                memset(type, 0, sizeof(char) * 33);
                 snprintf(type, 32, "h");
 
                 break;
@@ -106,7 +106,7 @@ uint8_t at24cxx(uint8_t argc, char **argv)
             case 'i' :
             {
                 /* set the type */
-                memset(type, 0, sizeof(char) * 32);
+                memset(type, 0, sizeof(char) * 33);
                 snprintf(type, 32, "i");
 
                 break;
@@ -116,7 +116,7 @@ uint8_t at24cxx(uint8_t argc, char **argv)
             case 'p' :
             {
                 /* set the type */
-                memset(type, 0, sizeof(char) * 32);
+                memset(type, 0, sizeof(char) * 33);
                 snprintf(type, 32, "p");
 
                 break;
@@ -126,7 +126,7 @@ uint8_t at24cxx(uint8_t argc, char **argv)
             case 'e' :
             {
                 /* set the type */
-                memset(type, 0, sizeof(char) * 32);
+                memset(type, 0, sizeof(char) * 33);
                 snprintf(type, 32, "e_%s", optarg);
 
                 break;
@@ -136,7 +136,7 @@ uint8_t at24cxx(uint8_t argc, char **argv)
             case 't' :
             {
                 /* set the type */
-                memset(type, 0, sizeof(char) * 32);
+                memset(type, 0, sizeof(char) * 33);
                 snprintf(type, 32, "t_%s", optarg);
 
                 break;

project/stm32f407/EW/stm32f407.dep

@@ -5,12 +5,12 @@
     <configuration>
         <name>Release</name>
         <outputs>
-            <file>$PROJ_DIR$\..\usr\src\getopt.c</file>
-            <file>$PROJ_DIR$\..\usr\src\stm32f4xx_it.c</file>
-            <file>$PROJ_DIR$\..\usr\src\shell.c</file>
             <file>$PROJ_DIR$\..\usr\src\main.c</file>
+            <file>$PROJ_DIR$\..\usr\src\shell.c</file>
+            <file>$PROJ_DIR$\..\usr\src\getopt.c</file>
             <file>$PROJ_DIR$\..\usr\src\stm32f4xx_hal_msp.c</file>
             <file>$PROJ_DIR$\..\usr\src\system_stm32f4xx.c</file>
+            <file>$PROJ_DIR$\..\usr\src\stm32f4xx_it.c</file>
         </outputs>
         <forcedrebuild>
             <name>[REBUILD_ALL]</name>

project/stm32f407/driver/src/stm32f407_driver_at24cxx_interface.c

@@ -147,12 +147,12 @@ void at24cxx_interface_delay_ms(uint32_t ms)
 void at24cxx_interface_debug_print(const char *const fmt, ...)
 {
     char str[256];
-    uint8_t len;
+    uint16_t len;
     va_list args;
 
     memset((char *)str, 0, sizeof(char) * 256); 
     va_start(args, fmt);
-    vsnprintf((char *)str, 256, (char const *)fmt, args);
+    vsnprintf((char *)str, 255, (char const *)fmt, args);
     va_end(args);
 
     len = strlen((char *)str);

project/stm32f407/interface/src/uart.c

@@ -203,13 +203,13 @@ uint16_t uart_flush(void)
 uint16_t uart_print(const char *const fmt, ...)
 {
     char str[256];
-    uint8_t len;
+    uint16_t len;
     va_list args;
 
     /* print to the buffer */
     memset((char *)str, 0, sizeof(char) * 256); 
     va_start(args, fmt);
-    vsnprintf((char *)str, 256, (char const *)fmt, args);
+    vsnprintf((char *)str, 255, (char const *)fmt, args);
     va_end(args);
 
     /* send the data */