X-Git-Url: https://git.librecmc.org/?a=blobdiff_plain;f=miscutils%2Fdevfsd.c;h=f3d935b2e716499686d0438fb83ee6bbd9315f82;hb=eb7f9acda147543079f261753a11d2afa340fc5e;hp=de046e818cc96ae23d4e329bb149cb54c0acc684;hpb=5af906e7c834301a0f237b50e1a1474ce0cf6da0;p=oweals%2Fbusybox.git diff --git a/miscutils/devfsd.c b/miscutils/devfsd.c index de046e818..f3d935b2e 100644 --- a/miscutils/devfsd.c +++ b/miscutils/devfsd.c @@ -1,6 +1,6 @@ /* vi: set sw=4 ts=4: */ /* - * Licensed under GPLv2 or later, see file LICENSE in this tarball for details. + * Licensed under GPLv2 or later, see file LICENSE in this source tree. */ /* @@ -53,28 +53,86 @@ The postal address is: Richard Gooch, c/o ATNF, P. O. Box 76, Epping, N.S.W., 2121, Australia. */ - -#include "busybox.h" +//config:config DEVFSD +//config: bool "devfsd (obsolete)" +//config: default n +//config: select PLATFORM_LINUX +//config: select FEATURE_SYSLOG +//config: help +//config: This is deprecated and should NOT be used anymore. +//config: Use linux >= 2.6 (optionally with hotplug) and mdev instead! +//config: See docs/mdev.txt for detailed instructions on how to use mdev +//config: instead. +//config: +//config: Provides compatibility with old device names on a devfs systems. +//config: You should set it to true if you have devfs enabled. +//config: The following keywords in devsfd.conf are supported: +//config: "CLEAR_CONFIG", "INCLUDE", "OPTIONAL_INCLUDE", "RESTORE", +//config: "PERMISSIONS", "EXECUTE", "COPY", "IGNORE", +//config: "MKOLDCOMPAT", "MKNEWCOMPAT","RMOLDCOMPAT", "RMNEWCOMPAT". +//config: +//config: But only if they are written UPPERCASE!!!!!!!! +//config: +//config:config DEVFSD_MODLOAD +//config: bool "Adds support for MODLOAD keyword in devsfd.conf" +//config: default y +//config: depends on DEVFSD +//config: help +//config: This actually doesn't work with busybox modutils but needs +//config: the external modutils. +//config: +//config:config DEVFSD_FG_NP +//config: bool "Enable the -fg and -np options" +//config: default y +//config: depends on DEVFSD +//config: help +//config: -fg Run the daemon in the foreground. +//config: -np Exit after parsing config. Do not poll for events. +//config: +//config:config DEVFSD_VERBOSE +//config: bool "Increases logging (and size)" +//config: default y +//config: depends on DEVFSD +//config: help +//config: Increases logging to stderr or syslog. +//config: +//config:config FEATURE_DEVFS +//config: bool "Use devfs names for all devices (obsolete)" +//config: default n +//config: select PLATFORM_LINUX +//config: help +//config: This is obsolete and should NOT be used anymore. +//config: Use linux >= 2.6 (optionally with hotplug) and mdev instead! +//config: +//config: For legacy systems -- if there is no way around devfsd -- this +//config: tells busybox to look for names like /dev/loop/0 instead of +//config: /dev/loop0. If your /dev directory has normal names instead of +//config: devfs names, you don't want this. + +//applet:IF_DEVFSD(APPLET(devfsd, BB_DIR_SBIN, BB_SUID_DROP)) + +//kbuild:lib-$(CONFIG_DEVFSD) += devfsd.o + +//usage:#define devfsd_trivial_usage +//usage: "mntpnt [-v]" IF_DEVFSD_FG_NP("[-fg][-np]") +//usage:#define devfsd_full_usage "\n\n" +//usage: "Manage devfs permissions and old device name symlinks\n" +//usage: "\n mntpnt The mount point where devfs is mounted" +//usage: "\n -v Print the protocol version numbers for devfsd" +//usage: "\n and the kernel-side protocol version and exit" +//usage: IF_DEVFSD_FG_NP( +//usage: "\n -fg Run in foreground" +//usage: "\n -np Exit after parsing the configuration file" +//usage: "\n and processing synthetic REGISTER events," +//usage: "\n don't poll for events" +//usage: ) + +#include "libbb.h" #include "xregex.h" -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include #include -#include -#include -#include +#include +#include /* Various defines taken from linux/major.h */ #define IDE0_MAJOR 3 @@ -91,7 +149,7 @@ /* Various defines taken from linux/devfs_fs.h */ #define DEVFSD_PROTOCOL_REVISION_KERNEL 5 -#define DEVFSD_IOCTL_BASE 'd' +#define DEVFSD_IOCTL_BASE 'd' /* These are the various ioctls */ #define DEVFSDIOC_GET_PROTO_REV _IOR(DEVFSD_IOCTL_BASE, 0, int) #define DEVFSDIOC_SET_EVENT_MASK _IOW(DEVFSD_IOCTL_BASE, 2, int) @@ -108,18 +166,18 @@ #define DEVFS_PATHLEN 1024 /* Never change this otherwise the binary interface will change */ -struct devfsd_notify_struct -{ /* Use native C types to ensure same types in kernel and user space */ - unsigned int type; /* DEVFSD_NOTIFY_* value */ - unsigned int mode; /* Mode of the inode or device entry */ - unsigned int major; /* Major number of device entry */ - unsigned int minor; /* Minor number of device entry */ - unsigned int uid; /* Uid of process, inode or device entry */ - unsigned int gid; /* Gid of process, inode or device entry */ - unsigned int overrun_count; /* Number of lost events */ - unsigned int namelen; /* Number of characters not including '\0' */ - /* The device name MUST come last */ - char devname[DEVFS_PATHLEN]; /* This will be '\0' terminated */ +struct devfsd_notify_struct { + /* Use native C types to ensure same types in kernel and user space */ + unsigned int type; /* DEVFSD_NOTIFY_* value */ + unsigned int mode; /* Mode of the inode or device entry */ + unsigned int major; /* Major number of device entry */ + unsigned int minor; /* Minor number of device entry */ + unsigned int uid; /* Uid of process, inode or device entry */ + unsigned int gid; /* Gid of process, inode or device entry */ + unsigned int overrun_count; /* Number of lost events */ + unsigned int namelen; /* Number of characters not including '\0' */ + /* The device name MUST come last */ + char devname[DEVFS_PATHLEN]; /* This will be '\0' terminated */ }; #define BUFFER_SIZE 16384 @@ -167,275 +225,187 @@ struct devfsd_notify_struct #define AC_RMNEWCOMPAT 10 #define AC_RESTORE 11 -struct permissions_type -{ - mode_t mode; - uid_t uid; - gid_t gid; +struct permissions_type { + mode_t mode; + uid_t uid; + gid_t gid; }; -struct execute_type -{ - char *argv[MAX_ARGS + 1]; /* argv[0] must always be the programme */ +struct execute_type { + char *argv[MAX_ARGS + 1]; /* argv[0] must always be the programme */ }; -struct copy_type -{ - const char *source; - const char *destination; +struct copy_type { + const char *source; + const char *destination; }; -struct action_type -{ - unsigned int what; - unsigned int when; +struct action_type { + unsigned int what; + unsigned int when; }; -struct config_entry_struct -{ - struct action_type action; - regex_t preg; - union - { +struct config_entry_struct { + struct action_type action; + regex_t preg; + union + { struct permissions_type permissions; struct execute_type execute; struct copy_type copy; - } - u; - struct config_entry_struct *next; + } + u; + struct config_entry_struct *next; }; -struct get_variable_info -{ - const struct devfsd_notify_struct *info; - const char *devname; - char devpath[STRING_LENGTH]; +struct get_variable_info { + const struct devfsd_notify_struct *info; + const char *devname; + char devpath[STRING_LENGTH]; }; -static void dir_operation(int , const char * , int, unsigned long* ); +static void dir_operation(int , const char * , int, unsigned long*); static void service(struct stat statbuf, char *path); -static int st_expr_expand(char *, unsigned, const char *, const char *(*) (const char *, void *), void *); +static int st_expr_expand(char *, unsigned, const char *, const char *(*)(const char *, void *), void *); static const char *get_old_name(const char *, unsigned, char *, unsigned, unsigned); -static int mksymlink (const char *oldpath, const char *newpath); -static void read_config_file (char *path, int optional, unsigned long *event_mask); -static void process_config_line (const char *, unsigned long *); -static int do_servicing (int, unsigned long); -static void service_name (const struct devfsd_notify_struct *); -static void action_permissions (const struct devfsd_notify_struct *, const struct config_entry_struct *); -static void action_execute (const struct devfsd_notify_struct *, const struct config_entry_struct *, +static int mksymlink(const char *oldpath, const char *newpath); +static void read_config_file(char *path, int optional, unsigned long *event_mask); +static void process_config_line(const char *, unsigned long *); +static int do_servicing(int, unsigned long); +static void service_name(const struct devfsd_notify_struct *); +static void action_permissions(const struct devfsd_notify_struct *, const struct config_entry_struct *); +static void action_execute(const struct devfsd_notify_struct *, const struct config_entry_struct *, const regmatch_t *, unsigned); -static void action_modload (const struct devfsd_notify_struct *info, const struct config_entry_struct *entry); -static void action_copy (const struct devfsd_notify_struct *, const struct config_entry_struct *, - const regmatch_t *, unsigned); -static void action_compat (const struct devfsd_notify_struct *, unsigned); -static void free_config (void); +static void action_modload(const struct devfsd_notify_struct *info, const struct config_entry_struct *entry); +static void action_copy(const struct devfsd_notify_struct *, const struct config_entry_struct *, + const regmatch_t *, unsigned); +static void action_compat(const struct devfsd_notify_struct *, unsigned); +static void free_config(void); static void restore(char *spath, struct stat source_stat, int rootlen); -static int copy_inode (const char *, const struct stat *, mode_t, const char *, const struct stat *); -static mode_t get_mode (const char *); -static void signal_handler (int); -static const char *get_variable (const char *, void *); -static int make_dir_tree (const char *); +static int copy_inode(const char *, const struct stat *, mode_t, const char *, const struct stat *); +static mode_t get_mode(const char *); +static void signal_handler(int); +static const char *get_variable(const char *, void *); +static int make_dir_tree(const char *); static int expand_expression(char *, unsigned, const char *, const char *(*)(const char *, void *), void *, - const char *, const regmatch_t *, unsigned ); -static void expand_regexp (char *, size_t, const char *, const char *, const regmatch_t *, unsigned ); + const char *, const regmatch_t *, unsigned); +static void expand_regexp(char *, size_t, const char *, const char *, const regmatch_t *, unsigned); static const char *expand_variable( char *, unsigned, unsigned *, const char *, - const char *(*) (const char *, void *), void * ); -static const char *get_variable_v2(const char *, const char *(*) (const char *, void *), void *); -static char get_old_ide_name (unsigned , unsigned); -static char *write_old_sd_name (char *, unsigned, unsigned, char *); + const char *(*)(const char *, void *), void *); +static const char *get_variable_v2(const char *, const char *(*)(const char *, void *), void *); +static char get_old_ide_name(unsigned, unsigned); +static char *write_old_sd_name(char *, unsigned, unsigned, const char *); /* busybox functions */ -static void msg_logger(int pri, const char * fmt, ... )__attribute__ ((format (printf, 2, 3))); -static void msg_logger_and_die(int pri, const char * fmt, ... )__attribute__ ((noreturn, format (printf, 2, 3))); -static void do_ioctl_and_die(int fd, int request, unsigned long event_mask_flag); -static void fork_and_execute(int die, char *arg0, char **arg ); -static int get_uid_gid ( int, const char *); -static void safe_memcpy( char * dest, const char * src, int len); -static unsigned int scan_dev_name_common(const char *d, unsigned int n, int addendum, char *ptr); -static unsigned int scan_dev_name(const char *d, unsigned int n, char *ptr); +static int get_uid_gid(int flag, const char *string); +static void safe_memcpy(char * dest, const char * src, int len); +static unsigned int scan_dev_name_common(const char *d, unsigned int n, int addendum, const char *ptr); +static unsigned int scan_dev_name(const char *d, unsigned int n, const char *ptr); /* Structs and vars */ static struct config_entry_struct *first_config = NULL; static struct config_entry_struct *last_config = NULL; -static const char *mount_point = NULL; +static char *mount_point = NULL; static volatile int caught_signal = FALSE; static volatile int caught_sighup = FALSE; -static struct initial_symlink_struct -{ - char *dest; - char *name; -} initial_symlinks[] = -{ - {"/proc/self/fd", "fd"}, - {"fd/0", "stdin"}, - {"fd/1", "stdout"}, - {"fd/2", "stderr"}, - {NULL, NULL}, +static struct initial_symlink_struct { + const char *dest; + const char *name; +} initial_symlinks[] = { + {"/proc/self/fd", "fd"}, + {"fd/0", "stdin"}, + {"fd/1", "stdout"}, + {"fd/2", "stderr"}, + {NULL, NULL}, }; -static struct event_type -{ - unsigned int type; /* The DEVFSD_NOTIFY_* value */ - const char *config_name; /* The name used in the config file */ -} event_types[] = -{ - {DEVFSD_NOTIFY_REGISTERED, "REGISTER"}, - {DEVFSD_NOTIFY_UNREGISTERED, "UNREGISTER"}, - {DEVFSD_NOTIFY_ASYNC_OPEN, "ASYNC_OPEN"}, - {DEVFSD_NOTIFY_CLOSE, "CLOSE"}, - {DEVFSD_NOTIFY_LOOKUP, "LOOKUP"}, - {DEVFSD_NOTIFY_CHANGE, "CHANGE"}, - {DEVFSD_NOTIFY_CREATE, "CREATE"}, - {DEVFSD_NOTIFY_DELETE, "DELETE"}, - {0xffffffff, NULL} +static struct event_type { + unsigned int type; /* The DEVFSD_NOTIFY_* value */ + const char *config_name; /* The name used in the config file */ +} event_types[] = { + {DEVFSD_NOTIFY_REGISTERED, "REGISTER"}, + {DEVFSD_NOTIFY_UNREGISTERED, "UNREGISTER"}, + {DEVFSD_NOTIFY_ASYNC_OPEN, "ASYNC_OPEN"}, + {DEVFSD_NOTIFY_CLOSE, "CLOSE"}, + {DEVFSD_NOTIFY_LOOKUP, "LOOKUP"}, + {DEVFSD_NOTIFY_CHANGE, "CHANGE"}, + {DEVFSD_NOTIFY_CREATE, "CREATE"}, + {DEVFSD_NOTIFY_DELETE, "DELETE"}, + {0xffffffff, NULL} }; /* Busybox messages */ -static const char * const bb_msg_proto_rev = "protocol revision"; -static const char * const bb_msg_bad_config = "bad %s config file: %s"; -static const char * const bb_msg_small_buffer = "buffer too small"; -static const char * const bb_msg_variable_not_found = "variable: %s not found"; - -/* Busybox functions */ -static void msg_logger(int pri, const char * fmt, ... ) -{ - va_list ap; - int ret; - - va_start(ap, fmt); - ret = access ("/dev/log", F_OK); - if (ret == 0) { - openlog(applet_name, 0, LOG_DAEMON); - vsyslog( pri , fmt, ap); - /* Man: A trailing newline is added when needed. */ - closelog(); - } - /* ENABLE_DEVFSD_VERBOSE is always enabled if msg_logger is used */ - if ((ENABLE_DEVFSD_VERBOSE && ret) || ENABLE_DEBUG) { - bb_error_msg(fmt, ap); - } - va_end(ap); -} - -static void msg_logger_and_die(int pri, const char* fmt, ...) -{ - va_list ap; - - va_start(ap, fmt); - msg_logger(pri, fmt, ap); - va_end(ap); - exit(EXIT_FAILURE); -} +static const char bb_msg_proto_rev[] ALIGN1 = "protocol revision"; +static const char bb_msg_bad_config[] ALIGN1 = "bad %s config file: %s"; +static const char bb_msg_small_buffer[] ALIGN1 = "buffer too small"; +static const char bb_msg_variable_not_found[] ALIGN1 = "variable: %s not found"; /* Busybox stuff */ -#if defined(CONFIG_DEVFSD_VERBOSE) || defined(CONFIG_DEBUG) -#define devfsd_error_msg(fmt, args...) bb_error_msg(fmt, ## args) -#define devfsd_perror_msg_and_die(fmt, args...) bb_perror_msg_and_die(fmt, ## args) -#define devfsd_error_msg_and_die(fmt, args...) bb_error_msg_and_die(fmt, ## args) -#if defined(CONFIG_DEBUG) -#define debug_msg_logger(x, fmt, args...) msg_logger(x, fmt, ## args) +#if ENABLE_DEVFSD_VERBOSE || ENABLE_DEBUG +#define info_logger(p, fmt, args...) bb_info_msg(fmt, ## args) +#define simple_info_logger(p, msg) bb_simple_info_msg(msg) +#define msg_logger(p, fmt, args...) bb_error_msg(fmt, ## args) +#define simple_msg_logger(p, msg) bb_simple_error_msg(msg) +#define msg_logger_and_die(p, fmt, args...) bb_error_msg_and_die(fmt, ## args) +#define simple_msg_logger_and_die(p, msg) bb_simple_error_msg_and_die(msg) +#define error_logger(p, fmt, args...) bb_perror_msg(fmt, ## args) +#define error_logger_and_die(p, fmt, args...) bb_perror_msg_and_die(fmt, ## args) #else -#define debug_msg_logger(x, fmt, args...) -#endif -#else -#define debug_msg_logger(x, fmt, args...) +#define info_logger(p, fmt, args...) #define msg_logger(p, fmt, args...) -#define msg_logger_and_die(p, fmt, args...) exit(1) -#define devfsd_perror_msg_and_die(fmt, args...) exit(1) -#define devfsd_error_msg_and_die(fmt, args...) exit(1) -#define devfsd_error_msg(fmt, args...) +#define simple_msg_logger(p, msg) +#define msg_logger_and_die(p, fmt, args...) exit(EXIT_FAILURE) +#define simple_msg_logger_and_die(p, msg) exit(EXIT_FAILURE) +#define error_logger(p, fmt, args...) +#define error_logger_and_die(p, fmt, args...) exit(EXIT_FAILURE) #endif -static void do_ioctl_and_die(int fd, int request, unsigned long event_mask_flag) -{ - if (ioctl (fd, request, event_mask_flag) == -1) - msg_logger_and_die(LOG_ERR, "ioctl"); -} - -static void fork_and_execute(int die, char *arg0, char **arg ) +static void safe_memcpy(char *dest, const char *src, int len) { - switch ( fork () ) - { - case 0: - /* Child */ - break; - case -1: - /* Parent: Error : die or return */ - msg_logger(LOG_ERR,(char *) bb_msg_memory_exhausted); - if(die) - exit(EXIT_FAILURE); - return; - default: - /* Parent : ok : return or exit */ - if(arg0 != NULL) - { - wait (NULL); - return; - } - exit (EXIT_SUCCESS); - } - /* Child : if arg0 != NULL do execvp */ - if(arg0 != NULL ) - { - execvp (arg0, arg); - msg_logger_and_die(LOG_ERR, "execvp"); - } -} - -static void safe_memcpy( char *dest, const char *src, int len) -{ - memcpy (dest , src , len ); + memcpy(dest , src, len); dest[len] = '\0'; } -static unsigned int scan_dev_name_common(const char *d, unsigned int n, int addendum, char *ptr) +static unsigned int scan_dev_name_common(const char *d, unsigned int n, int addendum, const char *ptr) { - if( d[n - 4]=='d' && d[n - 3]=='i' && d[n - 2]=='s' && d[n - 1]=='c') - return (2 + addendum); - else if( d[n - 2]=='c' && d[n - 1]=='d') - return (3 + addendum); - else if(ptr[0]=='p' && ptr[1]=='a' && ptr[2]=='r' && ptr[3]=='t') - return (4 + addendum); - else if( ptr[n - 2]=='m' && ptr[n - 1]=='t') - return (5 + addendum); - else - return 0; + if (d[n - 4] == 'd' && d[n - 3] == 'i' && d[n - 2] == 's' && d[n - 1] == 'c') + return 2 + addendum; + if (d[n - 2] == 'c' && d[n - 1] == 'd') + return 3 + addendum; + if (ptr[0] == 'p' && ptr[1] == 'a' && ptr[2] == 'r' && ptr[3] == 't') + return 4 + addendum; + if (ptr[n - 2] == 'm' && ptr[n - 1] == 't') + return 5 + addendum; + return 0; } -static unsigned int scan_dev_name(const char *d, unsigned int n, char *ptr) +static unsigned int scan_dev_name(const char *d, unsigned int n, const char *ptr) { - if(d[0]=='s' && d[1]=='c' && d[2]=='s' && d[3]=='i' && d[4]=='/') - { - if( d[n - 7]=='g' && d[n - 6]=='e' && d[n - 5]=='n' && - d[n - 4]=='e' && d[n - 3]=='r' && d[n - 2]=='i' && - d[n - 1]=='c' ) + if (d[0] == 's' && d[1] == 'c' && d[2] == 's' && d[3] == 'i' && d[4] == '/') { + if (d[n - 7] == 'g' && d[n - 6] == 'e' && d[n - 5] == 'n' + && d[n - 4] == 'e' && d[n - 3] == 'r' && d[n - 2] == 'i' && d[n - 1] == 'c' + ) return 1; return scan_dev_name_common(d, n, 0, ptr); } - else if(d[0]=='i' && d[1]=='d' && d[2]=='e' && d[3]=='/' && - d[4]=='h' && d[5]=='o' && d[6]=='s' && d[7]=='t') - { + if (d[0] == 'i' && d[1] == 'd' && d[2] == 'e' && d[3] == '/' + && d[4] == 'h' && d[5] == 'o' && d[6] == 's' && d[7] == 't' + ) return scan_dev_name_common(d, n, 4, ptr); - } - else if(d[0]=='s' && d[1]=='b' && d[2]=='p' && d[3]=='/') - { + if (d[0] == 's' && d[1] == 'b' && d[2] == 'p' && d[3] == '/') return 10; - } - else if(d[0]=='v' && d[1]=='c' && d[2]=='c' && d[3]=='/') - { + if (d[0] == 'v' && d[1] == 'c' && d[2] == 'c' && d[3] == '/') return 11; - } - else if(d[0]=='p' && d[1]=='t' && d[2]=='y' && d[3]=='/') - { + if (d[0] == 'p' && d[1] == 't' && d[2] == 'y' && d[3] == '/') return 12; - } return 0; } /* Public functions follow */ -int devfsd_main (int argc, char **argv) +int devfsd_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; +int devfsd_main(int argc, char **argv) { int print_version = FALSE; int do_daemon = TRUE; @@ -449,104 +419,98 @@ int devfsd_main (int argc, char **argv) if (argc < 2) bb_show_usage(); - for (count = 2; count < argc; ++count) - { - if(argv[count][0] == '-') - { - if(argv[count][1]=='v' && !argv[count][2]) /* -v */ - print_version = TRUE; - else if(ENABLE_DEVFSD_FG_NP && argv[count][1]=='f' - && argv[count][2]=='g' && !argv[count][3]) /* -fg */ - do_daemon = FALSE; - else if(ENABLE_DEVFSD_FG_NP && argv[count][1]=='n' - && argv[count][2]=='p' && !argv[count][3]) /* -np */ - no_polling = TRUE; + for (count = 2; count < argc; ++count) { + if (argv[count][0] == '-') { + if (argv[count][1] == 'v' && !argv[count][2]) /* -v */ + print_version = TRUE; + else if (ENABLE_DEVFSD_FG_NP && argv[count][1] == 'f' + && argv[count][2] == 'g' && !argv[count][3]) /* -fg */ + do_daemon = FALSE; + else if (ENABLE_DEVFSD_FG_NP && argv[count][1] == 'n' + && argv[count][2] == 'p' && !argv[count][3]) /* -np */ + no_polling = TRUE; else bb_show_usage(); } } - /* strip last / from mount point, so we don't need to check for it later */ - while( argv[1][1]!='\0' && argv[1][strlen(argv[1])-1] == '/' ) - argv[1][strlen(argv[1]) -1] = '\0'; - - mount_point = argv[1]; + mount_point = bb_simplify_path(argv[1]); - if (chdir (mount_point) != 0) - devfsd_perror_msg_and_die(mount_point); + xchdir(mount_point); - fd = xopen (".devfsd", O_RDONLY); - - if (fcntl (fd, F_SETFD, FD_CLOEXEC) != 0) - devfsd_perror_msg_and_die("FD_CLOEXEC"); - - if (ioctl (fd, DEVFSDIOC_GET_PROTO_REV, &proto_rev) == -1) - msg_logger_and_die(LOG_ERR, "ioctl"); + fd = xopen(".devfsd", O_RDONLY); + close_on_exec_on(fd); + xioctl(fd, DEVFSDIOC_GET_PROTO_REV, &proto_rev); /*setup initial entries */ - for (curr = initial_symlinks; curr->dest != NULL; ++curr) - symlink (curr->dest, curr->name); + for (curr = initial_symlinks; curr->dest != NULL; ++curr) + symlink(curr->dest, curr->name); /* NB: The check for CONFIG_FILE is done in read_config_file() */ - if ( print_version || (DEVFSD_PROTOCOL_REVISION_DAEMON != proto_rev) ) - { - printf( "%s v%s\nDaemon %s:\t%d\nKernel-side %s:\t%d\n", - applet_name,DEVFSD_VERSION,bb_msg_proto_rev, - DEVFSD_PROTOCOL_REVISION_DAEMON,bb_msg_proto_rev, proto_rev); + if (print_version || (DEVFSD_PROTOCOL_REVISION_DAEMON != proto_rev)) { + printf("%s v%s\nDaemon %s:\t%d\nKernel-side %s:\t%d\n", + applet_name, DEVFSD_VERSION, bb_msg_proto_rev, + DEVFSD_PROTOCOL_REVISION_DAEMON, bb_msg_proto_rev, proto_rev); if (DEVFSD_PROTOCOL_REVISION_DAEMON != proto_rev) - bb_error_msg_and_die( "%s mismatch!",bb_msg_proto_rev); + bb_error_msg_and_die("%s mismatch!", bb_msg_proto_rev); exit(EXIT_SUCCESS); /* -v */ } - /* Tell kernel we are special (i.e. we get to see hidden entries) */ - do_ioctl_and_die(fd, DEVFSDIOC_SET_EVENT_MASK, 0); - - sigemptyset (&new_action.sa_mask); - new_action.sa_flags = 0; + /* Tell kernel we are special(i.e. we get to see hidden entries) */ + xioctl(fd, DEVFSDIOC_SET_EVENT_MASK, 0); /* Set up SIGHUP and SIGUSR1 handlers */ + sigemptyset(&new_action.sa_mask); + new_action.sa_flags = 0; new_action.sa_handler = signal_handler; - if (sigaction (SIGHUP, &new_action, NULL) != 0 || sigaction (SIGUSR1, &new_action, NULL) != 0 ) - devfsd_error_msg_and_die( "sigaction"); + sigaction_set(SIGHUP, &new_action); + sigaction_set(SIGUSR1, &new_action); - printf("%s v%s started for %s\n",applet_name, DEVFSD_VERSION, mount_point); + printf("%s v%s started for %s\n", applet_name, DEVFSD_VERSION, mount_point); /* Set umask so that mknod(2), open(2) and mkdir(2) have complete control over permissions */ - umask (0); - read_config_file (CONFIG_FILE, FALSE, &event_mask); + umask(0); + read_config_file((char*)CONFIG_FILE, FALSE, &event_mask); /* Do the scan before forking, so that boot scripts see the finished product */ - dir_operation(SERVICE,mount_point,0,NULL); + dir_operation(SERVICE, mount_point, 0, NULL); if (ENABLE_DEVFSD_FG_NP && no_polling) - exit (0); - if (do_daemon) - { + exit(EXIT_SUCCESS); + + if (ENABLE_DEVFSD_VERBOSE || ENABLE_DEBUG) + logmode = LOGMODE_BOTH; + else if (do_daemon == TRUE) + logmode = LOGMODE_SYSLOG; + /* This is the default */ + /*else + logmode = LOGMODE_STDIO; */ + + if (do_daemon) { /* Release so that the child can grab it */ - do_ioctl_and_die(fd, DEVFSDIOC_RELEASE_EVENT_QUEUE, 0); - fork_and_execute(DIE, NULL, NULL); - setsid (); /* Prevent hangups and become pgrp leader */ - } else if(ENABLE_DEVFSD_FG_NP) { - setpgid (0, 0); /* Become process group leader */ + xioctl(fd, DEVFSDIOC_RELEASE_EVENT_QUEUE, 0); + bb_daemonize_or_rexec(0, argv); + } else if (ENABLE_DEVFSD_FG_NP) { + setpgid(0, 0); /* Become process group leader */ } - while (TRUE) - { - do_scan = do_servicing (fd, event_mask); + while (TRUE) { + do_scan = do_servicing(fd, event_mask); - free_config (); - read_config_file (CONFIG_FILE, FALSE, &event_mask); + free_config(); + read_config_file((char*)CONFIG_FILE, FALSE, &event_mask); if (do_scan) - dir_operation(SERVICE,mount_point,0,NULL); + dir_operation(SERVICE, mount_point, 0, NULL); } + if (ENABLE_FEATURE_CLEAN_UP) free(mount_point); } /* End Function main */ /* Private functions follow */ -static void read_config_file (char *path, int optional, unsigned long *event_mask) +static void read_config_file(char *path, int optional, unsigned long *event_mask) /* [SUMMARY] Read a configuration database. The path to read the database from. If this is a directory, all - entries in that directory will be read (except hidden entries). + entries in that directory will be read(except hidden entries). If TRUE, the routine will silently ignore a missing config file. The event mask is written here. This is not initialised. [RETURNS] Nothing. @@ -555,48 +519,41 @@ static void read_config_file (char *path, int optional, unsigned long *event_mas struct stat statbuf; FILE *fp; char buf[STRING_LENGTH]; - char *line=NULL; - - debug_msg_logger(LOG_INFO, "%s: %s", __FUNCTION__, path); + char *line = NULL; + char *p; - if (stat (path, &statbuf) == 0 ) - { + if (stat(path, &statbuf) == 0) { /* Don't read 0 length files: ignored */ - /*if( statbuf.st_size == 0 ) + /*if (statbuf.st_size == 0) return;*/ - if ( S_ISDIR (statbuf.st_mode) ) - { - /* strip last / from dirname so we don't need to check for it later */ - while( path && path[1]!='\0' && path[strlen(path)-1] == '/') - path[strlen(path) -1] = '\0'; - - dir_operation(READ_CONFIG, path, 0, event_mask); + if (S_ISDIR(statbuf.st_mode)) { + p = bb_simplify_path(path); + dir_operation(READ_CONFIG, p, 0, event_mask); + free(p); return; } - if ( ( fp = fopen (path, "r") ) != NULL ) - { - while (fgets (buf, STRING_LENGTH, fp) != NULL) - { + fp = fopen_for_read(path); + if (fp != NULL) { + while (fgets(buf, STRING_LENGTH, fp) != NULL) { /* Skip whitespace */ - for (line = buf; isspace (*line); ++line) - /*VOID*/; - if (line[0] == '\0' || line[0] == '#' ) + line = buf; + line = skip_whitespace(line); + if (line[0] == '\0' || line[0] == '#') continue; - process_config_line (line, event_mask); + process_config_line(line, event_mask); } - fclose (fp); + fclose(fp); } else { goto read_config_file_err; } } else { read_config_file_err: - if(optional == 0 && errno == ENOENT) - msg_logger_and_die(LOG_ERR, "read config file: %s: %m", path); + if (optional == 0 && errno == ENOENT) + error_logger_and_die(LOG_ERR, "read config file: %s", path); } - return; } /* End Function read_config_file */ -static void process_config_line (const char *line, unsigned long *event_mask) +static void process_config_line(const char *line, unsigned long *event_mask) /* [SUMMARY] Process a line from a configuration file. The configuration line. The event mask is written here. This is not initialised. @@ -608,103 +565,94 @@ static void process_config_line (const char *line, unsigned long *event_mask) char p[MAX_ARGS][STRING_LENGTH]; char when[STRING_LENGTH], what[STRING_LENGTH]; char name[STRING_LENGTH]; - char * msg=""; + const char *msg = ""; char *ptr; int i; /* !!!! Only Uppercase Keywords in devsfd.conf */ - static const char *const options[] = { - "CLEAR_CONFIG", "INCLUDE", "OPTIONAL_INCLUDE", - "RESTORE", "PERMISSIONS", "MODLOAD", "EXECUTE", - "COPY", "IGNORE", "MKOLDCOMPAT", "MKNEWCOMPAT", - "RMOLDCOMPAT", "RMNEWCOMPAT", 0 - }; - - debug_msg_logger(LOG_INFO, __FUNCTION__); - - for (count = 0; count < MAX_ARGS; ++count) p[count][0] = '\0'; - num_args = sscanf (line, "%s %s %s %s %s %s %s %s %s %s", + static const char options[] ALIGN1 = + "CLEAR_CONFIG\0""INCLUDE\0""OPTIONAL_INCLUDE\0" + "RESTORE\0""PERMISSIONS\0""MODLOAD\0""EXECUTE\0" + "COPY\0""IGNORE\0""MKOLDCOMPAT\0""MKNEWCOMPAT\0" + "RMOLDCOMPAT\0""RMNEWCOMPAT\0"; + + for (count = 0; count < MAX_ARGS; ++count) + p[count][0] = '\0'; + num_args = sscanf(line, "%s %s %s %s %s %s %s %s %s %s", when, name, what, p[0], p[1], p[2], p[3], p[4], p[5], p[6]); - i = index_in_str_array(options, when ); + i = index_in_strings(options, when); - /*"CLEAR_CONFIG"*/ - if( i == 0) - { - free_config (); + /* "CLEAR_CONFIG" */ + if (i == 0) { + free_config(); *event_mask = 0; return; } - if ( num_args < 2) + if (num_args < 2) goto process_config_line_err; /* "INCLUDE" & "OPTIONAL_INCLUDE" */ - if( i == 1 || i == 2 ) - { - st_expr_expand (name, STRING_LENGTH, name, get_variable, NULL ); - msg_logger(LOG_INFO, "%sinclude: %s",(toupper (when[0]) == 'I') ? "": "optional_", name); - read_config_file (name, (toupper (when[0]) == 'I') ? FALSE : TRUE, event_mask); + if (i == 1 || i == 2) { + st_expr_expand(name, STRING_LENGTH, name, get_variable, NULL); + info_logger(LOG_INFO, "%sinclude: %s", (toupper(when[0]) == 'I') ? "": "optional_", name); + read_config_file(name, (toupper(when[0]) == 'I') ? FALSE : TRUE, event_mask); return; } /* "RESTORE" */ - if( i == 3) - { - dir_operation(RESTORE,name, strlen (name),NULL); + if (i == 3) { + dir_operation(RESTORE, name, strlen(name),NULL); return; } if (num_args < 3) goto process_config_line_err; - new = xmalloc (sizeof *new); - memset (new, 0, sizeof *new); + new = xzalloc(sizeof *new); - for (count = 0; event_types[count].config_name != NULL; ++count) - { - if (strcasecmp (when, event_types[count].config_name) != 0) + for (count = 0; event_types[count].config_name != NULL; ++count) { + if (strcasecmp(when, event_types[count].config_name) != 0) continue; new->action.when = event_types[count].type; break; } - if (event_types[count].config_name == NULL) - { - msg="WHEN in"; + if (event_types[count].config_name == NULL) { + msg = "WHEN in"; goto process_config_line_err; } - i = index_in_str_array(options, what ); + i = index_in_strings(options, what); - switch(i) - { + switch (i) { case 4: /* "PERMISSIONS" */ new->action.what = AC_PERMISSIONS; /* Get user and group */ - if ( ( ptr = strchr (p[0], '.') ) == NULL ) - { - msg="UID.GID"; + ptr = strchr(p[0], '.'); + if (ptr == NULL) { + msg = "UID.GID"; goto process_config_line_err; /*"missing '.' in UID.GID"*/ } *ptr++ = '\0'; - new->u.permissions.uid = get_uid_gid (UID, p[0]); - new->u.permissions.gid = get_uid_gid (GID, ptr); + new->u.permissions.uid = get_uid_gid(UID, p[0]); + new->u.permissions.gid = get_uid_gid(GID, ptr); /* Get mode */ - new->u.permissions.mode = get_mode (p[1]); + new->u.permissions.mode = get_mode(p[1]); break; case 5: /* MODLOAD */ - /*This action will pass "/dev/$devname" (i.e. "/dev/" prefixed to + /*This action will pass "/dev/$devname"(i.e. "/dev/" prefixed to the device name) to the module loading facility. In addition, the /etc/modules.devfs configuration file is used.*/ - if (ENABLE_DEVFSD_MODLOAD) + if (ENABLE_DEVFSD_MODLOAD) new->action.what = AC_MODLOAD; - break; + break; case 6: /* EXECUTE */ new->action.what = AC_EXECUTE; num_args -= 3; for (count = 0; count < num_args; ++count) - new->u.execute.argv[count] = xstrdup (p[count]); + new->u.execute.argv[count] = xstrdup(p[count]); new->u.execute.argv[num_args] = NULL; break; @@ -714,8 +662,8 @@ static void process_config_line (const char *line, unsigned long *event_mask) if (num_args != 2) goto process_config_line_err; /* missing path and function in line */ - new->u.copy.source = xstrdup (p[0]); - new->u.copy.destination = xstrdup (p[1]); + new->u.copy.source = xstrdup(p[0]); + new->u.copy.destination = xstrdup(p[1]); break; case 8: /* IGNORE */ /* FALLTROUGH */ @@ -734,12 +682,12 @@ static void process_config_line (const char *line, unsigned long *event_mask) new->action.what = i - 2; break; default: - msg ="WHAT in"; + msg = "WHAT in"; goto process_config_line_err; /*esac*/ } /* switch (i) */ - xregcomp( &new->preg, name, REG_EXTENDED); + xregcomp(&new->preg, name, REG_EXTENDED); *event_mask |= 1 << new->action.when; new->next = NULL; @@ -749,11 +697,12 @@ static void process_config_line (const char *line, unsigned long *event_mask) last_config->next = new; last_config = new; return; -process_config_line_err: - msg_logger_and_die(LOG_ERR, bb_msg_bad_config, msg , line); + + process_config_line_err: + msg_logger_and_die(LOG_ERR, bb_msg_bad_config, msg, line); } /* End Function process_config_line */ -static int do_servicing (int fd, unsigned long event_mask) +static int do_servicing(int fd, unsigned long event_mask) /* [SUMMARY] Service devfs changes until a signal is received. The open control file. The event mask. @@ -762,37 +711,31 @@ static int do_servicing (int fd, unsigned long event_mask) { ssize_t bytes; struct devfsd_notify_struct info; - unsigned long tmp_event_mask; - - debug_msg_logger(LOG_INFO, __FUNCTION__); - /* Tell devfs what events we care about */ - tmp_event_mask = event_mask; - do_ioctl_and_die(fd, DEVFSDIOC_SET_EVENT_MASK, tmp_event_mask); - while (!caught_signal) - { + /* (void*) cast is only in order to match prototype */ + xioctl(fd, DEVFSDIOC_SET_EVENT_MASK, (void*)event_mask); + while (!caught_signal) { errno = 0; - bytes = read (fd, (char *) &info, sizeof info); + bytes = read(fd, (char *) &info, sizeof info); if (caught_signal) break; /* Must test for this first */ if (errno == EINTR) continue; /* Yes, the order is important */ if (bytes < 1) break; - service_name (&info); + service_name(&info); } - if (caught_signal) - { + if (caught_signal) { int c_sighup = caught_sighup; caught_signal = FALSE; caught_sighup = FALSE; - return (c_sighup); + return c_sighup; } - msg_logger_and_die(LOG_ERR, "read error on control file"); + simple_msg_logger_and_die(LOG_ERR, "read error on control file"); } /* End Function do_servicing */ -static void service_name (const struct devfsd_notify_struct *info) +static void service_name(const struct devfsd_notify_struct *info) /* [SUMMARY] Service a single devfs change. The devfs change. [RETURNS] Nothing. @@ -802,43 +745,41 @@ static void service_name (const struct devfsd_notify_struct *info) regmatch_t mbuf[MAX_SUBEXPR]; struct config_entry_struct *entry; - debug_msg_logger(LOG_INFO, __FUNCTION__); if (ENABLE_DEBUG && info->overrun_count > 0) - debug_msg_logger(LOG_ERR, "lost %u events", info->overrun_count); + msg_logger(LOG_ERR, "lost %u events", info->overrun_count); /* Discard lookups on "/dev/log" and "/dev/initctl" */ - if( info->type == DEVFSD_NOTIFY_LOOKUP && - ((info->devname[0]=='l' && info->devname[1]=='o' && - info->devname[2]=='g' && !info->devname[3]) || - ( info->devname[0]=='i' && info->devname[1]=='n' && - info->devname[2]=='i' && info->devname[3]=='t' && - info->devname[4]=='c' && info->devname[5]=='t' && - info->devname[6]=='l' && !info->devname[7]))) - return; - for (entry = first_config; entry != NULL; entry = entry->next) - { + if (info->type == DEVFSD_NOTIFY_LOOKUP + && ((info->devname[0] == 'l' && info->devname[1] == 'o' + && info->devname[2] == 'g' && !info->devname[3]) + || (info->devname[0] == 'i' && info->devname[1] == 'n' + && info->devname[2] == 'i' && info->devname[3] == 't' + && info->devname[4] == 'c' && info->devname[5] == 't' + && info->devname[6] == 'l' && !info->devname[7])) + ) + return; + + for (entry = first_config; entry != NULL; entry = entry->next) { /* First check if action matches the type, then check if name matches */ - if (info->type != entry->action.when || regexec (&entry->preg, info->devname, MAX_SUBEXPR, mbuf, 0) != 0 ) + if (info->type != entry->action.when + || regexec(&entry->preg, info->devname, MAX_SUBEXPR, mbuf, 0) != 0) continue; - for (n = 0; (n < MAX_SUBEXPR) && (mbuf[n].rm_so != -1); ++n) + for (n = 0;(n < MAX_SUBEXPR) && (mbuf[n].rm_so != -1); ++n) /* VOID */; - debug_msg_logger(LOG_INFO, "%s: action.what %d", __FUNCTION__, entry->action.what); - - switch (entry->action.what) - { + switch (entry->action.what) { case AC_PERMISSIONS: - action_permissions (info, entry); + action_permissions(info, entry); break; case AC_MODLOAD: - if(ENABLE_DEVFSD_MODLOAD) - action_modload (info, entry); + if (ENABLE_DEVFSD_MODLOAD) + action_modload(info, entry); break; case AC_EXECUTE: - action_execute (info, entry, mbuf, n); + action_execute(info, entry, mbuf, n); break; case AC_COPY: - action_copy (info, entry, mbuf, n); + action_copy(info, entry, mbuf, n); break; case AC_IGNORE: return; @@ -847,15 +788,15 @@ static void service_name (const struct devfsd_notify_struct *info) case AC_MKNEWCOMPAT: case AC_RMOLDCOMPAT: case AC_RMNEWCOMPAT: - action_compat (info, entry->action.what); + action_compat(info, entry->action.what); break; default: - msg_logger_and_die(LOG_ERR, "Unknown action"); + simple_msg_logger_and_die(LOG_ERR, "Unknown action"); } } } /* End Function service_name */ -static void action_permissions (const struct devfsd_notify_struct *info, +static void action_permissions(const struct devfsd_notify_struct *info, const struct config_entry_struct *entry) /* [SUMMARY] Update permissions for a device entry. The devfs change. @@ -865,18 +806,15 @@ static void action_permissions (const struct devfsd_notify_struct *info, { struct stat statbuf; - debug_msg_logger(LOG_INFO, __FUNCTION__); - - if ( stat (info->devname, &statbuf) != 0 || - chmod (info->devname,(statbuf.st_mode & S_IFMT) | (entry->u.permissions.mode & ~S_IFMT)) != 0 || - chown (info->devname, entry->u.permissions.uid, entry->u.permissions.gid) != 0) - { - msg_logger(LOG_ERR, "Can't chmod or chown: %s: %m",info->devname); - } + if (stat(info->devname, &statbuf) != 0 + || chmod(info->devname, (statbuf.st_mode & S_IFMT) | (entry->u.permissions.mode & ~S_IFMT)) != 0 + || chown(info->devname, entry->u.permissions.uid, entry->u.permissions.gid) != 0 + ) + error_logger(LOG_ERR, "Can't chmod or chown: %s", info->devname); } /* End Function action_permissions */ -static void action_modload (const struct devfsd_notify_struct *info, - const struct config_entry_struct *entry ATTRIBUTE_UNUSED) +static void action_modload(const struct devfsd_notify_struct *info, + const struct config_entry_struct *entry UNUSED_PARAM) /* [SUMMARY] Load a module. The devfs change. The config file entry. @@ -884,27 +822,25 @@ static void action_modload (const struct devfsd_notify_struct *info, */ { char *argv[6]; - char device[STRING_LENGTH]; - argv[0] = MODPROBE; - argv[1] = MODPROBE_SWITCH_1; /* "-k" */ - argv[2] = MODPROBE_SWITCH_2; /* "-C" */ - argv[3] = CONFIG_MODULES_DEVFS; - argv[4] = device; + argv[0] = (char*)MODPROBE; + argv[1] = (char*)MODPROBE_SWITCH_1; /* "-k" */ + argv[2] = (char*)MODPROBE_SWITCH_2; /* "-C" */ + argv[3] = (char*)CONFIG_MODULES_DEVFS; + argv[4] = concat_path_file("/dev", info->devname); /* device */ argv[5] = NULL; - snprintf (device, sizeof (device), "/dev/%s", info->devname); - debug_msg_logger(LOG_INFO, "%s: %s %s %s %s %s",__FUNCTION__, argv[0],argv[1],argv[2],argv[3],argv[4]); - fork_and_execute(DIE, argv[0], argv); + spawn_and_wait(argv); + free(argv[4]); } /* End Function action_modload */ -static void action_execute (const struct devfsd_notify_struct *info, - const struct config_entry_struct *entry, - const regmatch_t *regexpr, unsigned int numexpr) +static void action_execute(const struct devfsd_notify_struct *info, + const struct config_entry_struct *entry, + const regmatch_t *regexpr, unsigned int numexpr) /* [SUMMARY] Execute a programme. The devfs change. The config file entry. - The number of subexpression (start, end) offsets within the + The number of subexpression(start, end) offsets within the device name. The number of elements within <>. [RETURNS] Nothing. @@ -915,30 +851,28 @@ static void action_execute (const struct devfsd_notify_struct *info, char *argv[MAX_ARGS + 1]; char largv[MAX_ARGS + 1][STRING_LENGTH]; - debug_msg_logger(LOG_INFO ,__FUNCTION__); gv_info.info = info; gv_info.devname = info->devname; - snprintf (gv_info.devpath, sizeof (gv_info.devpath), "%s/%s", mount_point, info->devname); - for (count = 0; entry->u.execute.argv[count] != NULL; ++count) - { - expand_expression (largv[count], STRING_LENGTH, + snprintf(gv_info.devpath, sizeof(gv_info.devpath), "%s/%s", mount_point, info->devname); + for (count = 0; entry->u.execute.argv[count] != NULL; ++count) { + expand_expression(largv[count], STRING_LENGTH, entry->u.execute.argv[count], get_variable, &gv_info, - gv_info.devname, regexpr, numexpr ); + gv_info.devname, regexpr, numexpr); argv[count] = largv[count]; } argv[count] = NULL; - fork_and_execute(NO_DIE, argv[0], argv); + spawn_and_wait(argv); } /* End Function action_execute */ -static void action_copy (const struct devfsd_notify_struct *info, - const struct config_entry_struct *entry, - const regmatch_t *regexpr, unsigned int numexpr) +static void action_copy(const struct devfsd_notify_struct *info, + const struct config_entry_struct *entry, + const regmatch_t *regexpr, unsigned int numexpr) /* [SUMMARY] Copy permissions. The devfs change. The config file entry. - This list of subexpression (start, end) offsets within the + This list of subexpression(start, end) offsets within the device name. The number of elements in <>. [RETURNS] Nothing. @@ -950,39 +884,36 @@ static void action_copy (const struct devfsd_notify_struct *info, char source[STRING_LENGTH], destination[STRING_LENGTH]; int ret = 0; - debug_msg_logger(LOG_INFO, __FUNCTION__); - dest_stat.st_mode = 0; - if ( (info->type == DEVFSD_NOTIFY_CHANGE) && S_ISLNK (info->mode) ) + if ((info->type == DEVFSD_NOTIFY_CHANGE) && S_ISLNK(info->mode)) return; gv_info.info = info; gv_info.devname = info->devname; - snprintf (gv_info.devpath, sizeof (gv_info.devpath), "%s/%s", mount_point, info->devname); - expand_expression (source, STRING_LENGTH, entry->u.copy.source, + snprintf(gv_info.devpath, sizeof(gv_info.devpath), "%s/%s", mount_point, info->devname); + expand_expression(source, STRING_LENGTH, entry->u.copy.source, get_variable, &gv_info, gv_info.devname, regexpr, numexpr); - expand_expression (destination, STRING_LENGTH, entry->u.copy.destination, + expand_expression(destination, STRING_LENGTH, entry->u.copy.destination, get_variable, &gv_info, gv_info.devname, regexpr, numexpr); - if ( !make_dir_tree (destination) || lstat (source, &source_stat) != 0) + if (!make_dir_tree(destination) || lstat(source, &source_stat) != 0) return; - lstat (destination, &dest_stat); + lstat(destination, &dest_stat); new_mode = source_stat.st_mode & ~S_ISVTX; if (info->type == DEVFSD_NOTIFY_CREATE) new_mode |= S_ISVTX; - else if ( (info->type == DEVFSD_NOTIFY_CHANGE) && (dest_stat.st_mode & S_ISVTX) ) + else if ((info->type == DEVFSD_NOTIFY_CHANGE) &&(dest_stat.st_mode & S_ISVTX)) new_mode |= S_ISVTX; - ret = copy_inode (destination, &dest_stat, new_mode, source, &source_stat); + ret = copy_inode(destination, &dest_stat, new_mode, source, &source_stat); if (ENABLE_DEBUG && ret && (errno != EEXIST)) - debug_msg_logger(LOG_ERR, "copy_inode: %s to %s: %m", source, destination); - return; + error_logger(LOG_ERR, "copy_inode: %s to %s", source, destination); } /* End Function action_copy */ -static void action_compat (const struct devfsd_notify_struct *info, unsigned int action) +static void action_compat(const struct devfsd_notify_struct *info, unsigned int action) /* [SUMMARY] Process a compatibility request. The devfs change. The action to take. @@ -992,7 +923,7 @@ static void action_compat (const struct devfsd_notify_struct *info, unsigned int int ret; const char *compat_name = NULL; const char *dest_name = info->devname; - char *ptr=NULL; + const char *ptr; char compat_buf[STRING_LENGTH], dest_buf[STRING_LENGTH]; int mode, host, bus, target, lun; unsigned int i; @@ -1013,99 +944,90 @@ static void action_compat (const struct devfsd_notify_struct *info, unsigned int }; /* First construct compatibility name */ - switch (action) - { + switch (action) { case AC_MKOLDCOMPAT: case AC_RMOLDCOMPAT: - compat_name = get_old_name (info->devname, info->namelen, compat_buf, info->major, info->minor); + compat_name = get_old_name(info->devname, info->namelen, compat_buf, info->major, info->minor); break; case AC_MKNEWCOMPAT: case AC_RMNEWCOMPAT: - ptr = strrchr (info->devname, '/') + 1; - i=scan_dev_name(info->devname, info->namelen, ptr); - - debug_msg_logger(LOG_INFO, "%s: scan_dev_name = %d", __FUNCTION__, i); + ptr = bb_basename(info->devname); + i = scan_dev_name(info->devname, info->namelen, ptr); /* nothing found */ - if(i==0 || i > 9) + if (i == 0 || i > 9) return; - sscanf (info->devname +((i<6)?5:4), "host%d/bus%d/target%d/lun%d/", &host, &bus, &target, &lun); - snprintf (dest_buf, sizeof (dest_buf), "../%s", info->devname + ((i>5)?4:0)); + sscanf(info->devname + ((i < 6) ? 5 : 4), "host%d/bus%d/target%d/lun%d/", &host, &bus, &target, &lun); + snprintf(dest_buf, sizeof(dest_buf), "../%s", info->devname + (( i > 5) ? 4 : 0)); dest_name = dest_buf; compat_name = compat_buf; /* 1 == scsi/generic 2 == scsi/disc 3 == scsi/cd 6 == ide/host/disc 7 == ide/host/cd */ - if( i == 1 || i == 2 || i == 3 || i == 6 || i ==7 ) - sprintf ( compat_buf, fmt[i], host, bus, target, lun); + if (i == 1 || i == 2 || i == 3 || i == 6 || i ==7) + sprintf(compat_buf, fmt[i], host, bus, target, lun); /* 4 == scsi/part 8 == ide/host/part */ - if( i == 4 || i == 8) - sprintf ( compat_buf, fmt[i], host, bus, target, lun, atoi (ptr + 4) ); + if (i == 4 || i == 8) + sprintf(compat_buf, fmt[i], host, bus, target, lun, atoi(ptr + 4)); /* 5 == scsi/mt */ - if( i == 5) - { + if (i == 5) { rewind_ = info->devname[info->namelen - 1]; if (rewind_ != 'n') rewind_ = '\0'; mode=0; - if(ptr[2] == 'l' /*108*/ || ptr[2] == 'm'/*109*/) + if (ptr[2] == 'l' /*108*/ || ptr[2] == 'm'/*109*/) mode = ptr[2] - 107; /* 1 or 2 */ - if(ptr[2] == 'a') + if (ptr[2] == 'a') mode = 3; - sprintf (compat_buf, fmt [i], host, bus, target, lun, mode, rewind_); + sprintf(compat_buf, fmt[i], host, bus, target, lun, mode, rewind_); } /* 9 == ide/host/mt */ - if( i == 9 ) - snprintf (compat_buf, sizeof (compat_buf), fmt[i], host, bus, target, lun, ptr + 2); + if (i == 9) + snprintf(compat_buf, sizeof(compat_buf), fmt[i], host, bus, target, lun, ptr + 2); /* esac */ - } /* switch(action) */ + } /* switch (action) */ - if(compat_name == NULL ) + if (compat_name == NULL) return; - debug_msg_logger( LOG_INFO, "%s: %s", __FUNCTION__, compat_name); - /* Now decide what to do with it */ - switch (action) - { + switch (action) { case AC_MKOLDCOMPAT: case AC_MKNEWCOMPAT: - mksymlink (dest_name, compat_name); + mksymlink(dest_name, compat_name); break; case AC_RMOLDCOMPAT: case AC_RMNEWCOMPAT: - ret = unlink (compat_name); + ret = unlink(compat_name); if (ENABLE_DEBUG && ret) - debug_msg_logger(LOG_ERR, "unlink: %s: %m", compat_name); + error_logger(LOG_ERR, "unlink: %s", compat_name); break; /*esac*/ - } /* switch(action) */ + } /* switch (action) */ } /* End Function action_compat */ static void restore(char *spath, struct stat source_stat, int rootlen) { - char dpath[STRING_LENGTH]; + char *dpath; struct stat dest_stat; - debug_msg_logger(LOG_INFO, __FUNCTION__); - dest_stat.st_mode = 0; - snprintf (dpath, sizeof dpath, "%s%s", mount_point, spath + rootlen); - lstat (dpath, &dest_stat); - - if ( S_ISLNK (source_stat.st_mode) || (source_stat.st_mode & S_ISVTX) ) - copy_inode (dpath, &dest_stat, (source_stat.st_mode & ~S_ISVTX) , spath, &source_stat); - - if ( S_ISDIR (source_stat.st_mode) ) - dir_operation(RESTORE, spath, rootlen,NULL); + dpath = concat_path_file(mount_point, spath + rootlen); + lstat(dpath, &dest_stat); + free(dpath); + if (S_ISLNK(source_stat.st_mode) || (source_stat.st_mode & S_ISVTX)) + copy_inode(dpath, &dest_stat, (source_stat.st_mode & ~S_ISVTX), spath, &source_stat); + + if (S_ISDIR(source_stat.st_mode)) + dir_operation(RESTORE, spath, rootlen, NULL); } -static int copy_inode (const char *destpath, const struct stat *dest_stat, +static int copy_inode(const char *destpath, const struct stat *dest_stat, mode_t new_mode, const char *sourcepath, const struct stat *source_stat) /* [SUMMARY] Copy an inode. @@ -1123,75 +1045,74 @@ static int copy_inode (const char *destpath, const struct stat *dest_stat, struct sockaddr_un un_addr; char symlink_val[STRING_LENGTH]; - debug_msg_logger(LOG_INFO, __FUNCTION__); - - if ( (source_stat->st_mode & S_IFMT) == (dest_stat->st_mode & S_IFMT) ) - { + if ((source_stat->st_mode & S_IFMT) ==(dest_stat->st_mode & S_IFMT)) { /* Same type */ - if ( S_ISLNK (source_stat->st_mode) ) - { - if (( source_len = readlink (sourcepath, source_link, STRING_LENGTH - 1) ) < 0 || - ( dest_len = readlink (destpath , dest_link , STRING_LENGTH - 1) ) < 0 ) - return (FALSE); + if (S_ISLNK(source_stat->st_mode)) { + source_len = readlink(sourcepath, source_link, STRING_LENGTH - 1); + if ((source_len < 0) + || (dest_len = readlink(destpath, dest_link, STRING_LENGTH - 1)) < 0 + ) + return FALSE; source_link[source_len] = '\0'; dest_link[dest_len] = '\0'; - if ( (source_len != dest_len) || (strcmp (source_link, dest_link) != 0) ) - { - unlink (destpath); - symlink (source_link, destpath); + if ((source_len != dest_len) || (strcmp(source_link, dest_link) != 0)) { + unlink(destpath); + symlink(source_link, destpath); } - return (TRUE); + return TRUE; } /* Else not a symlink */ - chmod (destpath, new_mode & ~S_IFMT); - chown (destpath, source_stat->st_uid, source_stat->st_gid); - return (TRUE); + chmod(destpath, new_mode & ~S_IFMT); + chown(destpath, source_stat->st_uid, source_stat->st_gid); + return TRUE; } /* Different types: unlink and create */ - unlink (destpath); - switch (source_stat->st_mode & S_IFMT) - { + unlink(destpath); + switch (source_stat->st_mode & S_IFMT) { case S_IFSOCK: - if ( ( fd = socket (AF_UNIX, SOCK_STREAM, 0) ) < 0 ) + fd = socket(AF_UNIX, SOCK_STREAM, 0); + if (fd < 0) break; un_addr.sun_family = AF_UNIX; - snprintf (un_addr.sun_path, sizeof (un_addr.sun_path), "%s", destpath); - val = bind (fd, (struct sockaddr *) &un_addr, (int) sizeof un_addr); - close (fd); - if (val != 0 || chmod (destpath, new_mode & ~S_IFMT) != 0) + snprintf(un_addr.sun_path, sizeof(un_addr.sun_path), "%s", destpath); + val = bind(fd, (struct sockaddr *) &un_addr, (int) sizeof un_addr); + close(fd); + if (val != 0 || chmod(destpath, new_mode & ~S_IFMT) != 0) break; goto do_chown; case S_IFLNK: - if ( ( val = readlink (sourcepath, symlink_val, STRING_LENGTH - 1) ) < 0 ) + val = readlink(sourcepath, symlink_val, STRING_LENGTH - 1); + if (val < 0) break; symlink_val[val] = '\0'; - if (symlink (symlink_val, destpath) == 0) - return (TRUE); + if (symlink(symlink_val, destpath) == 0) + return TRUE; break; case S_IFREG: - if ( ( fd = open (destpath, O_RDONLY | O_CREAT, new_mode & ~S_IFMT) ) < 0 ) + fd = open(destpath, O_RDONLY | O_CREAT, new_mode & ~S_IFMT); + if (fd < 0) break; - close (fd); - if (chmod (destpath, new_mode & ~S_IFMT) != 0) + close(fd); + if (chmod(destpath, new_mode & ~S_IFMT) != 0) break; goto do_chown; case S_IFBLK: case S_IFCHR: case S_IFIFO: - if (mknod (destpath, new_mode, source_stat->st_rdev) != 0) + if (mknod(destpath, new_mode, source_stat->st_rdev) != 0) break; goto do_chown; case S_IFDIR: - if (mkdir (destpath, new_mode & ~S_IFMT) != 0) + if (mkdir(destpath, new_mode & ~S_IFMT) != 0) break; do_chown: - if (chown (destpath, source_stat->st_uid, source_stat->st_gid) == 0) - return (TRUE); + if (chown(destpath, source_stat->st_uid, source_stat->st_gid) == 0) + return TRUE; /*break;*/ } - return (FALSE); + return FALSE; } /* End Function copy_inode */ -static void free_config (void) +static void free_config(void) /* [SUMMARY] Free the configuration information. [RETURNS] Nothing. */ @@ -1199,30 +1120,25 @@ static void free_config (void) struct config_entry_struct *c_entry; void *next; - debug_msg_logger(LOG_INFO, __FUNCTION__); - - for (c_entry = first_config; c_entry != NULL; c_entry = next) - { + for (c_entry = first_config; c_entry != NULL; c_entry = next) { unsigned int count; next = c_entry->next; - regfree (&c_entry->preg); - if (c_entry->action.what == AC_EXECUTE) - { - for (count = 0; count < MAX_ARGS; ++count) - { + regfree(&c_entry->preg); + if (c_entry->action.what == AC_EXECUTE) { + for (count = 0; count < MAX_ARGS; ++count) { if (c_entry->u.execute.argv[count] == NULL) break; - free (c_entry->u.execute.argv[count]); + free(c_entry->u.execute.argv[count]); } } - free (c_entry); + free(c_entry); } first_config = NULL; last_config = NULL; } /* End Function free_config */ -static int get_uid_gid (int flag, const char *string) +static int get_uid_gid(int flag, const char *string) /* [SUMMARY] Convert a string to a UID or GID value. "UID" or "GID". The string. @@ -1231,33 +1147,30 @@ static int get_uid_gid (int flag, const char *string) { struct passwd *pw_ent; struct group *grp_ent; - static char *msg; + const char *msg; - if (ENABLE_DEVFSD_VERBOSE) - msg="user"; - - debug_msg_logger(LOG_INFO, __FUNCTION__); - - if(ENABLE_DEBUG && flag != UID && flag != GID) - msg_logger_and_die(LOG_ERR,"%s: flag != UID && flag != GID", __FUNCTION__); + if (isdigit(string[0]) || ((string[0] == '-') && isdigit(string[1]))) + return atoi(string); - if ( isdigit (string[0]) || ( (string[0] == '-') && isdigit (string[1]) ) ) - return atoi (string); + if (flag == UID && (pw_ent = getpwnam(string)) != NULL) + return pw_ent->pw_uid; - if ( flag == UID && ( pw_ent = getpwnam (string) ) != NULL ) - return (pw_ent->pw_uid); + if (ENABLE_DEVFSD_VERBOSE) + msg = "user"; - if ( flag == GID && ( grp_ent = getgrnam (string) ) != NULL ) - return (grp_ent->gr_gid); - else if(ENABLE_DEVFSD_VERBOSE) - msg="group"; + if (flag == GID) { + if ((grp_ent = getgrnam(string)) != NULL) + return grp_ent->gr_gid; + if (ENABLE_DEVFSD_VERBOSE) + msg = "group"; + } - if(ENABLE_DEVFSD_VERBOSE) - msg_logger(LOG_ERR,"unknown %s: %s, defaulting to %cid=0", msg, string, msg[0]); - return (0); + if (ENABLE_DEVFSD_VERBOSE) + msg_logger(LOG_ERR, "unknown %s: %s, defaulting to %cid=0", msg, string, msg[0]); + return 0; }/* End Function get_uid_gid */ -static mode_t get_mode (const char *string) +static mode_t get_mode(const char *string) /* [SUMMARY] Convert a string to a mode value. The string. [RETURNS] The mode value. @@ -1266,94 +1179,78 @@ static mode_t get_mode (const char *string) mode_t mode; int i; - debug_msg_logger(LOG_INFO, __FUNCTION__); - - if ( isdigit (string[0]) ) - return strtoul (string, NULL, 8); - if (strlen (string) != 9) + if (isdigit(string[0])) + return strtoul(string, NULL, 8); + if (strlen(string) != 9) msg_logger_and_die(LOG_ERR, "bad mode: %s", string); mode = 0; - i= S_IRUSR; - while(i>0) - { - if(string[0]=='r'||string[0]=='w'||string[0]=='x') - mode+=i; - i=i/2; + i = S_IRUSR; + while (i > 0) { + if (string[0] == 'r' || string[0] == 'w' || string[0] == 'x') + mode += i; + i = i / 2; string++; } - return (mode); + return mode; } /* End Function get_mode */ -static void signal_handler (int sig) +static void signal_handler(int sig) { - debug_msg_logger(LOG_INFO, __FUNCTION__); - caught_signal = TRUE; if (sig == SIGHUP) caught_sighup = TRUE; - msg_logger(LOG_INFO, "Caught signal %d", sig); + info_logger(LOG_INFO, "Caught signal %d", sig); } /* End Function signal_handler */ -static const char *get_variable (const char *variable, void *info) +static const char *get_variable(const char *variable, void *info) { + static char *hostname; + struct get_variable_info *gv_info = info; - static char hostname[STRING_LENGTH], sbuf[STRING_LENGTH]; - const char *field_names[] = { "hostname", "mntpt", "devpath", "devname", - "uid", "gid", "mode", hostname, mount_point, - gv_info->devpath, gv_info->devname, 0 }; + const char *field_names[] = { + "hostname", "mntpt", "devpath", "devname", "uid", "gid", "mode", + NULL, mount_point, gv_info->devpath, gv_info->devname, NULL + }; int i; - debug_msg_logger(LOG_INFO, __FUNCTION__); - - if (gethostname (hostname, STRING_LENGTH - 1) != 0) - msg_logger_and_die(LOG_ERR, "gethostname: %m"); - - /* Here on error we should do exit(RV_SYS_ERROR), instead we do exit(EXIT_FAILURE) */ - hostname[STRING_LENGTH - 1] = '\0'; + if (!hostname) + hostname = safe_gethostname(); + field_names[7] = hostname; /* index_in_str_array returns i>=0 */ - i=index_in_str_array(field_names, variable); - - if ( i > 6 || i < 0 || (i > 1 && gv_info == NULL)) - return (NULL); - if( i >= 0 && i <= 3) - { - debug_msg_logger(LOG_INFO, "%s: i=%d %s", __FUNCTION__, i ,field_names[i+7]); - return(field_names[i+7]); - } + i = index_in_str_array(field_names, variable); - if(i == 4 ) - sprintf (sbuf, "%u", gv_info->info->uid); - else if(i == 5) - sprintf (sbuf, "%u", gv_info->info->gid); - else if(i == 6) - sprintf (sbuf, "%o", gv_info->info->mode); - - debug_msg_logger(LOG_INFO, "%s: %s", __FUNCTION__, sbuf); - - return (sbuf); + if (i > 6 || i < 0 || (i > 1 && gv_info == NULL)) + return NULL; + if (i >= 0 && i <= 3) + return field_names[i + 7]; + + if (i == 4) + return auto_string(xasprintf("%u", gv_info->info->uid)); + if (i == 5) + return auto_string(xasprintf("%u", gv_info->info->gid)); + /* i == 6 */ + return auto_string(xasprintf("%o", gv_info->info->mode)); } /* End Function get_variable */ static void service(struct stat statbuf, char *path) { struct devfsd_notify_struct info; - debug_msg_logger(LOG_INFO, __FUNCTION__); - - memset (&info, 0, sizeof info); + memset(&info, 0, sizeof info); info.type = DEVFSD_NOTIFY_REGISTERED; info.mode = statbuf.st_mode; - info.major = major (statbuf.st_rdev); - info.minor = minor (statbuf.st_rdev); + info.major = major(statbuf.st_rdev); + info.minor = minor(statbuf.st_rdev); info.uid = statbuf.st_uid; info.gid = statbuf.st_gid; - snprintf (info.devname, sizeof (info.devname), "%s", path + strlen (mount_point) + 1); - info.namelen = strlen (info.devname); - service_name (&info); - if ( S_ISDIR (statbuf.st_mode) ) - dir_operation(SERVICE,path,0,NULL); + snprintf(info.devname, sizeof(info.devname), "%s", path + strlen(mount_point) + 1); + info.namelen = strlen(info.devname); + service_name(&info); + if (S_ISDIR(statbuf.st_mode)) + dir_operation(SERVICE, path, 0, NULL); } static void dir_operation(int type, const char * dir_name, int var, unsigned long *event_mask) @@ -1368,91 +1265,70 @@ static void dir_operation(int type, const char * dir_name, int var, unsigned lon struct stat statbuf; DIR *dp; struct dirent *de; - char path[STRING_LENGTH]; - - debug_msg_logger(LOG_INFO, __FUNCTION__); + char *path; - if((dp = opendir( dir_name))==NULL) - { - debug_msg_logger(LOG_ERR, "opendir: %s: %m", dir_name); + dp = warn_opendir(dir_name); + if (dp == NULL) return; - } - while ( (de = readdir (dp) ) != NULL ) - { + while ((de = readdir(dp)) != NULL) { - if(de->d_name && *de->d_name == '.' && (!de->d_name[1] || (de->d_name[1] == '.' && !de->d_name[2]))) + if (de->d_name && DOT_OR_DOTDOT(de->d_name)) continue; - snprintf (path, sizeof (path), "%s/%s", dir_name, de->d_name); - debug_msg_logger(LOG_ERR, "%s: %s", __FUNCTION__, path); - - if (lstat (path, &statbuf) != 0) - { - debug_msg_logger(LOG_ERR, "%s: %s: %m", __FUNCTION__, path); - continue; - } - switch(type) - { - case SERVICE: - service(statbuf,path); - break; - case RESTORE: - restore(path, statbuf, var); - break; - case READ_CONFIG: - read_config_file (path, var, event_mask); - break; + path = concat_path_file(dir_name, de->d_name); + if (lstat(path, &statbuf) == 0) { + switch (type) { + case SERVICE: + service(statbuf, path); + break; + case RESTORE: + restore(path, statbuf, var); + break; + case READ_CONFIG: + read_config_file(path, var, event_mask); + break; + } } + free(path); } - closedir (dp); + closedir(dp); } /* End Function do_scan_and_service */ -static int mksymlink (const char *oldpath, const char *newpath) +static int mksymlink(const char *oldpath, const char *newpath) /* [SUMMARY] Create a symlink, creating intervening directories as required. The string contained in the symlink. The name of the new symlink. [RETURNS] 0 on success, else -1. */ { - debug_msg_logger(LOG_INFO, __FUNCTION__); - - if ( !make_dir_tree (newpath) ) - return (-1); + if (!make_dir_tree(newpath)) + return -1; - if (symlink (oldpath, newpath) != 0) - { + if (symlink(oldpath, newpath) != 0) { if (errno != EEXIST) - { - debug_msg_logger(LOG_ERR, "%s: %s to %s: %m", __FUNCTION__, oldpath, newpath); - return (-1); - } + return -1; } - return (0); + return 0; } /* End Function mksymlink */ -static int make_dir_tree (const char *path) +static int make_dir_tree(const char *path) /* [SUMMARY] Creating intervening directories for a path as required. - The full pathname (including the leaf node). + The full pathname(including the leaf node). [RETURNS] TRUE on success, else FALSE. */ { - debug_msg_logger(LOG_INFO, __FUNCTION__); - - if (bb_make_directory( dirname((char *)path), -1, FILEUTILS_RECUR )==-1) - { - debug_msg_logger(LOG_ERR, "%s: %s: %m",__FUNCTION__, path); - return (FALSE); - } - return(TRUE); + if (bb_make_directory(dirname((char *)path), -1, FILEUTILS_RECUR) == -1) + return FALSE; + return TRUE; } /* End Function make_dir_tree */ static int expand_expression(char *output, unsigned int outsize, - const char *input, - const char *(*get_variable_func)(const char *variable, void *info), - void *info, - const char *devname, - const regmatch_t *ex, unsigned int numexp) + const char *input, + const char *(*get_variable_func)(const char *variable, void *info), + void *info, + const char *devname, + const regmatch_t *ex, unsigned int numexp) /* [SUMMARY] Expand environment variables and regular subexpressions in string. The output expanded expression is written here. The size of the output buffer. @@ -1470,17 +1346,15 @@ static int expand_expression(char *output, unsigned int outsize, { char temp[STRING_LENGTH]; - debug_msg_logger(LOG_INFO, __FUNCTION__); - - if ( !st_expr_expand (temp, STRING_LENGTH, input, get_variable_func, info) ) - return (FALSE); - expand_regexp (output, outsize, temp, devname, ex, numexp); - return (TRUE); + if (!st_expr_expand(temp, STRING_LENGTH, input, get_variable_func, info)) + return FALSE; + expand_regexp(output, outsize, temp, devname, ex, numexp); + return TRUE; } /* End Function expand_expression */ -static void expand_regexp (char *output, size_t outsize, const char *input, - const char *devname, - const regmatch_t *ex, unsigned int numex ) +static void expand_regexp(char *output, size_t outsize, const char *input, + const char *devname, + const regmatch_t *ex, unsigned int numex) /* [SUMMARY] Expand all occurrences of the regular subexpressions \0 to \9. The output expanded expression is written here. The size of the output buffer. @@ -1499,8 +1373,6 @@ static void expand_regexp (char *output, size_t outsize, const char *input, const char last_exp = '0' - 1 + numex; int c = -1; - debug_msg_logger(LOG_INFO, __FUNCTION__); - /* Guarantee NULL termination by writing an explicit '\0' character into the very last byte */ if (outsize) @@ -1508,25 +1380,21 @@ static void expand_regexp (char *output, size_t outsize, const char *input, /* Copy the input string into the output buffer, replacing '\\' with '\' and '\0' .. '\9' with subexpressions 0 .. 9, if they exist. Other \x codes are deleted */ - while ( (c != '\0') && (outsize != 0) ) - { + while ((c != '\0') && (outsize != 0)) { c = *input; ++input; - if (c == '\\') - { + if (c == '\\') { c = *input; ++input; - if (c != '\\') - { - if ((c >= '0') && (c <= last_exp)) - { + if (c != '\\') { + if ((c >= '0') && (c <= last_exp)) { const regmatch_t *subexp = ex + (c - '0'); unsigned int sublen = subexp->rm_eo - subexp->rm_so; /* Range checking */ if (sublen > outsize) sublen = outsize; - strncpy (output, devname + subexp->rm_so, sublen); + strncpy(output, devname + subexp->rm_so, sublen); output += sublen; outsize -= sublen; } @@ -1542,11 +1410,10 @@ static void expand_regexp (char *output, size_t outsize, const char *input, /* from compat_name.c */ -struct translate_struct -{ - char *match; /* The string to match to (up to length) */ - char *format; /* Format of output, "%s" takes data past match string, - NULL is effectively "%s" (just more efficient) */ +struct translate_struct { + const char *match; /* The string to match to(up to length) */ + const char *format; /* Format of output, "%s" takes data past match string, + NULL is effectively "%s"(just more efficient) */ }; static struct translate_struct translate_table[] = @@ -1582,8 +1449,8 @@ static struct translate_struct translate_table[] = {NULL, NULL} }; -const char *get_old_name (const char *devname, unsigned int namelen, - char *buffer, unsigned int major, unsigned int minor) +const char *get_old_name(const char *devname, unsigned int namelen, + char *buffer, unsigned int major, unsigned int minor) /* [SUMMARY] Translate a kernel-supplied name into an old name. The device name provided by the kernel. The length of the name. @@ -1594,14 +1461,13 @@ const char *get_old_name (const char *devname, unsigned int namelen, */ { const char *compat_name = NULL; - char *ptr; + const char *ptr; struct translate_struct *trans; unsigned int i; char mode; int indexx; const char *pty1; const char *pty2; - size_t len; /* 1 to 5 "scsi/" , 6 to 9 "ide/host", 10 sbp/, 11 vcc/, 12 pty/ */ static const char *const fmt[] = { NULL , @@ -1620,118 +1486,101 @@ const char *get_old_name (const char *devname, unsigned int namelen, NULL }; - debug_msg_logger(LOG_INFO, __FUNCTION__); - - for (trans = translate_table; trans->match != NULL; ++trans) - { - len = strlen (trans->match); - - if (strncmp (devname, trans->match, len) == 0) - { + for (trans = translate_table; trans->match != NULL; ++trans) { + char *after_match = is_prefixed_with(devname, trans->match); + if (after_match) { if (trans->format == NULL) - return (devname + len); - sprintf (buffer, trans->format, devname + len); - return (buffer); + return after_match; + sprintf(buffer, trans->format, after_match); + return buffer; } } - ptr = (strrchr (devname, '/') + 1); + ptr = bb_basename(devname); i = scan_dev_name(devname, namelen, ptr); - if( i > 0 && i < 13) + if (i > 0 && i < 13) compat_name = buffer; else return NULL; - debug_msg_logger(LOG_INFO, "%s: scan_dev_name = %d", __FUNCTION__, i); - /* 1 == scsi/generic, 3 == scsi/cd, 10 == sbp/ */ - if( i == 1 || i == 3 || i == 10 ) - sprintf (buffer, fmt[i], minor); + if (i == 1 || i == 3 || i == 10) + sprintf(buffer, fmt[i], minor); /* 2 ==scsi/disc, 4 == scsi/part */ - if( i == 2 || i == 4) - compat_name = write_old_sd_name (buffer, major, minor,((i == 2)?"":(ptr + 4))); + if (i == 2 || i == 4) + compat_name = write_old_sd_name(buffer, major, minor, ((i == 2) ? "" : (ptr + 4))); /* 5 == scsi/mt */ - if( i == 5) - { + if (i == 5) { mode = ptr[2]; if (mode == 'n') mode = '\0'; - sprintf (buffer, fmt[i], minor & 0x1f, mode); + sprintf(buffer, fmt[i], minor & 0x1f, mode); if (devname[namelen - 1] != 'n') ++compat_name; } /* 6 == ide/host/disc, 7 == ide/host/cd, 8 == ide/host/part */ - if( i == 6 || i == 7 || i == 8 ) + if (i == 6 || i == 7 || i == 8) /* last arg should be ignored for i == 6 or i== 7 */ - sprintf (buffer, fmt[i] , get_old_ide_name (major, minor), ptr + 4); + sprintf(buffer, fmt[i] , get_old_ide_name(major, minor), ptr + 4); /* 9 == ide/host/mt */ - if( i == 9 ) - sprintf (buffer, fmt[i], ptr + 2, minor & 0x7f); + if (i == 9) + sprintf(buffer, fmt[i], ptr + 2, minor & 0x7f); /* 11 == vcc/ */ - if( i == 11 ) - { - sprintf (buffer, fmt[i], devname + 4); + if (i == 11) { + sprintf(buffer, fmt[i], devname + 4); if (buffer[3] == '0') buffer[3] = '\0'; } /* 12 == pty/ */ - if( i == 12 ) - { + if (i == 12) { pty1 = "pqrstuvwxyzabcde"; pty2 = "0123456789abcdef"; - indexx = atoi (devname + 5); - sprintf (buffer, fmt[i], (devname[4] == 'm') ? 'p' : 't', pty1[indexx >> 4], pty2[indexx & 0x0f]); + indexx = atoi(devname + 5); + sprintf(buffer, fmt[i], (devname[4] == 'm') ? 'p' : 't', pty1[indexx >> 4], pty2[indexx & 0x0f]); } - - if(ENABLE_DEBUG && compat_name!=NULL) - msg_logger(LOG_INFO, "%s: compat_name %s", __FUNCTION__, compat_name); - - return (compat_name); + return compat_name; } /* End Function get_old_name */ -static char get_old_ide_name (unsigned int major, unsigned int minor) +static char get_old_ide_name(unsigned int major, unsigned int minor) /* [SUMMARY] Get the old IDE name for a device. The major number for the device. The minor number for the device. [RETURNS] The drive letter. */ { - char letter='y'; /* 121 */ - char c='a'; /* 97 */ - int i=IDE0_MAJOR; - - debug_msg_logger(LOG_INFO, __FUNCTION__); + char letter = 'y'; /* 121 */ + char c = 'a'; /* 97 */ + int i = IDE0_MAJOR; /* I hope it works like the previous code as it saves a few bytes. Tito ;P */ do { - if( i==IDE0_MAJOR || i==IDE1_MAJOR || i==IDE2_MAJOR || - i==IDE3_MAJOR || i==IDE4_MAJOR || i==IDE5_MAJOR || - i==IDE6_MAJOR || i==IDE7_MAJOR || i==IDE8_MAJOR || - i==IDE9_MAJOR ) - { - if((unsigned int)i==major) - { - letter=c; + if (i == IDE0_MAJOR || i == IDE1_MAJOR || i == IDE2_MAJOR + || i == IDE3_MAJOR || i == IDE4_MAJOR || i == IDE5_MAJOR + || i == IDE6_MAJOR || i == IDE7_MAJOR || i == IDE8_MAJOR + || i == IDE9_MAJOR + ) { + if ((unsigned int)i == major) { + letter = c; break; } - c+=2; + c += 2; } i++; - } while(i<=IDE9_MAJOR); + } while (i <= IDE9_MAJOR); if (minor > 63) ++letter; - return (letter); + return letter; } /* End Function get_old_ide_name */ -static char *write_old_sd_name (char *buffer, +static char *write_old_sd_name(char *buffer, unsigned int major, unsigned int minor, - char *part) + const char *part) /* [SUMMARY] Write the old SCSI disc name to a buffer. The buffer to write to. The major number for the device. @@ -1742,23 +1591,19 @@ static char *write_old_sd_name (char *buffer, { unsigned int disc_index; - debug_msg_logger(LOG_INFO, __FUNCTION__); - - if (major == 8) - { - sprintf (buffer, "sd%c%s", 'a' + (minor >> 4), part); - return (buffer); + if (major == 8) { + sprintf(buffer, "sd%c%s", 'a' + (minor >> 4), part); + return buffer; } - if ( (major > 64) && (major < 72) ) - { - disc_index = ( (major - 64) << 4 ) + (minor >> 4); + if ((major > 64) && (major < 72)) { + disc_index = ((major - 64) << 4) +(minor >> 4); if (disc_index < 26) - sprintf (buffer, "sd%c%s", 'a' + disc_index, part); + sprintf(buffer, "sd%c%s", 'a' + disc_index, part); else - sprintf (buffer, "sd%c%c%s", 'a' + (disc_index / 26) - 1, 'a' + disc_index % 26,part); - return (buffer); + sprintf(buffer, "sd%c%c%s", 'a' +(disc_index / 26) - 1, 'a' + disc_index % 26, part); + return buffer; } - return (NULL); + return NULL; } /* End Function write_old_sd_name */ @@ -1766,10 +1611,10 @@ static char *write_old_sd_name (char *buffer, /*EXPERIMENTAL_FUNCTION*/ -int st_expr_expand (char *output, unsigned int length, const char *input, - const char *(*get_variable_func) (const char *variable, - void *info), - void *info) +int st_expr_expand(char *output, unsigned int length, const char *input, + const char *(*get_variable_func)(const char *variable, + void *info), + void *info) /* [SUMMARY] Expand an expression using Borne Shell-like unquoted rules. The output expanded expression is written here. The size of the output buffer. @@ -1789,55 +1634,50 @@ int st_expr_expand (char *output, unsigned int length, const char *input, struct passwd *pwent; char buffer[BUFFER_SIZE], tmp[STRING_LENGTH]; - debug_msg_logger(LOG_INFO, __FUNCTION__); - if (length > BUFFER_SIZE) length = BUFFER_SIZE; - for (; TRUE; ++input) - { - switch (ch = *input) - { + for (; TRUE; ++input) { + switch (ch = *input) { case '$': /* Variable expansion */ - input = expand_variable (buffer, length, &out_pos, ++input, get_variable_func, info); + input = expand_variable(buffer, length, &out_pos, ++input, get_variable_func, info); if (input == NULL) - return (FALSE); + return FALSE; break; case '~': /* Home directory expansion */ ch = input[1]; - if ( isspace (ch) || (ch == '/') || (ch == '\0') ) - { + if (isspace(ch) ||(ch == '/') ||(ch == '\0')) { /* User's own home directory: leave separator for next time */ - if ( ( env = getenv ("HOME") ) == NULL ) - { - msg_logger(LOG_INFO, bb_msg_variable_not_found, "HOME"); - return (FALSE); + env = getenv("HOME"); + if (env == NULL) { + info_logger(LOG_INFO, bb_msg_variable_not_found, "HOME"); + return FALSE; } - len = strlen (env); + len = strlen(env); if (len + out_pos >= length) goto st_expr_expand_out; - memcpy (buffer + out_pos, env, len + 1); + memcpy(buffer + out_pos, env, len + 1); out_pos += len; continue; } /* Someone else's home directory */ - for (ptr = ++input; !isspace (ch) && (ch != '/') && (ch != '\0'); ch = *++ptr) - /* VOID */ ; + for (ptr = ++input; !isspace(ch) && (ch != '/') && (ch != '\0'); ch = *++ptr) + /* VOID */; len = ptr - input; if (len >= sizeof tmp) goto st_expr_expand_out; - safe_memcpy (tmp, input, len); + safe_memcpy(tmp, input, len); input = ptr - 1; - if ( ( pwent = getpwnam (tmp) ) == NULL ) - { - msg_logger(LOG_INFO, "no pwent for: %s", tmp); - return (FALSE); + pwent = getpwnam(tmp); + if (pwent == NULL) { + info_logger(LOG_INFO, "no pwent for: %s", tmp); + return FALSE; } - len = strlen (pwent->pw_dir); + len = strlen(pwent->pw_dir); if (len + out_pos >= length) goto st_expr_expand_out; - memcpy (buffer + out_pos, pwent->pw_dir, len + 1); + memcpy(buffer + out_pos, pwent->pw_dir, len + 1); out_pos += len; break; case '\0': @@ -1846,29 +1686,28 @@ int st_expr_expand (char *output, unsigned int length, const char *input, if (out_pos >= length) goto st_expr_expand_out; buffer[out_pos++] = ch; - if (ch == '\0') - { - memcpy (output, buffer, out_pos); - return (TRUE); + if (ch == '\0') { + memcpy(output, buffer, out_pos); + return TRUE; } break; /* esac */ } } - return (FALSE); + return FALSE; st_expr_expand_out: - msg_logger(LOG_INFO, bb_msg_small_buffer); - return (FALSE); + simple_info_logger(LOG_INFO, bb_msg_small_buffer); + return FALSE; } /* End Function st_expr_expand */ /* Private functions follow */ -static const char *expand_variable (char *buffer, unsigned int length, - unsigned int *out_pos, const char *input, - const char *(*func) (const char *variable, - void *info), - void *info) +static const char *expand_variable(char *buffer, unsigned int length, + unsigned int *out_pos, const char *input, + const char *(*func)(const char *variable, + void *info), + void *info) /* [SUMMARY] Expand a variable. The buffer to write to. The length of the output buffer. @@ -1888,83 +1727,75 @@ static const char *expand_variable (char *buffer, unsigned int length, const char *env, *ptr; char tmp[STRING_LENGTH]; - debug_msg_logger(LOG_INFO, __FUNCTION__); - ch = input[0]; - if (ch == '$') - { + if (ch == '$') { /* Special case for "$$": PID */ - sprintf ( tmp, "%d", (int) getpid () ); - len = strlen (tmp); + sprintf(tmp, "%d", (int) getpid()); + len = strlen(tmp); if (len + *out_pos >= length) goto expand_variable_out; - memcpy (buffer + *out_pos, tmp, len + 1); + memcpy(buffer + *out_pos, tmp, len + 1); out_pos += len; - return (input); + return input; } /* Ordinary variable expansion, possibly in braces */ - if (ch != '{') - { + if (ch != '{') { /* Simple variable expansion */ - for (ptr = input; isalnum (ch) || (ch == '_') || (ch == ':');ch = *++ptr) - /* VOID */ ; + for (ptr = input; isalnum(ch) || (ch == '_') || (ch == ':'); ch = *++ptr) + /* VOID */; len = ptr - input; if ((size_t)len >= sizeof tmp) goto expand_variable_out; - safe_memcpy (tmp, input, len); + safe_memcpy(tmp, input, len); input = ptr - 1; - if ( ( env = get_variable_v2 (tmp, func, info) ) == NULL ) - { - msg_logger(LOG_INFO, bb_msg_variable_not_found, tmp); - return (NULL); + env = get_variable_v2(tmp, func, info); + if (env == NULL) { + info_logger(LOG_INFO, bb_msg_variable_not_found, tmp); + return NULL; } - len = strlen (env); + len = strlen(env); if (len + *out_pos >= length) goto expand_variable_out; - memcpy (buffer + *out_pos, env, len + 1); + memcpy(buffer + *out_pos, env, len + 1); *out_pos += len; - return (input); + return input; } /* Variable in braces: check for ':' tricks */ ch = *++input; - for (ptr = input; isalnum (ch) || (ch == '_'); ch = *++ptr) + for (ptr = input; isalnum(ch) || (ch == '_'); ch = *++ptr) /* VOID */; - if (ch == '}') - { + if (ch == '}') { /* Must be simple variable expansion with "${var}" */ len = ptr - input; if ((size_t)len >= sizeof tmp) goto expand_variable_out; - safe_memcpy (tmp, input, len); - ptr = expand_variable (buffer, length, out_pos, tmp, func, info ); + safe_memcpy(tmp, input, len); + ptr = expand_variable(buffer, length, out_pos, tmp, func, info); if (ptr == NULL) - return (NULL); - return (input + len); + return NULL; + return input + len; } - if (ch != ':' || ptr[1] != '-' ) - { - msg_logger(LOG_INFO, "illegal char in var name"); - return (NULL); + if (ch != ':' || ptr[1] != '-') { + simple_info_logger(LOG_INFO, "illegal char in var name"); + return NULL; } /* It's that handy "${var:-word}" expression. Check if var is defined */ len = ptr - input; if ((size_t)len >= sizeof tmp) goto expand_variable_out; - safe_memcpy (tmp, input, len); + safe_memcpy(tmp, input, len); /* Move input pointer to ':' */ input = ptr; /* First skip to closing brace, taking note of nested expressions */ ptr += 2; ch = ptr[0]; - for (open_braces = 1; open_braces > 0; ch = *++ptr) - { - switch (ch) - { + for (open_braces = 1; open_braces > 0; ch = *++ptr) { + switch (ch) { case '{': ++open_braces; break; @@ -1972,26 +1803,26 @@ static const char *expand_variable (char *buffer, unsigned int length, --open_braces; break; case '\0': - msg_logger(LOG_INFO,"\"}\" not found in: %s", input); - return (NULL); + info_logger(LOG_INFO, "\"}\" not found in: %s", input); + return NULL; default: break; } } --ptr; /* At this point ptr should point to closing brace of "${var:-word}" */ - if ( ( env = get_variable_v2 (tmp, func, info) ) != NULL ) - { + env = get_variable_v2(tmp, func, info); + if (env != NULL) { /* Found environment variable, so skip the input to the closing brace and return the variable */ input = ptr; - len = strlen (env); + len = strlen(env); if (len + *out_pos >= length) goto expand_variable_out; - memcpy (buffer + *out_pos, env, len + 1); + memcpy(buffer + *out_pos, env, len + 1); *out_pos += len; - return (input); + return input; } /* Environment variable was not found, so process word. Advance input pointer to start of word in "${var:-word}" */ @@ -2000,26 +1831,26 @@ static const char *expand_variable (char *buffer, unsigned int length, if ((size_t)len >= sizeof tmp) goto expand_variable_out; - safe_memcpy (tmp, input, len); + safe_memcpy(tmp, input, len); input = ptr; - if ( !st_expr_expand (tmp, STRING_LENGTH, tmp, func, info ) ) - return (NULL); - len = strlen (tmp); + if (!st_expr_expand(tmp, STRING_LENGTH, tmp, func, info)) + return NULL; + len = strlen(tmp); if (len + *out_pos >= length) goto expand_variable_out; - memcpy (buffer + *out_pos, tmp, len + 1); + memcpy(buffer + *out_pos, tmp, len + 1); *out_pos += len; - return (input); + return input; expand_variable_out: - msg_logger(LOG_INFO, bb_msg_small_buffer); - return (NULL); + simple_info_logger(LOG_INFO, bb_msg_small_buffer); + return NULL; } /* End Function expand_variable */ -static const char *get_variable_v2 (const char *variable, - const char *(*func) (const char *variable, void *info), - void *info) +static const char *get_variable_v2(const char *variable, + const char *(*func)(const char *variable, void *info), + void *info) /* [SUMMARY] Get a variable from the environment or . The variable name. A function which will be used to get the variable. If this returns @@ -2030,15 +1861,12 @@ static const char *get_variable_v2 (const char *variable, { const char *value; - debug_msg_logger(LOG_INFO, __FUNCTION__); - - if (func != NULL) - { - value = (*func) (variable, info); + if (func != NULL) { + value = (*func)(variable, info); if (value != NULL) - return (value); + return value; } - return getenv (variable); + return getenv(variable); } /* End Function get_variable */ /* END OF CODE */