From 070407809efa2c53a395f27c9ab84a2e2bd9a750 Mon Sep 17 00:00:00 2001 From: Steven Moreland Date: Mon, 4 Apr 2022 19:34:42 +0000 Subject: [PATCH] Print error when netlink socket not created. The panic here didn't show what error might be encountered (EPERM, for instance). Bug: 228077254 Test: check log on error Change-Id: I56f34f87430a68266af85ba1d0abb2aeb2c05407 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3569267 Reviewed-by: Daniel Verkamp Tested-by: kokoro Reviewed-by: Andrew Walbran Commit-Queue: Steven Moreland --- devices/src/acpi.rs | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/devices/src/acpi.rs b/devices/src/acpi.rs index b36931247a..ecf12d48b2 100644 --- a/devices/src/acpi.rs +++ b/devices/src/acpi.rs @@ -320,7 +320,13 @@ fn acpi_event_pwrbtn_class( fn get_acpi_event_group() -> Option { // Create netlink generic socket which will be used to query about given family name - let netlink_ctrl_sock = NetlinkGenericSocket::new(0).unwrap(); + let netlink_ctrl_sock = match NetlinkGenericSocket::new(0) { + Ok(sock) => sock, + Err(e) => { + error!("netlink generic socket creation error: {}", e); + return None; + } + }; let nlmsg_family_response = netlink_ctrl_sock .family_name_query("acpi_event".to_string())