]> git.donarmstrong.com Git - qmk_firmware.git/commitdiff
[Keyboard] Cannon Keys Satisfaction75: Fix buffer sizes for sprintfs. (#6954)
authorNoshBar <noshbar+git@gmail.com>
Mon, 7 Oct 2019 21:35:28 +0000 (23:35 +0200)
committernoroadsleft <18669334+noroadsleft@users.noreply.github.com>
Mon, 7 Oct 2019 21:35:28 +0000 (14:35 -0700)
sprintf always adds a NULL terminator at the end of the buffer it works on.
A few places made just enough space for the resulting string, but not the terminator.
So this just adds one to the buffer size to make room for it.

keyboards/cannonkeys/satisfaction75/satisfaction_oled.c

index 4e22587db85163e90c80987051b4930b8c6119ea..6553bd13004e1c2a3e021e03f73c839bb2db57d8 100644 (file)
@@ -91,8 +91,8 @@ void draw_default(){
   if (hour == 0){
     hour = 12;
   }
-  char hour_str[2] = "";
-  char min_str[2] = "";
+  char hour_str[3] = "";
+  char min_str[3] = "";
 
   sprintf(hour_str, "%02d", hour);
   sprintf(min_str, "%02d", minute);
@@ -199,11 +199,11 @@ void draw_clock(){
   if (hour == 0){
     hour = 12;
   }
-  char hour_str[2] = "";
-  char min_str[2] = "";
-  char year_str[4] = "";
-  char month_str[2] = "";
-  char day_str[2] = "";
+  char hour_str[3] = "";
+  char min_str[3] = "";
+  char year_str[5] = "";
+  char month_str[3] = "";
+  char day_str[3] = "";
 
   sprintf(hour_str, "%02d", hour);
   sprintf(min_str, "%02d", minute);