fdo-mirrors/libinput
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/libinput/libinput.git synced 2026-01-03 18:30:12 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

tools: record: fix dmi recording

Browse source 
Processing os-release in the same buffer that the dmi modalias used caused the
dmi to be recorded as 'dmi: "VERSION_ID=31"'. The cause for that was simply
that the dmi modalias was read but not printed until after the os-release
information was processed.

Fix this two-fold: rearrange that each part now reads and prints in
one go, and rename the buffers so we don't re-use them.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
(cherry picked from commit 850925910f)
This commit is contained in:
Peter Hutterer 2020-03-09 10:16:04 +10:00
parent a1c8909337
commit 2189fd2c2e
1 changed files with 25 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

46
tools/libinput-record.c
View file

@ -1430,37 +1430,26 @@ print_system_header(struct record_context *ctx)
struct utsname u;
const char *kernel = "unknown";
FILE *dmi, *osrelease;
char buf[2048] = "unknown";
if (uname(&u) != -1)
kernel = u.release;
dmi = fopen("/sys/class/dmi/id/modalias", "r");
if (dmi) {
if (fgets(buf, sizeof(buf), dmi)) {
buf[strlen(buf) - 1] = '\0'; /* linebreak */
} else {
sprintf(buf, "unknown");
}
fclose(dmi);
}
char dmistr[2048] = "unknown";
iprintf(ctx, "system:\n");
indent_push(ctx);
/* /etc/os-release version and distribution name */
osrelease = fopen("/etc/os-release", "r");
if (!osrelease)
osrelease = fopen("/usr/lib/os-release", "r");
if (osrelease) {
char *distro = NULL, *version = NULL;
char osrstr[256] = "unknown";
while (fgets(buf, sizeof(buf), osrelease)) {
buf[strlen(buf) - 1] = '\0'; /* linebreak */
while (fgets(osrstr, sizeof(osrstr), osrelease)) {
osrstr[strlen(osrstr) - 1] = '\0'; /* linebreak */
if (!distro && strneq(buf, "ID=", 3))
distro = safe_strdup(&buf[3]);
else if (!version && strneq(buf, "VERSION_ID=", 11))
version = safe_strdup(&buf[11]);
if (!distro && strneq(osrstr, "ID=", 3))
distro = safe_strdup(&osrstr[3]);
else if (!version && strneq(osrstr, "VERSION_ID=", 11))
version = safe_strdup(&osrstr[11]);
if (distro && version) {
iprintf(ctx, "os: \"%s:%s\"\n", distro, version);
@ -1471,8 +1460,23 @@ print_system_header(struct record_context *ctx)
free(version);
fclose(osrelease);
}
/* kernel version */
if (uname(&u) != -1)
kernel = u.release;
iprintf(ctx, "kernel: \"%s\"\n", kernel);
iprintf(ctx, "dmi: \"%s\"\n", buf);
/* dmi modalias */
dmi = fopen("/sys/class/dmi/id/modalias", "r");
if (dmi) {
if (fgets(dmistr, sizeof(dmistr), dmi)) {
dmistr[strlen(dmistr) - 1] = '\0'; /* linebreak */
} else {
sprintf(dmistr, "unknown");
}
fclose(dmi);
}
iprintf(ctx, "dmi: \"%s\"\n", dmistr);
indent_pop(ctx);
}