Prevent a segfault if no argument, by Geoffrey Lee <glee@bluesat.unsw.edu.au>

This commit is contained in:
Glenn L McGrath 2002-11-10 21:52:59 +00:00
parent f57674e9a9
commit 59870e89ae

View File

@ -1,9 +1,9 @@
/* /*
* od implementation for busybox * Mini xargs implementation for busybox
* Based on code from util-linux v 2.11l
* *
* Copyright (c) 1990 * Copyright (C) 1999,2000 by Lineo, inc. and Erik Andersen
* The Regents of the University of California. All rights reserved. * Copyright (C) 1999,2000,2001 by Erik Andersen <andersee@debian.org>
* Remixed by Mark Whitley <markw@codepoet.org>
* *
* This program is free software; you can redistribute it and/or modify * 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 * it under the terms of the GNU General Public License as published by
@ -19,248 +19,88 @@
* along with this program; if not, write to the Free Software * along with this program; if not, write to the Free Software
* Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
* *
* Original copyright notice is retained at the end of this file.
*/ */
#include <ctype.h> #include <stdio.h>
#include <getopt.h>
#include <stdlib.h> #include <stdlib.h>
#include "dump.h" #include <string.h>
#include "busybox.h" #include "busybox.h"
extern FS *fshead; /* head of format strings */ int xargs_main(int argc, char **argv)
extern int blocksize; /* data block size */
extern int length; /* max bytes to read */
#define ishexdigit(c) \
((c >= '0' && c <= '9') || (c >= 'a' && c <= 'f') || (c >= 'A' && c <= 'F'))
static void
odoffset(int argc, char ***argvp)
{ {
extern off_t skip; char *cmd_to_be_executed;
register char *num, *p; char *file_to_act_on;
int base; int i;
char *end; int len;
/* /*
* The offset syntax of od(1) was genuinely bizarre. First, if * No options are supported in this version of xargs; no getopt.
* it started with a plus it had to be an offset. Otherwise, if
* there were at least two arguments, a number or lower-case 'x'
* followed by a number makes it an offset. By default it was
* octal; if it started with 'x' or '0x' it was hex. If it ended
* in a '.', it was decimal. If a 'b' or 'B' was appended, it
* multiplied the number by 512 or 1024 byte units. There was
* no way to assign a block count to a hex offset.
* *
* We assumes it's a file if the offset is bad. * Re: The missing -t flag: Most programs that produce output also print
* the filename, so xargs doesn't really need to do it again. Supporting
* the -t flag =greatly= bloats up the size of this app and the memory it
* uses because you have to buffer all the input file strings in memory. If
* you really want to see the filenames that xargs will act on, just run it
* once with no args and xargs will echo the filename. Simple.
*/ */
p = **argvp;
if (*p != '+' && (argc < 2 ||
(!isdigit(p[0]) && (p[0] != 'x' || !ishexdigit(p[1])))))
return;
base = 0; argv++;
/* len = argc; /* arg = count for ' ' + trailing '\0' */
* skip over leading '+', 'x[0-9a-fA-f]' or '0x', and /* Store the command to be executed (taken from the command line) */
* set base. if (argc == 1) {
*/ /* default behavior is to echo all the filenames */
if (p[0] == '+') argv[0] = "/bin/echo";
++p; len++; /* space for trailing '\0' */
if (p[0] == 'x' && ishexdigit(p[1])) { } else {
++p; argc--;
base = 16; }
} else if (p[0] == '0' && p[1] == 'x') { /* concatenate all the arguments passed to xargs together */
p += 2; for (i = 0; i < argc; i++)
base = 16; len += strlen(argv[i]);
cmd_to_be_executed = xmalloc (len);
for (i = len = 0; i < argc; i++) {
len = sprintf(cmd_to_be_executed + len, "%s ", argv[i]);
} }
/* skip over the number */ /* Now, read in one line at a time from stdin, and store this
if (base == 16) * line to be used later as an argument to the command */
for (num = p; ishexdigit(*p); ++p); while ((file_to_act_on = get_line_from_file(stdin)) !=NULL) {
else
for (num = p; isdigit(*p); ++p);
/* check for no number */ FILE *cmd_output;
if (num == p) char *output_line;
return; char *execstr;
/* if terminates with a '.', base is decimal */ /* eat the newline off the filename. */
if (*p == '.') { chomp(file_to_act_on);
if (base)
return; /* eat blank lines */
base = 10; if (file_to_act_on[0] == 0)
continue;
/* assemble the command and execute it */
bb_asprintf(&execstr, "%s%s", cmd_to_be_executed, file_to_act_on);
cmd_output = popen(execstr, "r");
if (cmd_output == NULL)
perror_msg_and_die("popen");
/* harvest the output */
while ((output_line = get_line_from_file(cmd_output)) != NULL) {
fputs(output_line, stdout);
free(output_line);
} }
skip = strtol(num, &end, base ? base : 8); /* clean up */
pclose(cmd_output);
/* if end isn't the same as p, we got a non-octal digit */ free(execstr);
if (end != p) free(file_to_act_on);
skip = 0;
else {
if (*p) {
if (*p == 'b')
skip *= 512;
else if (*p == 'B')
skip *= 1024;
++p;
}
if (*p)
skip = 0;
else {
++*argvp;
/*
* If the offset uses a non-octal base, the base of
* the offset is changed as well. This isn't pretty,
* but it's easy.
*/
#define TYPE_OFFSET 7
if (base == 16) {
fshead->nextfu->fmt[TYPE_OFFSET] = 'x';
fshead->nextfs->nextfu->fmt[TYPE_OFFSET] = 'x';
} else if (base == 10) {
fshead->nextfu->fmt[TYPE_OFFSET] = 'd';
fshead->nextfs->nextfu->fmt[TYPE_OFFSET] = 'd';
}
}
}
} }
static void odprecede(void) #ifdef CONFIG_FEATURE_CLEAN_UP
{ free(cmd_to_be_executed);
static int first = 1; #endif
if (first) { return 0;
first = 0;
add("\"%07.7_Ao\n\"");
add("\"%07.7_ao \"");
} else
add("\" \"");
} }
int od_main(int argc, char **argv) /* vi: set sw=4 ts=4: */
{
int ch;
extern enum _vflag vflag;
vflag = FIRST;
length = -1;
while ((ch = getopt(argc, argv, "aBbcDdeFfHhIiLlOoPpswvXx")) != EOF)
switch (ch) {
case 'a':
odprecede();
add("16/1 \"%3_u \" \"\\n\"");
break;
case 'B':
case 'o':
odprecede();
add("8/2 \" %06o \" \"\\n\"");
break;
case 'b':
odprecede();
add("16/1 \"%03o \" \"\\n\"");
break;
case 'c':
odprecede();
add("16/1 \"%3_c \" \"\\n\"");
break;
case 'd':
odprecede();
add("8/2 \" %05u \" \"\\n\"");
break;
case 'D':
odprecede();
add("4/4 \" %010u \" \"\\n\"");
break;
case 'e': /* undocumented in od */
case 'F':
odprecede();
add("2/8 \" %21.14e \" \"\\n\"");
break;
case 'f':
odprecede();
add("4/4 \" %14.7e \" \"\\n\"");
break;
case 'H':
case 'X':
odprecede();
add("4/4 \" %08x \" \"\\n\"");
break;
case 'h':
case 'x':
odprecede();
add("8/2 \" %04x \" \"\\n\"");
break;
case 'I':
case 'L':
case 'l':
odprecede();
add("4/4 \" %11d \" \"\\n\"");
break;
case 'i':
odprecede();
add("8/2 \" %6d \" \"\\n\"");
break;
case 'O':
odprecede();
add("4/4 \" %011o \" \"\\n\"");
break;
case 'v':
vflag = ALL;
break;
case 'P':
case 'p':
case 's':
case 'w':
case '?':
default:
error_msg("od: od(1) has been deprecated for hexdump(1).\n");
if (ch != '?') {
error_msg("od: hexdump(1) compatibility doesn't support the -%c option%s\n",
ch, ch == 's' ? "; see strings(1)." : ".");
}
show_usage();
}
if (!fshead) {
add("\"%07.7_Ao\n\"");
add("\"%07.7_ao \" 8/2 \"%06o \" \"\\n\"");
}
argc -= optind;
argv += optind;
odoffset(argc, &argv);
return(dump(argv));
}
/*-
* Copyright (c) 1990 The Regents of the University of California.
* All rights reserved.
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions
* are met:
* 1. Redistributions of source code must retain the above copyright
* notice, this list of conditions and the following disclaimer.
* 2. Redistributions in binary form must reproduce the above copyright
* notice, this list of conditions and the following disclaimer in the
* documentation and/or other materials provided with the distribution.
* 3. Neither the name of the University nor the names of its contributors
* may be used to endorse or promote products derived from this software
* without specific prior written permission.
*
* THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
* ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
* ARE DISCLAIMED. IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
* FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
* DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
* OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
* HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
* LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
* OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
* SUCH DAMAGE.
*/