Re: msvcrt: implement _pipe

[Alexandre Julliard]

Juan Lang <juan_lang(a)yahoo.com> writes:
> +    fd = msvcrt_alloc_fd(readHandle, wxflags);
> +    if (fd != -1)
> +    {
> +      pfds[0] = fd;
> +      fd = msvcrt_alloc_fd(writeHandle, wxflags);
> +      if (fd != -1)
> +        pfds[1] = fd;
> +    }
> +    if (fd == -1)
> +    {
> +      CloseHandle(readHandle);
> +      CloseHandle(writeHandle);
> +      *MSVCRT__errno() = MSVCRT_EMFILE;
> +      ret = -1;
> +    }

You need to free pfds[0] on failure if it was allocated.

-- 
Alexandre Julliard
julliard(a)winehq.org