From 192c35f6eb2bb85e28c697a45c7d80a43ec6e743 Mon Sep 17 00:00:00 2001 From: Eric Andersen Date: Thu, 2 Sep 2004 22:22:17 +0000 Subject: [PATCH] Tito writes: The second patch contains: 1) a size optimization for adduser.c 2) removes a warning about an unused variable in syslogd.c if CONFIG_FEATURE_REMOTE_LOG is not set 3)cosmetic fixes for addgroup_full_usage and adduser_full_usage Ciao, Tito --- include/usage.h | 4 ++-- loginutils/adduser.c | 9 ++------- sysklogd/syslogd.c | 2 ++ 3 files changed, 6 insertions(+), 9 deletions(-) diff --git a/include/usage.h b/include/usage.h index a0b812a..f87f367 100644 --- a/include/usage.h +++ b/include/usage.h @@ -4,14 +4,14 @@ #define addgroup_trivial_usage \ "[-g GID] group_name [user_name]" #define addgroup_full_usage \ - "Adds a group to the system" \ + "Adds a group to the system\n\n" \ "Options:\n" \ "\t-g GID\t\tspecify gid" #define adduser_trivial_usage \ "[OPTIONS] user_name" #define adduser_full_usage \ - "Adds a user to the system" \ + "Adds a user to the system\n\n" \ "Options:\n" \ "\t-h DIR\t\tAssign home directory DIR\n" \ "\t-g GECOS\tAssign gecos field GECOS\n" \ diff --git a/loginutils/adduser.c b/loginutils/adduser.c index 7687888..7fa05a0 100644 --- a/loginutils/adduser.c +++ b/loginutils/adduser.c @@ -305,13 +305,8 @@ int adduser_main(int argc, char **argv) if (usegroup) { /* Add user to a group that already exists */ - struct group *g; - - g = getgrnam(usegroup); - if (g == NULL) - bb_error_msg_and_die("group %s does not exist", usegroup); - - pw.pw_gid = g->gr_gid; + pw.pw_gid = my_getgrnam(usegroup); + /* exits on error */ } /* grand finale */ diff --git a/sysklogd/syslogd.c b/sysklogd/syslogd.c index 741c806..6e7652c 100644 --- a/sysklogd/syslogd.c +++ b/sysklogd/syslogd.c @@ -386,7 +386,9 @@ static void logMessage(int pri, char *msg) time_t now; char *timestamp; static char res[20] = ""; +#ifdef CONFIG_FEATURE_REMOTE_LOG static char line[512]; +#endif CODE *c_pri, *c_fac; if (pri != 0) { -- 2.7.4