patch to mkpasswd.c - allows selection of specific user
Christopher Faylor
cgf@redhat.com
Mon Nov 12 21:22:00 GMT 2001
On Tue, Nov 13, 2001 at 04:02:32PM +1100, Mathew Boorman wrote:
>Darn, now I'm told about Marks patch!
>Anyway, onward...
>
>Mark Bradshaw:
>
>>@@ -135,6 +145,7 @@ enum_users (LPWSTR servername, int print
>> default:
>> fprintf (stderr, "NetUserEnum() failed with %ld\n", rc);
>>+ if ( rc == 2221 ) printf("That user doesn't appear to exist.\n");
>
>The appropriate error codes are in <lmerr.h> around, I noted some were
>slightly different in name though.
>I believe this message should go to stderr anyway, otherwise you would end
>up with a polluted /etc/passwd file.
Oops. This points out a couple of problems that I didn't notice before.
1) Never use a raw number like the above, as Mathew has said.
2) This is not the correct format for an if statement. You aren't adhering
to the GNU formatting conventions. Please use the formatting of the
code that you are patching. This is good advice for whatever project
you are on.
Thanks,
cgf
More information about the Cygwin-patches
mailing list