Submitted By: Jim Gifford (jim at cross-lfs dot org) Date: 01-25-2009 Initial Package Version: 1.13.2 Origin: Upstream Upstream Status: Applied Description: This is a branch update for busybox-1.13.2, and should be rechecked periodically. diff -Naur busybox-1.13.2.orig/archival/tar.c busybox-1.13.2/archival/tar.c --- busybox-1.13.2.orig/archival/tar.c 2008-11-09 09:28:02.000000000 -0800 +++ busybox-1.13.2/archival/tar.c 2009-01-25 12:52:44.955898588 -0800 @@ -591,8 +591,6 @@ struct TarBallInfo tbInfo; tbInfo.hlInfoHead = NULL; - - fchmod(tar_fd, 0644); tbInfo.tarFd = tar_fd; tbInfo.verboseFlag = verboseFlag; diff -Naur busybox-1.13.2.orig/modutils/depmod.c busybox-1.13.2/modutils/depmod.c --- busybox-1.13.2.orig/modutils/depmod.c 2008-11-09 09:28:03.000000000 -0800 +++ busybox-1.13.2/modutils/depmod.c 2009-01-25 12:52:44.229924290 -0800 @@ -33,7 +33,7 @@ enum { ARG_a = (1<<0), /* All modules, ignore mods in argv */ ARG_A = (1<<1), /* Only emit .ko that are newer than modules.dep file */ - ARG_b = (1<<2), /* not /lib/modules/$(uname -r)/ but this base-dir */ + ARG_b = (1<<2), /* base directory when modules are in staging area */ ARG_e = (1<<3), /* with -F, print unresolved symbols */ ARG_F = (1<<4), /* System.map that contains the symbols */ ARG_n = (1<<5) /* dry-run, print to stdout only */ @@ -57,7 +57,7 @@ *first = info; info->dnext = info->dprev = info; - info->name = xstrdup(fname); + info->name = xasprintf("/%s", fname); info->modname = filename2modname(fname, NULL); for (ptr = image; ptr < image + len - 10; ptr++) { if (strncmp(ptr, "depends=", 8) == 0) { @@ -123,44 +123,61 @@ } } +static void xfreopen_write(const char *file, FILE *f) +{ + if (freopen(file, "w", f) == NULL) + bb_perror_msg_and_die("can't open '%s'", file); +} + int depmod_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; int depmod_main(int argc UNUSED_PARAM, char **argv) { module_info *modules = NULL, *m, *dep; - char *moddir_base = (char *)CONFIG_DEFAULT_MODULES_DIR; + const char *moddir_base = "/"; + char *moddir, *version; + struct utsname uts; int tmp; getopt32(argv, "aAb:eF:n", &moddir_base, NULL); argv += optind; /* goto modules location */ + xchdir(moddir_base); /* If a version is provided, then that kernel version's module directory * is used, rather than the current kernel version (as returned by * "uname -r"). */ - xchdir(moddir_base); - if (*argv && (sscanf(*argv, "%d.%d.%d", &tmp, &tmp, &tmp) == 3)) { - xchdir(*argv++); + if (*argv && sscanf(*argv, "%d.%d.%d", &tmp, &tmp, &tmp) == 3) { + version = *argv++; } else { - struct utsname uts; uname(&uts); - xchdir(uts.release); + version = uts.release; } - /* If no modules are given on the command-line, -a is on per default. */ - option_mask32 |= *argv == NULL; + moddir = concat_path_file(&CONFIG_DEFAULT_MODULES_DIR[1], version); /* Scan modules */ - moddir_base = xrealloc_getcwd_or_warn(NULL); - do { - recursive_action((option_mask32 & ARG_a) ? moddir_base : *argv, - ACTION_RECURSE, parse_module, NULL, &modules, 0); - } while (!(option_mask32 & ARG_a) && *(++argv)); + if (*argv) { + char *modfile; + struct stat sb; + do { + modfile = concat_path_file(moddir, *argv); + xstat(modfile, &sb); + parse_module(modfile, &sb, &modules, 0); + free(modfile); + } while (*(++argv)); + } else { + recursive_action(moddir, ACTION_RECURSE, + parse_module, NULL, &modules, 0); + } + + /* Prepare for writing out the dep files */ + xchdir(moddir); if (ENABLE_FEATURE_CLEAN_UP) - free(moddir_base); + free(moddir); /* Generate dependency and alias files */ if (!(option_mask32 & ARG_n)) - freopen(CONFIG_DEFAULT_DEPMOD_FILE, "w", stdout); + xfreopen_write(CONFIG_DEFAULT_DEPMOD_FILE, stdout); for (m = modules; m != NULL; m = m->next) { printf("%s:", m->name); @@ -174,12 +191,12 @@ dep->dprev->dnext = dep->dnext; dep->dnext = dep->dprev = dep; } - puts(""); + bb_putchar('\n'); } #if ENABLE_FEATURE_MODUTILS_ALIAS if (!(option_mask32 & ARG_n)) - freopen("modules.alias", "w", stdout); + xfreopen_write("modules.alias", stdout); for (m = modules; m != NULL; m = m->next) { while (m->aliases) { printf("alias %s %s\n", @@ -190,7 +207,7 @@ #endif #if ENABLE_FEATURE_MODUTILS_SYMBOLS if (!(option_mask32 & ARG_n)) - freopen("modules.symbols", "w", stdout); + xfreopen_write("modules.symbols", stdout); for (m = modules; m != NULL; m = m->next) { while (m->symbols) { printf("alias symbol:%s %s\n", diff -Naur busybox-1.13.2.orig/testsuite/mdev.tests busybox-1.13.2/testsuite/mdev.tests --- busybox-1.13.2.orig/testsuite/mdev.tests 2008-11-09 09:28:19.000000000 -0800 +++ busybox-1.13.2/testsuite/mdev.tests 2009-01-25 12:52:44.443204916 -0800 @@ -111,6 +111,22 @@ # continuing to use directory structure from prev test rm -rf mdev.testdir/dev/* +echo "sda 0:0 644 =block/ @echo @echo TEST" >mdev.testdir/etc/mdev.conf +testing "mdev move and command" \ + "env - PATH=$PATH ACTION=add DEVPATH=/block/sda chroot mdev.testdir /mdev 2>&1; + ls -lnR mdev.testdir/dev | $FILTER_LS2" \ +"\ +@echo TEST +mdev.testdir/dev: +drwxr-xr-x 2 0 0 block + +mdev.testdir/dev/block: +brw-r--r-- 1 0 0 sda +" \ + "" "" + +# continuing to use directory structure from prev test +rm -rf mdev.testdir/dev/* echo "@8,0 :1 644" >mdev.testdir/etc/mdev.conf testing "mdev #maj,min and no explicit uid" \ "env - PATH=$PATH ACTION=add DEVPATH=/block/sda chroot mdev.testdir /mdev 2>&1; diff -Naur busybox-1.13.2.orig/util-linux/mdev.c busybox-1.13.2/util-linux/mdev.c --- busybox-1.13.2.orig/util-linux/mdev.c 2008-11-09 09:28:22.000000000 -0800 +++ busybox-1.13.2/util-linux/mdev.c 2009-01-25 12:52:44.443204916 -0800 @@ -179,8 +179,9 @@ unsigned i, n; #endif char *a = val; - s = strchr(val, ' '); - val = (s && s[1]) ? s+1 : NULL; + s = strchrnul(val, ' '); + val = (s[0] && s[1]) ? s+1 : NULL; + s[0] = '\0'; #if ENABLE_FEATURE_MDEV_RENAME_REGEXP /* substitute %1..9 with off[1..9], if any */ n = 0;