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