New issue
Advanced search Search tips

Issue 819358 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 816620
Owner:
Closed: May 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Fuchsia
Pri: 3
Type: Bug



Sign in to add a comment

IPCFuzzingTest.MsgBadPayloadShort flaked under Fuchsia/x64/Debug FYI bot

Project Member Reported by w...@chromium.org, Mar 6 2018

Issue description

The test hung in https://ci.chromium.org/buildbot/chromium.fyi/Fuchsia%20(dbg)/16894 while waiting for one of the two expected messages.

The test base pages use of base::RunLoop::QuitCurrentWhenIdleDeprecated(), so in principle if two IPC messages were delivered to the main message-loop while it was not scheduled then it could conceivably enter RunLoop::Run(), process both "quit" operations and leave a second Run() "hung", never exiting.

However, looking at the test itself, there is a simple lock-step of Send->ExpectMessage->Send->ExpectMessage, which means we shouldn't be able to get into a double-quitted state.

This flake was on Fuchsia, where we believe that the ChromeIPC+Mojo+Zircon stack can lose IPC messages on process-exit, so this may be a different example of the same underlying issue.
 

Comment 1 by w...@chromium.org, Mar 6 2018

Sure enough, the FuzzServerListener used by the FuzzServerClient child process does exit immediately after processing the second incoming IPC message, and as part of processing each incoming IPC message it calls RoundtripAckReply() to send a response - so likely a dupe of  issue 816620 .

Comment 2 by w...@chromium.org, May 30 2018

Mergedinto: 816620
Status: Duplicate (was: Assigned)

Sign in to add a comment