From 8768a600ff85736384d971efa13a39925e199c8d Mon Sep 17 00:00:00 2001 From: Rob Bradford Date: Tue, 14 Dec 2021 10:39:36 +0000 Subject: [PATCH] vmm: Only warn on error when setting up SIGWINCH handler Setting up the SIGWINCH handler requires at least Linux 5.7. However this functionality is not required for basic PTY operation. Fixes: #3456 Signed-off-by: Rob Bradford (cherry picked from commit afe386bc1392b6c66126beda7a7c235f94e35833) --- vmm/src/device_manager.rs | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/vmm/src/device_manager.rs b/vmm/src/device_manager.rs index 909087cab..0a38561ed 100644 --- a/vmm/src/device_manager.rs +++ b/vmm/src/device_manager.rs @@ -1712,9 +1712,14 @@ impl DeviceManager { let seccomp_filter = get_seccomp_filter(&self.seccomp_action, Thread::PtyForeground).unwrap(); - let pipe = start_sigwinch_listener(seccomp_filter, pty)?; - - self.console_resize_pipe = Some(Arc::new(pipe)); + match start_sigwinch_listener(seccomp_filter, pty) { + Ok(pipe) => { + self.console_resize_pipe = Some(Arc::new(pipe)); + } + Err(e) => { + warn!("Ignoring error from setting up SIGWINCH listener: {}", e) + } + } Ok(()) }