monitor_unotify: Add more logs to status pipe failures

happyCoder92 · copybara-github · commit 380e04e6803a · 2025-04-25T00:12:26.000-07:00
PiperOrigin-RevId: 751297429
Change-Id: I21c2a3ed309d74210282fd277c996596d8ceac19
diff --git a/sandboxed_api/sandbox2/monitor_unotify.cc b/sandboxed_api/sandbox2/monitor_unotify.cc
@@ -298,6 +298,7 @@ void UnotifyMonitor::Run() {
       break;
     }
     if (pfds[0].revents & POLLHUP) {
+      LOG(ERROR) << "Status pipe hangup";
       SetExitStatusCode(Result::INTERNAL_ERROR, Result::FAILED_MONITOR);
       break;
     }
@@ -341,6 +342,7 @@ void UnotifyMonitor::SetExitStatusFromStatusPipe() {
       SetExitStatusCode(Result::SIGNALED, status);
     }
   } else {
+    LOG(ERROR) << "Unexpected exit code: " << code;
     SetExitStatusCode(Result::INTERNAL_ERROR, Result::FAILED_MONITOR);
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -298,6 +298,7 @@ void UnotifyMonitor::Run() {`
`298`	`298`	`break;`
`299`	`299`	`}`
`300`	`300`	`if (pfds[0].revents & POLLHUP) {`
	`301`	`+ LOG(ERROR) << "Status pipe hangup";`
`301`	`302`	`SetExitStatusCode(Result::INTERNAL_ERROR, Result::FAILED_MONITOR);`
`302`	`303`	`break;`
`303`	`304`	`}`
`@@ -341,6 +342,7 @@ void UnotifyMonitor::SetExitStatusFromStatusPipe() {`
`341`	`342`	`SetExitStatusCode(Result::SIGNALED, status);`
`342`	`343`	`}`
`343`	`344`	`} else {`
	`345`	`+ LOG(ERROR) << "Unexpected exit code: " << code;`
`344`	`346`	`SetExitStatusCode(Result::INTERNAL_ERROR, Result::FAILED_MONITOR);`
`345`	`347`	`}`
`346`	`348`	`}`