98 lines
2.9 KiB
Diff
98 lines
2.9 KiB
Diff
From 850925910f7f6db16f0aa645892d9121760de7b2 Mon Sep 17 00:00:00 2001
|
|
From: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Date: Mon, 9 Mar 2020 10:16:04 +1000
|
|
Subject: [PATCH libinput] tools: record: fix dmi recording
|
|
|
|
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>
|
|
---
|
|
tools/libinput-record.c | 46 ++++++++++++++++++++++-------------------
|
|
1 file changed, 25 insertions(+), 21 deletions(-)
|
|
|
|
diff --git a/tools/libinput-record.c b/tools/libinput-record.c
|
|
index 0b10d8bf..6d45efc5 100644
|
|
--- a/tools/libinput-record.c
|
|
+++ b/tools/libinput-record.c
|
|
@@ -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);
|
|
}
|
|
|
|
--
|
|
2.24.1
|
|
|