Potential fix for pull request finding

CloudyPadmal · Copilot · web-flow · commit 82c5e11de928 · 2026-07-06T20:31:15.000Z
Co-authored-by: Copilot Autofix powered by AI &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/src/sdcard/sdcard.c b/src/sdcard/sdcard.c
@@ -154,7 +154,7 @@ static bool read_config_file(void) {
         return false;
     }
 
-    TCHAR buf[6] = {0}; // Config file should have a magic header of 5 bytes
+    TCHAR buf[6] = {0}; // 5-byte magic + null terminator
     UINT bytes_read = 0;
     if (f_read(&file, buf, sizeof buf - 1, &bytes_read) != FR_OK
         || bytes_read != sizeof buf - 1) {
@@ -163,7 +163,8 @@ static bool read_config_file(void) {
     }
     f_close(&file);
 
-    return true;
+    // Magic header must be exactly "PSLAB".
+    return (buf[0] == 'P' && buf[1] == 'S' && buf[2] == 'L' && buf[3] == 'A' && buf[4] == 'B');
 }
 
 response_t SDCARD_standalone_check(void) {

Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,7 @@ static bool read_config_file(void) {`
`154`	`154`	`return false;`
`155`	`155`	`}`
`156`	`156`
`157`		`- TCHAR buf[6] = {0}; // Config file should have a magic header of 5 bytes`
	`157`	`+ TCHAR buf[6] = {0}; // 5-byte magic + null terminator`
`158`	`158`	`UINT bytes_read = 0;`
`159`	`159`	`if (f_read(&file, buf, sizeof buf - 1, &bytes_read) != FR_OK`
`160`	`160`	`\|\| bytes_read != sizeof buf - 1) {`
`@@ -163,7 +163,8 @@ static bool read_config_file(void) {`
`163`	`163`	`}`
`164`	`164`	`f_close(&file);`
`165`	`165`
`166`		`- return true;`
	`166`	`+ // Magic header must be exactly "PSLAB".`
	`167`	`+ return (buf[0] == 'P' && buf[1] == 'S' && buf[2] == 'L' && buf[3] == 'A' && buf[4] == 'B');`
`167`	`168`	`}`
`168`	`169`
`169`	`170`	`response_t SDCARD_standalone_check(void) {`