[BusyBox-cvs] busybox/applets applets.c,1.19,1.20
Glenn McGrath
bug1 at busybox.net
Mon May 26 14:09:17 UTC 2003
Update of /var/cvs/busybox/applets
In directory winder:/tmp/cvs-serv10627/applets
Modified Files:
applets.c
Log Message:
Vodz, last_path_87, formatiing changes
Index: applets.c
===================================================================
RCS file: /var/cvs/busybox/applets/applets.c,v
retrieving revision 1.19
retrieving revision 1.20
diff -u -d -r1.19 -r1.20
--- applets.c 19 Mar 2003 09:11:13 -0000 1.19
+++ applets.c 26 May 2003 14:09:12 -0000 1.20
@@ -20,7 +20,7 @@
* along with this program; if not, write to the Free Software
* Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
*
- * Based in part on code from sash, Copyright (c) 1999 by David I. Bell
+ * Based in part on code from sash, Copyright (c) 1999 by David I. Bell
* Permission has been granted to redistribute this code under the GPL.
*
*/
@@ -44,7 +44,7 @@
#ifdef CONFIG_FEATURE_SUID
-static void check_suid ( struct BB_applet *app );
+static void check_suid (struct BB_applet *app);
#ifdef CONFIG_FEATURE_SUID_CONFIG
@@ -53,21 +53,22 @@
#include "pwd_.h"
#include "grp_.h"
-static int parse_config_file ( void );
+static int parse_config_file (void);
static int config_ok;
#define CONFIG_FILE "/etc/busybox.conf"
/* applets [] is const, so we have to define this "override" structure */
-struct BB_suid_config {
- struct BB_applet *m_applet;
+struct BB_suid_config
+{
+ struct BB_applet *m_applet;
- uid_t m_uid;
- gid_t m_gid;
- mode_t m_mode;
+ uid_t m_uid;
+ gid_t m_gid;
+ mode_t m_mode;
- struct BB_suid_config *m_next;
+ struct BB_suid_config *m_next;
};
static struct BB_suid_config *suid_config;
@@ -78,79 +79,84 @@
-extern void bb_show_usage(void)
+extern void
+bb_show_usage (void)
{
- const char *format_string;
- const char *usage_string = usage_messages;
- int i;
+ const char *format_string;
+ const char *usage_string = usage_messages;
+ int i;
- for (i = applet_using - applets; i > 0; ) {
- if (!*usage_string++) {
- --i;
- }
+ for (i = applet_using - applets; i > 0;) {
+ if (!*usage_string++) {
+ --i;
}
+ }
- format_string = "%s\n\nUsage: %s %s\n\n";
- if(*usage_string == '\b')
- format_string = "%s\n\nNo help available.\n\n";
- fprintf(stderr, format_string, bb_msg_full_version, applet_using->name, usage_string);
+ format_string = "%s\n\nUsage: %s %s\n\n";
+ if (*usage_string == '\b')
+ format_string = "%s\n\nNo help available.\n\n";
+ fprintf (stderr, format_string, bb_msg_full_version, applet_using->name,
+ usage_string);
- exit(EXIT_FAILURE);
+ exit (EXIT_FAILURE);
}
-static int applet_name_compare(const void *x, const void *y)
+static int
+applet_name_compare (const void *x, const void *y)
{
- const char *name = x;
- const struct BB_applet *applet = y;
+ const char *name = x;
+ const struct BB_applet *applet = y;
- return strcmp(name, applet->name);
+ return strcmp (name, applet->name);
}
extern const size_t NUM_APPLETS;
-struct BB_applet *find_applet_by_name(const char *name)
+struct BB_applet *
+find_applet_by_name (const char *name)
{
- return bsearch(name, applets, NUM_APPLETS, sizeof(struct BB_applet),
- applet_name_compare);
+ return bsearch (name, applets, NUM_APPLETS, sizeof (struct BB_applet),
+ applet_name_compare);
}
-void run_applet_by_name(const char *name, int argc, char **argv)
+void
+run_applet_by_name (const char *name, int argc, char **argv)
{
- static int recurse_level = 0;
- extern int been_there_done_that; /* From busybox.c */
+ static int recurse_level = 0;
+ extern int been_there_done_that; /* From busybox.c */
#ifdef CONFIG_FEATURE_SUID_CONFIG
- if ( recurse_level == 0 )
- config_ok = parse_config_file ( );
+ if (recurse_level == 0)
+ config_ok = parse_config_file ();
#endif
- recurse_level++;
- /* Do a binary search to find the applet entry given the name. */
- if ((applet_using = find_applet_by_name(name)) != NULL) {
- bb_applet_name = applet_using->name;
- if (argv[1] && strcmp(argv[1], "--help") == 0) {
- if (strcmp(applet_using->name, "busybox")==0) {
- if(argv[2])
- applet_using = find_applet_by_name(argv[2]);
- else
- applet_using = NULL;
- }
- if(applet_using)
- bb_show_usage();
- been_there_done_that=1;
- busybox_main(0, NULL);
- }
+ recurse_level++;
+ /* Do a binary search to find the applet entry given the name. */
+ if ((applet_using = find_applet_by_name (name)) != NULL) {
+ bb_applet_name = applet_using->name;
+ if (argv[1] && strcmp (argv[1], "--help") == 0) {
+ if (strcmp (applet_using->name, "busybox") == 0) {
+ if (argv[2])
+ applet_using = find_applet_by_name (argv[2]);
+ else
+ applet_using = NULL;
+ }
+ if (applet_using)
+ bb_show_usage ();
+ been_there_done_that = 1;
+ busybox_main (0, NULL);
+ }
#ifdef CONFIG_FEATURE_SUID
- check_suid ( applet_using );
+ check_suid (applet_using);
#endif
- exit((*(applet_using->main)) (argc, argv));
- }
- /* Just in case they have renamed busybox - Check argv[1] */
- if (recurse_level == 1) {
- run_applet_by_name("busybox", argc, argv);
- }
- recurse_level--;
+ exit ((*(applet_using->main)) (argc, argv));
+ }
+ /* Just in case they have renamed busybox - Check argv[1] */
+ if (recurse_level == 1) {
+ run_applet_by_name ("busybox", argc, argv);
+ }
+ recurse_level--;
}
@@ -159,254 +165,279 @@
#ifdef CONFIG_FEATURE_SUID_CONFIG
/* check if u is member of group g */
-static int ingroup ( uid_t u, gid_t g )
+static int
+ingroup (uid_t u, gid_t g)
{
- struct group *grp = getgrgid ( g );
+ struct group *grp = getgrgid (g);
- if ( grp ) {
- char **mem;
+ if (grp) {
+ char **mem;
- for ( mem = grp-> gr_mem; *mem; mem++ ) {
- struct passwd *pwd = getpwnam ( *mem );
+ for (mem = grp->gr_mem; *mem; mem++) {
+ struct passwd *pwd = getpwnam (*mem);
- if ( pwd && ( pwd-> pw_uid == u ))
- return 1;
- }
+ if (pwd && (pwd->pw_uid == u))
+ return 1;
}
- return 0;
+ }
+ return 0;
}
#endif
-void check_suid ( struct BB_applet *applet )
+void
+check_suid (struct BB_applet *applet)
{
- uid_t ruid = getuid ( ); /* real [ug]id */
- uid_t rgid = getgid ( );
+ uid_t ruid = getuid (); /* real [ug]id */
+ uid_t rgid = getgid ();
#ifdef CONFIG_FEATURE_SUID_CONFIG
- if ( config_ok ) {
- struct BB_suid_config *sct;
+ if (config_ok) {
+ struct BB_suid_config *sct;
- for ( sct = suid_config; sct; sct = sct-> m_next ) {
- if ( sct-> m_applet == applet )
- break;
- }
- if ( sct ) {
- mode_t m = sct-> m_mode;
+ for (sct = suid_config; sct; sct = sct->m_next) {
+ if (sct->m_applet == applet)
+ break;
+ }
+ if (sct) {
+ mode_t m = sct->m_mode;
- if ( sct-> m_uid == ruid ) /* same uid */
- m >>= 6;
- else if (( sct-> m_gid == rgid ) || ingroup ( ruid, sct-> m_gid )) /* same group / in group */
- m >>= 3;
+ if (sct->m_uid == ruid) /* same uid */
+ m >>= 6;
+ else if ((sct->m_gid == rgid) || ingroup (ruid, sct->m_gid)) /* same group / in group */
+ m >>= 3;
- if (!( m & S_IXOTH )) /* is x bit not set ? */
- bb_error_msg_and_die ( "You have no permission to run this applet!" );
+ if (!(m & S_IXOTH)) /* is x bit not set ? */
+ bb_error_msg_and_die ("You have no permission to run this applet!");
- if (( sct-> m_mode & ( S_ISGID | S_IXGRP )) == ( S_ISGID | S_IXGRP )) { /* *both* have to be set for sgid */
- if ( setegid ( sct-> m_gid ))
- bb_error_msg_and_die ( "BusyBox binary has insufficient rights to set proper GID for applet!" );
- }
- else
- setgid ( rgid ); /* no sgid -> drop */
+ if ((sct->m_mode & (S_ISGID | S_IXGRP)) == (S_ISGID | S_IXGRP)) { /* *both* have to be set for sgid */
+ if (setegid (sct->m_gid))
+ bb_error_msg_and_die
+ ("BusyBox binary has insufficient rights to set proper GID for applet!");
+ } else
+ setgid (rgid); /* no sgid -> drop */
- if ( sct-> m_mode & S_ISUID ) {
- if ( seteuid ( sct-> m_uid ))
- bb_error_msg_and_die ( "BusyBox binary has insufficient rights to set proper UID for applet!" );
- }
- else
- setuid ( ruid ); /* no suid -> drop */
- }
- else { /* default: drop all priviledges */
- setgid ( rgid );
- setuid ( ruid );
- }
- return;
+ if (sct->m_mode & S_ISUID) {
+ if (seteuid (sct->m_uid))
+ bb_error_msg_and_die
+ ("BusyBox binary has insufficient rights to set proper UID for applet!");
+ } else
+ setuid (ruid); /* no suid -> drop */
+ } else {
+ /* default: drop all priviledges */
+ setgid (rgid);
+ setuid (ruid);
}
- else {
-#ifndef CONFIG_FEATURE_SUID_CONFIG_QUIET
- static int onetime = 0;
+ return;
+ } else {
+#ifndef CONFIG_FEATURE_SUID_CONFIG_QUIET
+ static int onetime = 0;
- if ( !onetime ) {
- onetime = 1;
- fprintf ( stderr, "Using fallback suid method\n" );
- }
-#endif
+ if (!onetime) {
+ onetime = 1;
+ fprintf (stderr, "Using fallback suid method\n");
}
#endif
+ }
+#endif
- if ( applet-> need_suid == _BB_SUID_ALWAYS ) {
- if ( geteuid ( ) != 0 )
- bb_error_msg_and_die ( "This applet requires root priviledges!" );
- }
- else if ( applet-> need_suid == _BB_SUID_NEVER ) {
- setgid ( rgid ); /* drop all priviledges */
- setuid ( ruid );
- }
+ if (applet->need_suid == _BB_SUID_ALWAYS) {
+ if (geteuid () != 0)
+ bb_error_msg_and_die ("This applet requires root priviledges!");
+ } else if (applet->need_suid == _BB_SUID_NEVER) {
+ setgid (rgid); /* drop all priviledges */
+ setuid (ruid);
+ }
}
#ifdef CONFIG_FEATURE_SUID_CONFIG
-#define parse_error(x) { err=x; goto pe_label; }
+#define parse_error(x) { err=x; goto pe_label; }
-int parse_config_file ( void )
+int
+parse_config_file (void)
{
- struct stat st;
- char *err = 0;
- FILE *f = 0;
- int lc = 0;
+ struct stat st;
+ char *err = 0;
+ FILE *f = 0;
+ int lc = 0;
- suid_config = 0;
+ suid_config = 0;
- /* is there a config file ? */
- if ( stat ( CONFIG_FILE, &st ) == 0 ) {
- /* is it owned by root with no write perm. for group and others ? */
- if ( S_ISREG( st. st_mode ) && ( st. st_uid == 0 ) && (!( st. st_mode & ( S_IWGRP | S_IWOTH )))) {
- /* that's ok .. then try to open it */
- f = fopen ( CONFIG_FILE, "r" );
+ /* is there a config file ? */
+ if (stat (CONFIG_FILE, &st) == 0) {
+ /* is it owned by root with no write perm. for group and others ? */
+ if (S_ISREG (st.st_mode) && (st.st_uid == 0)
+ && (!(st.st_mode & (S_IWGRP | S_IWOTH)))) {
+ /* that's ok .. then try to open it */
+ f = fopen (CONFIG_FILE, "r");
- if ( f ) {
- char buffer [256];
- int section = 0;
+ if (f) {
+ char buffer[256];
+ int section = 0;
- while ( fgets ( buffer, sizeof( buffer ) - 1, f )) {
- char c = buffer [0];
- char *p;
+ while (fgets (buffer, sizeof (buffer) - 1, f)) {
+ char c = buffer[0];
+ char *p;
- lc++;
+ lc++;
- p = strchr ( buffer, '#' );
- if ( p )
- *p = 0;
- p = buffer + bb_strlen ( buffer );
- while (( p > buffer ) && isspace ( *--p ))
- *p = 0;
+ p = strchr (buffer, '#');
+ if (p)
+ *p = 0;
+ p = buffer + bb_strlen (buffer);
+ while ((p > buffer) && isspace (*--p))
+ *p = 0;
- if ( p == buffer )
- continue;
+ if (p == buffer)
+ continue;
- if ( c == '[' ) {
- p = strchr ( buffer, ']' );
+ if (c == '[') {
+ p = strchr (buffer, ']');
- if ( !p || ( p == ( buffer + 1 ))) /* no matching ] or empty [] */
- parse_error ( "malformed section header" );
+ if (!p || (p == (buffer + 1))) /* no matching ] or empty [] */
+ parse_error ("malformed section header");
- *p = 0;
+ *p = 0;
- if ( strcasecmp ( buffer + 1, "SUID" ) == 0 )
- section = 1;
- else
- section = -1; /* unknown section - just skip */
- }
- else if ( section ) {
- switch ( section ) {
- case 1: { /* SUID */
- int l;
- struct BB_applet *applet;
+ if (strcasecmp (buffer + 1, "SUID") == 0)
+ section = 1;
+ else
+ section = -1; /* unknown section - just skip */
+ } else if (section) {
+ switch (section) {
+ case 1:{ /* SUID */
+ int l;
+ struct BB_applet *applet;
- p = strchr ( buffer, '=' ); /* <key>[::space::]*=[::space::]*<value> */
+ p = strchr (buffer, '='); /* <key>[::space::]*=[::space::]*<value> */
- if ( !p || ( p == ( buffer + 1 ))) /* no = or key is empty */
- parse_error ( "malformed keyword" );
+ if (!p || (p == (buffer + 1))) /* no = or key is empty */
+ parse_error ("malformed keyword");
- l = p - buffer;
- while ( isspace ( buffer [--l] )) { } /* skip whitespace */
+ l = p - buffer;
+ while (isspace (buffer[--l])) {
+ /* skip whitespace */
+ }
- buffer [l+1] = 0;
+ buffer[l + 1] = 0;
- if (( applet = find_applet_by_name ( buffer ))) {
- struct BB_suid_config *sct = xmalloc ( sizeof( struct BB_suid_config ));
+ if ((applet = find_applet_by_name (buffer))) {
+ struct BB_suid_config *sct =
+ xmalloc (sizeof (struct BB_suid_config));
- sct-> m_applet = applet;
- sct-> m_next = suid_config;
- suid_config = sct;
+ sct->m_applet = applet;
+ sct->m_next = suid_config;
+ suid_config = sct;
- while ( isspace ( *++p )) { } /* skip whitespace */
+ while (isspace (*++p)) {
+ /* skip whitespace */
+ }
- sct-> m_mode = 0;
+ sct->m_mode = 0;
- switch ( *p++ ) {
- case 'S': sct-> m_mode |= S_ISUID; break;
- case 's': sct-> m_mode |= S_ISUID; /* no break */
- case 'x': sct-> m_mode |= S_IXUSR; break;
- case '-': break;
- default : parse_error ( "invalid user mode" );
- }
+ switch (*p++) {
+ case 'S':
+ sct->m_mode |= S_ISUID;
+ break;
+ case 's':
+ sct->m_mode |= S_ISUID;
+ /* no break */
+ case 'x':
+ sct->m_mode |= S_IXUSR;
+ break;
+ case '-':
+ break;
+ default:
+ parse_error ("invalid user mode");
+ }
- switch ( *p++ ) {
- case 's': sct-> m_mode |= S_ISGID; /* no break */
- case 'x': sct-> m_mode |= S_IXGRP; break;
- case 'S': break;
- case '-': break;
- default : parse_error ( "invalid group mode" );
- }
+ switch (*p++) {
+ case 's':
+ sct->m_mode |= S_ISGID;
+ /* no break */
+ case 'x':
+ sct->m_mode |= S_IXGRP;
+ break;
+ case 'S':
+ break;
+ case '-':
+ break;
+ default:
+ parse_error ("invalid group mode");
+ }
- switch ( *p ) {
- case 't':
- case 'x': sct-> m_mode |= S_IXOTH; break;
- case 'T':
- case '-': break;
- default : parse_error ( "invalid other mode" );
- }
+ switch (*p) {
+ case 't':
+ case 'x':
+ sct->m_mode |= S_IXOTH;
+ break;
+ case 'T':
+ case '-':
+ break;
+ default:
+ parse_error ("invalid other mode");
+ }
- while ( isspace ( *++p )) { } /* skip whitespace */
+ while (isspace (*++p)) {
+ /* skip whitespace */
+ }
- if ( isdigit ( *p )) {
- sct-> m_uid = strtol ( p, &p, 10 );
- if ( *p++ != '.' )
- parse_error ( "parsing <uid>.<gid>" );
- }
- else {
- struct passwd *pwd;
- char *p2 = strchr ( p, '.' );
-
- if ( !p2 )
- parse_error ( "parsing <uid>.<gid>" );
+ if (isdigit (*p)) {
+ sct->m_uid = strtol (p, &p, 10);
+ if (*p++ != '.')
+ parse_error ("parsing <uid>.<gid>");
+ } else {
+ struct passwd *pwd;
+ char *p2 = strchr (p, '.');
- *p2 = 0;
- pwd = getpwnam ( p );
+ if (!p2)
+ parse_error ("parsing <uid>.<gid>");
- if ( !pwd )
- parse_error ( "invalid user name" );
+ *p2 = 0;
+ pwd = getpwnam (p);
- sct-> m_uid = pwd-> pw_uid;
- p = p2 + 1;
- }
- if ( isdigit ( *p ))
- sct-> m_gid = strtol ( p, &p, 10 );
- else {
- struct group *grp = getgrnam ( p );
+ if (!pwd)
+ parse_error ("invalid user name");
- if ( !grp )
- parse_error ( "invalid group name" );
+ sct->m_uid = pwd->pw_uid;
+ p = p2 + 1;
+ }
+ if (isdigit (*p))
+ sct->m_gid = strtol (p, &p, 10);
+ else {
+ struct group *grp = getgrnam (p);
- sct-> m_gid = grp-> gr_gid;
- }
- }
- break;
- }
- default: /* unknown - skip */
- break;
- }
- }
- else
- parse_error ( "keyword not within section" );
- }
- fclose ( f );
- return 1;
+ if (!grp)
+ parse_error ("invalid group name");
+
+ sct->m_gid = grp->gr_gid;
+ }
+ }
+ break;
+ }
+ default: /* unknown - skip */
+ break;
}
+ } else
+ parse_error ("keyword not within section");
}
+ fclose (f);
+ return 1;
+ }
}
- return 0; /* no config file or not readable (not an error) */
+ }
+ return 0; /* no config file or not readable (not an error) */
-pe_label:
- fprintf ( stderr, "Parse error in %s, line %d: %s\n", CONFIG_FILE, lc, err );
+pe_label:
+ fprintf (stderr, "Parse error in %s, line %d: %s\n", CONFIG_FILE, lc, err);
- if ( f )
- fclose ( f );
- return 0;
+ if (f)
+ fclose (f);
+ return 0;
}
#endif
More information about the busybox-cvs
mailing list