-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

According to Bruno Haible on 12/10/2009 6:19 PM:
> Is there any reason why the pipe2_safer code is not in a file of its own?

Only because unistd-safer didn't split fd_safer_flag and dup_safer_flag
into separate files.

> Here is a proposed patch:

It looks okay, except for:

> +          fd[i] = fd_safer_flag (fd[i], flags);

unistd-safer.h doesn't declare fd_safer_flag unless the cloexec module was
also in use.  Therefore,

>  Depends-on:
> -cloexec

...I think this portion of the change is incorrect.

- --
Don't work too hard, make some time for fun as well!

Eric Blake             e...@byu.net
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAkshoTkACgkQ84KuGfSFAYD/BgCgtiMM0PtgSmV2P1VrTFcfE79k
VNoAn0N5BX7Ky0dvKpT36wW7euAOp1La
=18SG
-----END PGP SIGNATURE-----


Reply via email to