This is fine, though I don't think the BUGS section is appropriate.
Unless someone would give any real reasons why EFAULT should
be kept listed in manpage, I'm going to remove it altogether
from the manpage within one week.

Jaromir

Darren Reed wrote:
> 
> Comments on the the following change for pipe(2) ?
> 
> Darren
> 
> Index: pipe.2
> ===================================================================
> RCS file: /cvsroot/basesrc/lib/libc/sys/pipe.2,v
> retrieving revision 1.11
> diff -c -r1.11 pipe.2
> *** pipe.2	1999/12/02 21:42:38	1.11
> --- pipe.2	2001/10/03 14:08:37
> ***************
> *** 97,107 ****
>   Too many descriptors are active.
>   .It Bq Er ENFILE
>   The system file table is full.
> - .It Bq Er EFAULT
> - The
> - .Fa fildes
> - buffer is in an invalid area of the process's address
> - space.
>   .El
>   .Sh SEE ALSO
>   .Xr sh 1 ,
> --- 97,102 ----
> ***************
> *** 114,119 ****
> --- 109,127 ----
>   .Fn pipe
>   function conforms to
>   .St -p1003.1-90 .
> + .Sh BUGS
> + At present, EFAULT is not returned by
> + .Fn pipe ,
> + rather, the current implementation does not handle a bad address
> + gracefully.  A core dump as a result of either
> + .Dv SIGSEGV
> + or
> + .Dv SIGBUS
> + can be expected, instead, if a bad address is supplied as the
> + .Fa fildes
> + buffer.  A future implementation of
> + .Fn pipe ,
> + may return EFAULT instead of generating a core dump.
>   .Sh HISTORY
>   A
>   .Fn pipe
> 


-- 
Jaromir Dolecek <jdolecek@NetBSD.org>      http://www.ics.muni.cz/~dolecek/
NetBSD - just plain best OS! -=*=- Got spare MCA cards or docs? Hand me them!