Skip to content

Commit 409902d

Browse files
hannesweisbachtekka007
authored andcommitted
Fix potential buffer overflow (#1365)
Use snprintf instead of sprintf to avoid a potential buffer overflow.
1 parent b7bed69 commit 409902d

File tree

1 file changed

+2
-2
lines changed
  • hal/architecture/Linux/drivers/core

1 file changed

+2
-2
lines changed

hal/architecture/Linux/drivers/core/GPIO.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ GPIOClass::GPIOClass()
5050
}
5151

5252
if (strncmp("gpiochip", de->d_name, 8) == 0) {
53-
sprintf(file, "/sys/class/gpio/%s/base", de->d_name);
53+
snprintf(file, sizeof(file), "/sys/class/gpio/%s/base", de->d_name);
5454
f = fopen(file, "r");
5555
int base;
5656
if (fscanf(f, "%d", &base) == EOF) {
@@ -59,7 +59,7 @@ GPIOClass::GPIOClass()
5959
}
6060
fclose(f);
6161

62-
sprintf(file, "/sys/class/gpio/%s/ngpio", de->d_name);
62+
snprintf(file, sizeof(file), "/sys/class/gpio/%s/ngpio", de->d_name);
6363
f = fopen(file, "r");
6464
int ngpio;
6565
if (fscanf(f, "%d", &ngpio) == EOF) {

0 commit comments

Comments
 (0)