diff options
author | Glenn L McGrath <bug1@ihug.co.nz> | 2004-09-14 16:08:02 +0000 |
---|---|---|
committer | Glenn L McGrath <bug1@ihug.co.nz> | 2004-09-14 16:08:02 +0000 |
commit | d2bd7d8dd5e277e7852c76003aeea592399e164b (patch) | |
tree | 1d3904a691ced8309f39e4638c1efb826778c42d | |
parent | e11db0ed3502506adf43ce0985288af976dd3ba2 (diff) | |
download | busybox-w32-d2bd7d8dd5e277e7852c76003aeea592399e164b.tar.gz busybox-w32-d2bd7d8dd5e277e7852c76003aeea592399e164b.tar.bz2 busybox-w32-d2bd7d8dd5e277e7852c76003aeea592399e164b.zip |
Patch from tito to add argument checking.
-rw-r--r-- | loginutils/addgroup.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/loginutils/addgroup.c b/loginutils/addgroup.c index cfbd71624..804d6961c 100644 --- a/loginutils/addgroup.c +++ b/loginutils/addgroup.c | |||
@@ -148,7 +148,7 @@ int addgroup_main(int argc, char **argv) | |||
148 | 148 | ||
149 | /* get remaining args */ | 149 | /* get remaining args */ |
150 | if(bb_getopt_ulflags(argc, argv, "g:", &group)) { | 150 | if(bb_getopt_ulflags(argc, argv, "g:", &group)) { |
151 | gid = strtol(group, NULL, 10); | 151 | gid = bb_xgetlarg(group, 10, 0, LONG_MAX); |
152 | } | 152 | } |
153 | 153 | ||
154 | if (optind < argc) { | 154 | if (optind < argc) { |
@@ -160,10 +160,10 @@ int addgroup_main(int argc, char **argv) | |||
160 | 160 | ||
161 | if (optind < argc) { | 161 | if (optind < argc) { |
162 | user = argv[optind]; | 162 | user = argv[optind]; |
163 | optind++; | ||
164 | } else { | 163 | } else { |
165 | user = ""; | 164 | user = ""; |
166 | } | 165 | } |
166 | |||
167 | if_i_am_not_root(); | 167 | if_i_am_not_root(); |
168 | 168 | ||
169 | /* werk */ | 169 | /* werk */ |