[PATCH] fix addgroup command line parsing

[Tito]

Hi,

addgroup actually does:

addgroup   group
addgroup   -g num group
addgroup   user group

so:

addgroup -g num user group

should not be permitted.
This patch fixes the problem for me.

diff -uN loginutils/addgroup.c.orig loginutils/addgroup.c
--- loginutils/addgroup.c.orig  2007-05-30 14:41:30.000000000 +0200
+++ loginutils/addgroup.c       2007-07-29 16:22:50.000000000 +0200
@@ -148,6 +148,10 @@
        if (argc == 2) {
                struct group *gr;

+               /* There was -g on the commandline: error out */
+               if (gid)
+                       bb_show_usage();
+
                /* check if group and user exist */
                xuname2uid(argv[0]); /* unknown user: exit */
                xgroup2gid(argv[1]); /* unknown group: exit */


Only little tested.
Please apply if you like it.

Ciao,
Tito