*
* Copyright (c) 2002 by Robert Griebl, griebl@gmx.de
* Copyright (c) 2003 by Andrew Dennison, andrew.dennison@motec.com.au
+ * Copyright (c) 2005 by Jim Bauer, jfbauer@nfr.com
*
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ * Portions Copyright (c) 2005 by Yann E. MORIN, yann.morin.1998@anciens.enib.fr
*
+ * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
*/
#include <sys/utsname.h>
+#include <sys/types.h>
+#include <sys/wait.h>
#include <getopt.h>
#include <stdlib.h>
#include <unistd.h>
#include <fcntl.h>
#include "busybox.h"
+struct mod_opt_t { /* one-way list of options to pass to a module */
+ char * m_opt_val;
+ struct mod_opt_t * m_next;
+};
+struct dep_t { /* one-way list of dependency rules */
+ /* a dependency rule */
+ char * m_name; /* the module name*/
+ char * m_path; /* the module file path */
+ struct mod_opt_t * m_options; /* the module options */
-struct dep_t {
- char * m_module;
- char * m_options;
-
- int m_isalias : 1;
- int m_reserved : 15;
+ int m_isalias : 1; /* the module is an alias */
+ int m_reserved : 15; /* stuffin' */
- int m_depcnt : 16;
- char ** m_deparr;
+ int m_depcnt : 16; /* the number of dependable module(s) */
+ char ** m_deparr; /* the list of dependable module(s) */
- struct dep_t * m_next;
+ struct dep_t * m_next; /* the next dependency rule */
};
-struct mod_list_t {
- char * m_module;
- char * m_options;
+struct mod_list_t { /* two-way list of modules to process */
+ /* a module description */
+ char * m_name;
+ char * m_path;
+ struct mod_opt_t * m_options;
struct mod_list_t * m_prev;
struct mod_list_t * m_next;
static struct dep_t *depend;
static int autoclean, show_only, quiet, do_syslog, verbose;
-static int k_version;
-int parse_tag_value ( char *buffer, char **ptag, char **pvalue )
+static int parse_tag_value ( char *buffer, char **ptag, char **pvalue )
{
char *tag, *value;
buffer++;
tag = value = buffer;
while ( !isspace ( *value ))
- value++;
+ if (!*value) return 0;
+ else value++;
*value++ = 0;
while ( isspace ( *value ))
value++;
+ if (!*value) return 0;
*ptag = tag;
*pvalue = value;
- return bb_strlen( tag ) && bb_strlen( value );
+ return 1;
}
/* Jump through hoops to simulate how fgets() grabs just one line at a
return 0;
}
+/*
+ * This function appends an option to a list
+ */
+struct mod_opt_t *append_option( struct mod_opt_t *opt_list, char *opt )
+{
+ struct mod_opt_t *ol = opt_list;
+
+ if( ol ) {
+ while( ol-> m_next ) {
+ ol = ol-> m_next;
+ }
+ ol-> m_next = xmalloc( sizeof( struct mod_opt_t ) );
+ ol = ol-> m_next;
+ } else {
+ ol = opt_list = xmalloc( sizeof( struct mod_opt_t ) );
+ }
+
+ ol-> m_opt_val = bb_xstrdup( opt );
+ ol-> m_next = NULL;
+
+ return opt_list;
+}
+
+#if ENABLE_FEATURE_MODPROBE_MULTIPLE_OPTIONS
+/* static char* parse_command_string( char* src, char **dst );
+ * src: pointer to string containing argument
+ * dst: pointer to where to store the parsed argument
+ * return value: the pointer to the first char after the parsed argument,
+ * NULL if there was no argument parsed (only trailing spaces).
+ * Note that memory is allocated with bb_xstrdup when a new argument was
+ * parsed. Don't forget to free it!
+ */
+#define ARG_EMPTY 0x00
+#define ARG_IN_DQUOTES 0x01
+#define ARG_IN_SQUOTES 0x02
+static char *parse_command_string( char *src, char **dst )
+{
+ int opt_status = ARG_EMPTY;
+ char* tmp_str;
+
+ /* Dumb you, I have nothing to do... */
+ if( src == NULL ) return src;
+
+ /* Skip leading spaces */
+ while( *src == ' ' ) {
+ src++;
+ }
+ /* Is the end of string reached? */
+ if( *src == '\0' ) {
+ return NULL;
+ }
+ /* Reached the start of an argument
+ * By the way, we duplicate a little too much
+ * here but what is too much is freed later. */
+ *dst = tmp_str = bb_xstrdup( src );
+ /* Get to the end of that argument */
+ while( ( *tmp_str != '\0' )
+ && ( ( *tmp_str != ' ' )
+ || ( opt_status & ( ARG_IN_DQUOTES | ARG_IN_SQUOTES ) ) ) ) {
+ switch( *tmp_str ) {
+ case '\'':
+ if( opt_status & ARG_IN_DQUOTES ) {
+ /* Already in double quotes, keep current char as is */
+ } else {
+ /* shift left 1 char, until end of string: get rid of the opening/closing quotes */
+ memmove( tmp_str, tmp_str + 1, strlen( tmp_str ) );
+ /* mark me: we enter or leave single quotes */
+ opt_status ^= ARG_IN_SQUOTES;
+ /* Back one char, as we need to re-scan the new char there. */
+ tmp_str--;
+ }
+ break;
+ case '"':
+ if( opt_status & ARG_IN_SQUOTES ) {
+ /* Already in single quotes, keep current char as is */
+ } else {
+ /* shift left 1 char, until end of string: get rid of the opening/closing quotes */
+ memmove( tmp_str, tmp_str + 1, strlen( tmp_str ) );
+ /* mark me: we enter or leave double quotes */
+ opt_status ^= ARG_IN_DQUOTES;
+ /* Back one char, as we need to re-scan the new char there. */
+ tmp_str--;
+ }
+ break;
+ case '\\':
+ if( opt_status & ARG_IN_SQUOTES ) {
+ /* Between single quotes: keep as is. */
+ } else {
+ switch( *(tmp_str+1) ) {
+ case 'a':
+ case 'b':
+ case 't':
+ case 'n':
+ case 'v':
+ case 'f':
+ case 'r':
+ case '0':
+ /* We escaped a special character. For now, keep
+ * both the back-slash and the following char. */
+ tmp_str++; src++;
+ break;
+ default:
+ /* We escaped a space or a single or double quote,
+ * or a back-slash, or a non-escapable char. Remove
+ * the '\' and keep the new current char as is. */
+ memmove( tmp_str, tmp_str + 1, strlen( tmp_str ) );
+ break;
+ }
+ }
+ break;
+ /* Any other char that is special shall appear here.
+ * Example: $ starts a variable
+ case '$':
+ do_variable_expansion();
+ break;
+ * */
+ default:
+ /* any other char is kept as is. */
+ break;
+ }
+ tmp_str++; /* Go to next char */
+ src++; /* Go to next char to find the end of the argument. */
+ }
+ /* End of string, but still no ending quote */
+ if( opt_status & ( ARG_IN_DQUOTES | ARG_IN_SQUOTES ) ) {
+ bb_error_msg_and_die( "unterminated (single or double) quote in options list: %s", src );
+ }
+ *tmp_str = '\0';
+ *dst = xrealloc( *dst, strlen( *dst ) );
+ return src;
+}
+#else
+#define parse_command_string(src, dst) (0)
+#endif /* ENABLE_FEATURE_MODPROBE_MULTIPLE_OPTIONS */
+
+/*
+ * This function builds a list of dependency rules from /lib/modules/`uname -r\modules.dep.
+ * It then fills every modules and aliases with their default options, found by parsing
+ * modprobe.conf (or modules.conf, or conf.modules).
+ */
static struct dep_t *build_dep ( void )
{
int fd;
struct utsname un;
struct dep_t *first = 0;
struct dep_t *current = 0;
- char buffer[256];
+ char buffer[2048];
char *filename = buffer;
int continuation_line = 0;
+ int k_version;
k_version = 0;
if ( uname ( &un ))
- return 0;
+ bb_error_msg_and_die("can't determine kernel version");
// check for buffer overflow in following code
if ( bb_strlen ( un.release ) > ( sizeof( buffer ) - 64 )) {
continue;
}
+ /* Is this a new module dep description? */
if ( !continuation_line ) {
+ /* find the dep beginning */
char *col = strchr ( buffer, ':' );
+ char *dot = col;
if ( col ) {
+ /* This line is a dep description */
char *mods;
+ char *modpath;
char *mod;
- int ext = 0;
+ /* Find the beginning of the module file name */
*col = 0;
mods = strrchr ( buffer, '/' );
if ( !mods )
- mods = buffer;
+ mods = buffer; /* no path for this module */
else
- mods++;
-
-#if defined(CONFIG_FEATURE_2_6_MODULES)
- if ((k_version > 4) && ( *(col-3) == '.' ) &&
- ( *(col-2) == 'k' ) && ( *(col-1) == 'o' ))
- ext = 3;
+ mods++; /* there was a path for this module... */
+
+ /* find the path of the module */
+ modpath = strchr ( buffer, '/' ); /* ... and this is the path */
+ if ( !modpath )
+ modpath = buffer; /* module with no path */
+ /* find the end of the module name in the file name */
+ if ( ENABLE_FEATURE_2_6_MODULES &&
+ (k_version > 4) && ( *(col-3) == '.' ) &&
+ ( *(col-2) == 'k' ) && ( *(col-1) == 'o' ) )
+ dot = col - 3;
else
-#endif
if (( *(col-2) == '.' ) && ( *(col-1) == 'o' ))
- ext = 2;
+ dot = col - 2;
- mod = bb_xstrndup ( mods, col - mods - ext );
+ mod = bb_xstrndup ( mods, dot - mods );
+ /* enqueue new module */
if ( !current ) {
first = current = (struct dep_t *) xmalloc ( sizeof ( struct dep_t ));
}
current-> m_next = (struct dep_t *) xmalloc ( sizeof ( struct dep_t ));
current = current-> m_next;
}
- current-> m_module = mod;
- current-> m_options = 0;
+ current-> m_name = mod;
+ current-> m_path = bb_xstrdup(modpath);
+ current-> m_options = NULL;
current-> m_isalias = 0;
current-> m_depcnt = 0;
current-> m_deparr = 0;
current-> m_next = 0;
- //printf ( "%s:\n", mod );
p = col + 1;
}
else
+ /* this line is not a dep description */
p = 0;
}
else
+ /* It's a dep description continuation */
p = buffer;
while ( p && *p && isblank(*p))
p++;
+ /* p points to the first dependable module; if NULL, no dependable module */
if ( p && *p ) {
char *end = &buffer [l-1];
char *deps;
do
{
+ /* search the end of the dependency */
next = strchr (p, ' ' );
if (next)
{
else
next = end;
+ /* find the beginning of the module file name */
deps = strrchr ( p, '/' );
if ( !deps || ( deps < p )) {
else
deps++;
-#if defined(CONFIG_FEATURE_2_6_MODULES)
- if ((k_version > 4) && ( *(next-2) == '.' ) && *(next-1) == 'k' &&
- ( *next == 'o' ))
+ /* find the end of the module name in the file name */
+ if ( ENABLE_FEATURE_2_6_MODULES &&
+ (k_version > 4) && ( *(next-2) == '.' ) &&
+ ( *(next-1) == 'k' ) && ( *next == 'o' ) )
ext = 3;
else
-#endif
if (( *(next-1) == '.' ) && ( *next == 'o' ))
ext = 2;
continue;
dep = bb_xstrndup ( deps, next - deps - ext + 1 );
+ /* Add the new dependable module name */
current-> m_depcnt++;
current-> m_deparr = (char **) xrealloc ( current-> m_deparr,
sizeof ( char *) * current-> m_depcnt );
current-> m_deparr [current-> m_depcnt - 1] = dep;
- //printf ( " %d) %s\n", current-> m_depcnt, current-> m_deparr [current-> m_depcnt -1] );
p = next + 2;
} while (next < end);
}
+ /* is there other dependable module(s) ? */
if ( buffer [l-1] == '\\' )
continuation_line = 1;
else
// alias parsing is not 100% correct (no correct handling of continuation lines within an alias) !
- if (( fd = open ( "/etc/modules.conf", O_RDONLY )) < 0 )
- if (( fd = open ( "/etc/conf.modules", O_RDONLY )) < 0 )
- return first;
+ if (!ENABLE_FEATURE_2_6_MODULES
+ || ( fd = open ( "/etc/modprobe.conf", O_RDONLY )) < 0 )
+ if (( fd = open ( "/etc/modules.conf", O_RDONLY )) < 0 )
+ if (( fd = open ( "/etc/conf.modules", O_RDONLY )) < 0 )
+ return first;
continuation_line = 0;
while ( reads ( fd, buffer, sizeof( buffer ))) {
char *alias, *mod;
if ( parse_tag_value ( buffer + 6, &alias, &mod )) {
- // fprintf ( stderr, "ALIAS: '%s' -> '%s'\n", alias, mod );
-
+ /* handle alias as a module dependent on the aliased module */
if ( !current ) {
first = current = (struct dep_t *) xcalloc ( 1, sizeof ( struct dep_t ));
}
current-> m_next = (struct dep_t *) xcalloc ( 1, sizeof ( struct dep_t ));
current = current-> m_next;
}
- current-> m_module = bb_xstrdup ( alias );
+ current-> m_name = bb_xstrdup ( alias );
current-> m_isalias = 1;
- if (( strcmp ( alias, "off" ) == 0 ) || ( strcmp ( alias, "null" ) == 0 )) {
+ if (( strcmp ( mod, "off" ) == 0 ) || ( strcmp ( mod, "null" ) == 0 )) {
current-> m_depcnt = 0;
current-> m_deparr = 0;
}
else if (( strncmp ( buffer, "options", 7 ) == 0 ) && isspace ( buffer [7] )) {
char *mod, *opt;
+ /* split the line in the module/alias name, and options */
if ( parse_tag_value ( buffer + 8, &mod, &opt )) {
struct dep_t *dt;
+ /* find the corresponding module */
for ( dt = first; dt; dt = dt-> m_next ) {
- if ( strcmp ( dt-> m_module, mod ) == 0 )
+ if ( strcmp ( dt-> m_name, mod ) == 0 )
break;
}
if ( dt ) {
- dt-> m_options = xrealloc ( dt-> m_options, bb_strlen( opt ) + 1 );
- strcpy ( dt-> m_options, opt );
-
- // fprintf ( stderr, "OPTION: '%s' -> '%s'\n", dt-> m_module, dt-> m_options );
+ if ( ENABLE_FEATURE_MODPROBE_MULTIPLE_OPTIONS ) {
+ char* new_opt = NULL;
+ while( ( opt = parse_command_string( opt, &new_opt ) ) ) {
+ dt-> m_options = append_option( dt-> m_options, new_opt );
+ }
+ } else {
+ dt-> m_options = append_option( dt-> m_options, opt );
+ }
}
}
}
static int already_loaded (const char *name)
{
int fd;
- char buffer[256];
+ char buffer[4096];
fd = open ("/proc/modules", O_RDONLY);
if (fd < 0)
p = strchr (buffer, ' ');
if (p) {
*p = 0;
+ for( p = buffer; ENABLE_FEATURE_2_6_MODULES && *p; p++ ) {
+ *p = ((*p)=='-')?'_':*p;
+ }
if (strcmp (name, buffer) == 0) {
close (fd);
return 1;
static int mod_process ( struct mod_list_t *list, int do_insert )
{
- char lcmd [256];
- int rc = 1;
+ int rc = 0;
+ char **argv = NULL;
+ struct mod_opt_t *opts;
+ int argc_malloc; /* never used when CONFIG_FEATURE_CLEAN_UP not defined */
+ int argc;
while ( list ) {
- *lcmd = '\0';
+ argc = 0;
+ if( ENABLE_FEATURE_CLEAN_UP )
+ argc_malloc = 0;
+ /* If CONFIG_FEATURE_CLEAN_UP is not defined, then we leak memory
+ * each time we allocate memory for argv.
+ * But it is (quite) small amounts of memory that leak each
+ * time a module is loaded, and it is reclaimed when modprobe
+ * exits anyway (even when standalone shell?).
+ * This could become a problem when loading a module with LOTS of
+ * dependencies, with LOTS of options for each dependencies, with
+ * very little memory on the target... But in that case, the module
+ * would not load because there is no more memory, so there's no
+ * problem. */
+ /* enough for minimal insmod (5 args + NULL) or rmmod (3 args + NULL) */
+ argv = (char**) malloc( 6 * sizeof( char* ) );
if ( do_insert ) {
- if (already_loaded (list->m_module) != 1)
- snprintf ( lcmd, sizeof( lcmd ) - 1, "insmod %s %s %s %s %s",
- do_syslog ? "-s" : "", autoclean ? "-k" : "",
- quiet ? "-q" : "", list-> m_module, list-> m_options ?
- list-> m_options : "" );
+ if (already_loaded (list->m_name) != 1) {
+ argv[argc++] = "insmod";
+ if (do_syslog)
+ argv[argc++] = "-s";
+ if (autoclean)
+ argv[argc++] = "-k";
+ if (quiet)
+ argv[argc++] = "-q";
+ else if(verbose) /* verbose and quiet are mutually exclusive */
+ argv[argc++] = "-v";
+ argv[argc++] = list-> m_path;
+ if( ENABLE_FEATURE_CLEAN_UP )
+ argc_malloc = argc;
+ opts = list-> m_options;
+ while( opts ) {
+ /* Add one more option */
+ argc++;
+ argv = (char**) xrealloc( argv, ( argc + 1 ) * sizeof( char* ) );
+ argv[argc-1] = opts-> m_opt_val;
+ opts = opts-> m_next;
+ }
+ }
} else {
- if (already_loaded (list->m_module) != 0)
- snprintf ( lcmd, sizeof( lcmd ) - 1, "rmmod %s %s",
- do_syslog ? "-s" : "", list-> m_module );
+ /* modutils uses short name for removal */
+ if (already_loaded (list->m_name) != 0) {
+ argv[argc++] = "rmmod";
+ if (do_syslog)
+ argv[argc++] = "-s";
+ argv[argc++] = list->m_name;
+ if( ENABLE_FEATURE_CLEAN_UP )
+ argc_malloc = argc;
+ }
}
+ argv[argc] = NULL;
- if ( verbose )
- printf ( "%s\n", lcmd );
- if ( !show_only && *lcmd) {
- int rc2 = system ( lcmd );
- if (do_insert) rc = rc2; /* only last module matters */
- else if (!rc2) rc = 0; /* success if remove any mod */
+ if (argc) {
+ if (verbose) {
+ printf("%s module %s\n", do_insert?"Loading":"Unloading", list-> m_name );
+ }
+ if (!show_only) {
+ int rc2 = 0;
+ int status;
+ switch (fork()) {
+ case -1:
+ rc2 = 1;
+ break;
+ case 0: //child
+ execvp(argv[0], argv);
+ bb_perror_msg_and_die("exec of %s", argv[0]);
+ /* NOTREACHED */
+ default:
+ if (wait(&status) == -1) {
+ rc2 = 1;
+ break;
+ }
+ if (WIFEXITED(status))
+ rc2 = WEXITSTATUS(status);
+ if (WIFSIGNALED(status))
+ rc2 = WTERMSIG(status);
+ break;
+ }
+ if (do_insert) {
+ rc = rc2; /* only last module matters */
+ }
+ else if (!rc2) {
+ rc = 0; /* success if remove any mod */
+ }
+ }
+ if( ENABLE_FEATURE_CLEAN_UP )
+ /* the last value in the array has index == argc, but
+ * it is the terminating NULL, so we must not free it. */
+ while( argc_malloc < argc ) {
+ free( argv[argc_malloc++] );
+ }
+ }
+ if( ENABLE_FEATURE_CLEAN_UP ) {
+ free( argv );
+ argv = NULL;
}
-
list = do_insert ? list-> m_prev : list-> m_next;
}
return (show_only) ? 0 : rc;
}
+/*
+ * Builds the dependency list (aka stack) of a module.
+ * head: the highest module in the stack (last to insmod, first to rmmod)
+ * tail: the lowest module in the stack (first to insmod, last to rmmod)
+ */
static void check_dep ( char *mod, struct mod_list_t **head, struct mod_list_t **tail )
{
struct mod_list_t *find;
struct dep_t *dt;
- char *opt = 0;
- int lm;
-
- // remove .o extension
- lm = bb_strlen ( mod );
-
-#if defined(CONFIG_FEATURE_2_6_MODULES)
- if ((k_version > 4) && ( mod [lm-3] == '.' ) &&
- ( mod [lm-2] == 'k' ) && ( mod [lm-1] == 'o' ))
- mod [lm-3] = 0;
- else
-#endif
- if (( mod [lm-2] == '.' ) && ( mod [lm-1] == 'o' ))
- mod [lm-2] = 0;
+ struct mod_opt_t *opt = 0;
+ char *path = 0;
// check dependencies
for ( dt = depend; dt; dt = dt-> m_next ) {
- if ( strcmp ( dt-> m_module, mod ) == 0 ) {
- opt = dt-> m_options;
+ if ( strcmp ( dt-> m_name, mod ) == 0) {
break;
}
}
+ if( !dt ) {
+ bb_error_msg ("module %s not found.", mod);
+ return;
+ }
+
// resolve alias names
- while ( dt && dt-> m_isalias ) {
+ while ( dt-> m_isalias ) {
if ( dt-> m_depcnt == 1 ) {
struct dep_t *adt;
for ( adt = depend; adt; adt = adt-> m_next ) {
- if ( strcmp ( adt-> m_module, dt-> m_deparr [0] ) == 0 )
+ if ( strcmp ( adt-> m_name, dt-> m_deparr [0] ) == 0 )
break;
}
if ( adt ) {
+ /* This is the module we are aliased to */
+ struct mod_opt_t *opts = dt-> m_options;
+ /* Option of the alias are appended to the options of the module */
+ while( opts ) {
+ adt-> m_options = append_option( adt-> m_options, opts-> m_opt_val );
+ opts = opts-> m_next;
+ }
dt = adt;
- mod = dt-> m_module;
- if ( !opt )
- opt = dt-> m_options;
}
- else
+ else {
+ bb_error_msg ("module %s not found.", mod);
return;
+ }
}
- else
+ else {
+ bb_error_msg ("Bad alias %s", dt-> m_name);
return;
+ }
}
+ mod = dt-> m_name;
+ path = dt-> m_path;
+ opt = dt-> m_options;
+
// search for duplicates
for ( find = *head; find; find = find-> m_next ) {
- if ( !strcmp ( mod, find-> m_module )) {
+ if ( !strcmp ( mod, find-> m_name )) {
// found -> dequeue it
if ( find-> m_prev )
if ( !find ) { // did not find a duplicate
find = (struct mod_list_t *) xmalloc ( sizeof(struct mod_list_t));
- find-> m_module = mod;
+ find-> m_name = mod;
+ find-> m_path = path;
find-> m_options = opt;
}
if ( dt ) {
int i;
+ /* Add all dependable module for that new module */
for ( i = 0; i < dt-> m_depcnt; i++ )
check_dep ( dt-> m_deparr [i], head, tail );
}
}
-
-
static int mod_insert ( char *mod, int argc, char **argv )
{
struct mod_list_t *tail = 0;
check_dep ( mod, &head, &tail );
if ( head && tail ) {
- if ( argc ) {
+ if( argc ) {
int i;
- int l = 0;
-
// append module args
for ( i = 0; i < argc; i++ )
- l += ( bb_strlen ( argv [i] ) + 1 );
-
- head-> m_options = xrealloc ( head-> m_options, l + 1 );
- head-> m_options [0] = 0;
-
- for ( i = 0; i < argc; i++ ) {
- strcat ( head-> m_options, argv [i] );
- strcat ( head-> m_options, " " );
- }
+ head->m_options = append_option( head->m_options, argv[i] );
}
// process tail ---> head
}
-
-
extern int modprobe_main(int argc, char** argv)
{
int opt;
+ int rc = EXIT_SUCCESS;
int remove_opt = 0;
autoclean = show_only = quiet = do_syslog = verbose = 0;
show_only++;
break;
case 'q':
- quiet++;
+ quiet++; verbose=0;
break;
case 'r':
remove_opt++;
do_syslog++;
break;
case 'v':
- verbose++;
+ verbose++; quiet=0;
break;
case 'V':
default:
bb_error_msg_and_die ( "could not parse modules.dep\n" );
if (remove_opt) {
- int rc = EXIT_SUCCESS;
do {
if (mod_remove ( optind < argc ?
bb_xstrdup (argv [optind]) : NULL )) {
rc = EXIT_FAILURE;
}
} while ( ++optind < argc );
+ } else {
+ if (optind >= argc)
+ bb_error_msg_and_die ( "No module or pattern provided\n" );
- return rc;
+ if ( mod_insert ( bb_xstrdup ( argv [optind] ), argc - optind - 1, argv + optind + 1 ))
+ bb_error_msg_and_die ( "failed to load module %s", argv [optind] );
}
- if (optind >= argc)
- bb_error_msg_and_die ( "No module or pattern provided\n" );
-
- if ( mod_insert ( bb_xstrdup ( argv [optind] ), argc - optind - 1, argv + optind + 1 ))
- bb_error_msg_and_die ( "failed to load module %s", argv [optind] );
+ /* Here would be a good place to free up memory allocated during the dependencies build. */
- return EXIT_SUCCESS;
+ return rc;
}
-
-