commit 08bdb15d748d7d9fb6a01894fe9034699a01b764
author Lorenzo Colitti <lorenzo@google.com> Tue Mar 07 00:22:33 2017 +0900
committer Kazuhiro Inaba <kinaba@google.com> Fri Jul 07 06:30:00 2017 +0000
tree a80a232f481238f0aa570498c767f32671d826a1
parent 6ee31c8e8fa4cd73be8c1d992431c1f77d6c0eab

Dup fds to stop finalizers from invalidating them.

The hostside VPN tests were failing because finalizers were
closing the ParcelFileDescriptors that we use to get socekt fds.
The close operations were causing the fds to be marked as invalid
(i.e., -1), causing the tests' system calls on them to fail with
EBADF.

Fix this by dup(2)ing the fds and closing the original objects.

Also, add some asserts to debug this sort of failure.

Fix: 35927643
Test: HostsideVpnTests passed 20/20 times on bullhead
Change-Id: If88530b5bd32622bd4726cd6f0907f731209bb43
(cherry picked from commit a7ec8300d98e92c1363303f7f103bf0787d13a27)