On 01/16/2015 01:38 AM, Michael Kerrisk (man-pages) wrote: > This patch was broken. (I think you must have prepared a preceding > patch that you did not send. Sorry for that, I had a busy tree and was trying to push out all the temporary patches I had pending. > I take your point, but I think a little more could be said. I instead > applied the patch below. Okay? Yes, modulo what I expect is a typo. Comments inline. > Thanks, > > Michael > > diff --git a/man3/getpw.3 b/man3/getpw.3 > index cf7eb69..9f2732f 100644 > --- a/man3/getpw.3 > +++ b/man3/getpw.3 > @@ -75,16 +75,26 @@ The > function returns 0 on success; on error, it returns \-1, and > .I errno > is set to indicate the error. > + > +If > +.I uid > +is not found in the password database, > +.BR getpw () > +returns 0, sets Should be "returns -1" not "returns 0". > +.I errno > +to 0, and leaves > +.I buf > +unchanged. > .SH ERRORS > .TP > +.BR 0 " or " ENOENT > +No user corresponding to > +.IR uid . > +.TP > .B EINVAL > .I buf > is NULL. > .TP > -.B ENOENT > -No user corresponding to > -.IR uid . > -.TP > .B ENOMEM > Insufficient memory to allocate > .I passwd > OK with that change. Cheers, Carlos. -- To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
