diff options
| author | Jean Delvare <jdelvare@suse.de> | 2017-11-02 16:17:50 +0100 |
|---|---|---|
| committer | Jean Delvare <jdelvare@suse.de> | 2017-11-02 16:19:33 +0100 |
| commit | def2845efacab3a3973fb0218ac5077a162f8f1e (patch) | |
| tree | 92e582afad63413db1cdfa182d781a1648e1f70c /tools | |
| parent | 53a53775ae837c6d43e0abe3f39302b04a6bccfa (diff) | |
| download | i2c-tools-git-def2845efacab3a3973fb0218ac5077a162f8f1e.tar.gz | |
i2c-tools: i2cbusses: Avoid buffer overflows in sysfs paths
sprintf isn't safe, use snprintf instead.
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/i2cbusses.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tools/i2cbusses.c b/tools/i2cbusses.c index dad22ea..cb78cc7 100644 --- a/tools/i2cbusses.c +++ b/tools/i2cbusses.c @@ -220,18 +220,18 @@ struct i2c_adap *gather_i2c_busses(void) /* this should work for kernels 2.6.5 or higher and */ /* is preferred because is unambiguous */ - sprintf(n, "%s/%s/name", sysfs, de->d_name); + snprintf(n, NAME_MAX, "%s/%s/name", sysfs, de->d_name); f = fopen(n, "r"); /* this seems to work for ISA */ if(f == NULL) { - sprintf(n, "%s/%s/device/name", sysfs, de->d_name); + snprintf(n, NAME_MAX, "%s/%s/device/name", sysfs, de->d_name); f = fopen(n, "r"); } /* non-ISA is much harder */ /* and this won't find the correct bus name if a driver has more than one bus */ if(f == NULL) { - sprintf(n, "%s/%s/device", sysfs, de->d_name); + snprintf(n, NAME_MAX, "%s/%s/device", sysfs, de->d_name); if(!(ddir = opendir(n))) continue; while ((dde = readdir(ddir)) != NULL) { @@ -240,8 +240,8 @@ struct i2c_adap *gather_i2c_busses(void) if (!strcmp(dde->d_name, "..")) continue; if ((!strncmp(dde->d_name, "i2c-", 4))) { - sprintf(n, "%s/%s/device/%s/name", - sysfs, de->d_name, dde->d_name); + snprintf(n, NAME_MAX, "%s/%s/device/%s/name", + sysfs, de->d_name, dde->d_name); if((f = fopen(n, "r"))) goto found; } |