Hi Alin,

How do you typically run into this issue? Do you also need to call into
CloseHandle(p->fd)?

Thanks,
Sairam

On 8/12/16, 12:46 AM, "Alin Serdean" <aserd...@cloudbasesolutions.com>
wrote:

>Disconnect named pipes that failed connection.
>
>Found by testing.
>
>Signed-off-by: Alin Gabriel Serdean <aserd...@cloudbasesolutions.com>
>---
> lib/stream-windows.c | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/lib/stream-windows.c b/lib/stream-windows.c
>index e0fe012..637920b 100644
>--- a/lib/stream-windows.c
>+++ b/lib/stream-windows.c
>@@ -431,6 +431,7 @@ pwindows_accept(struct pstream *pstream, struct
>stream **new_streamp)
>             } else {
>                 VLOG_ERR_RL(&rl, "Could not connect named pipe. Last "
>                             "error: %s", ovs_lasterror_to_string());
>+                DisconnectNamedPipe(p->fd);
>                 return EINVAL;
>             }
>         }
>@@ -446,6 +447,7 @@ pwindows_accept(struct pstream *pstream, struct
>stream **new_streamp)
>         } else if (last_error != ERROR_PIPE_CONNECTED) {
>             VLOG_ERR_RL(&rl, "Could not connect synchronous named pipe.
>Last "
>                         "error: %s", ovs_lasterror_to_string());
>+            DisconnectNamedPipe(p->fd);
>             return EINVAL;
>         } else {
>             /* If the pipe is connected, signal an event. */
>-- 
>2.9.2.windows.1
>_______________________________________________
>dev mailing list
>dev@openvswitch.org
>https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailma
>n_listinfo_dev&d=CwIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=Dc
>ruz40PROJ40ROzSpxyQSLw6fcrOWpJgEcEmNR3JEQ&m=Ohnj6Y2LGSidb7c9QztcCWN_PnQ2BN
>1cKD9mvIj5ePs&s=_jgmzDVj6xperAHhNFY0GZ2Sf-4flhrqO3orPSsNQrY&e= 

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to