only allow alphanumeric input keys.

davidflowers · davidflowers · commit c74efd15873b · 2024-03-22T12:39:52.000-04:00
diff --git a/secure_boot_and_secure_firmware_upgrade_over_canfd/icsp_inhibit.X/main.c b/secure_boot_and_secure_firmware_upgrade_over_canfd/icsp_inhibit.X/main.c
@@ -33,6 +33,7 @@
 #include <stdio.h>
 #include <string.h>
 #include <stdbool.h>
+#include <ctype.h>
 
 #define USER_INPUT_BUFFER_SIZE 50
 #define UNLOCK_COMMAND "LOCKDEVICE"
@@ -84,7 +85,7 @@ int main(void)
             
     PrintWarning();
 
-    while (1)
+    while(1)
     {
         char* userInput = ScanInput();
 
@@ -124,7 +125,7 @@ static char* ScanInput(void)
             ResetPrompt();
         }
         
-        if((key != ENTER) && (userInputOffset < sizeof(userInput)))
+        if(isalnum(key) && (userInputOffset < sizeof(userInput)))
         {
             userInput[userInputOffset++] = key;
             printf("%c", key);

Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,7 @@`
`33`	`33`	`#include <stdio.h>`
`34`	`34`	`#include <string.h>`
`35`	`35`	`#include <stdbool.h>`
	`36`	`+#include <ctype.h>`
`36`	`37`
`37`	`38`	`#define USER_INPUT_BUFFER_SIZE 50`
`38`	`39`	`#define UNLOCK_COMMAND "LOCKDEVICE"`
`@@ -84,7 +85,7 @@ int main(void)`
`84`	`85`
`85`	`86`	`PrintWarning();`
`86`	`87`
`87`		`- while (1)`
	`88`	`+ while(1)`
`88`	`89`	`{`
`89`	`90`	`char* userInput = ScanInput();`
`90`	`91`
`@@ -124,7 +125,7 @@ static char* ScanInput(void)`
`124`	`125`	`ResetPrompt();`
`125`	`126`	`}`
`126`	`127`
`127`		`- if((key != ENTER) && (userInputOffset < sizeof(userInput)))`
	`128`	`+ if(isalnum(key) && (userInputOffset < sizeof(userInput)))`
`128`	`129`	`{`
`129`	`130`	`userInput[userInputOffset++] = key;`
`130`	`131`	`printf("%c", key);`