tree c4682f19cefae51bf089633f099364cc91c606f0
parent 143b6d3e3c0419fa069fc22d6e904a4b15dc80d2
author Devon H. O'Dell <dhobsd@google.com> 1541529595 -0800
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1542669646 +0000

[fdio] Do not close VMOFile again on error

In f097da4cae2bead51359d4eddefb0b8b4fb850bd, the node handle was closed
when a failure was encountered in creating the VMOFile. Subsequently,
ef4119d3fa5da5f70797f2e1f0d2683bad99f154 noticed that Pipes and VMOFiles
should always be closed, and unconditionally closed the handle. This
change removes the second close, which would likely fail (the handle
already being closed), masking the error from creating the VMOFile.

This change also fixes some style around the fixed code. The call to
NewVMOFile should be on a single line, and the comma after the last
argument is unnecessary and confusing.

Change-Id: I014c6ca6ab7eb51328182d0cbb7457e2711c1745