diff options
Diffstat (limited to 'libbb')
-rw-r--r-- | libbb/fclose_nonstdin.c | 10 | ||||
-rw-r--r-- | libbb/getopt32.c | 2 | ||||
-rw-r--r-- | libbb/wfopen_input.c | 25 |
3 files changed, 29 insertions, 8 deletions
diff --git a/libbb/fclose_nonstdin.c b/libbb/fclose_nonstdin.c index 88e8474f2..768ee946d 100644 --- a/libbb/fclose_nonstdin.c +++ b/libbb/fclose_nonstdin.c | |||
@@ -16,8 +16,10 @@ | |||
16 | 16 | ||
17 | int fclose_if_not_stdin(FILE *f) | 17 | int fclose_if_not_stdin(FILE *f) |
18 | { | 18 | { |
19 | if (f != stdin) { | 19 | /* Some more paranoid applets want ferror() check too */ |
20 | return fclose(f); | 20 | int r = ferror(f); /* NB: does NOT set errno! */ |
21 | } | 21 | if (r) errno = EIO; /* so we'll help it */ |
22 | return 0; | 22 | if (f != stdin) |
23 | return (r | fclose(f)); /* fclose does set errno on error */ | ||
24 | return r; | ||
23 | } | 25 | } |
diff --git a/libbb/getopt32.c b/libbb/getopt32.c index f48ccaa6a..9e53dfd08 100644 --- a/libbb/getopt32.c +++ b/libbb/getopt32.c | |||
@@ -281,6 +281,8 @@ Special characters: | |||
281 | 281 | ||
282 | /* Code here assumes that 'unsigned' is at least 32 bits wide */ | 282 | /* Code here assumes that 'unsigned' is at least 32 bits wide */ |
283 | 283 | ||
284 | const char *const bb_argv_dash[] = { "-", NULL }; | ||
285 | |||
284 | const char *opt_complementary; | 286 | const char *opt_complementary; |
285 | 287 | ||
286 | enum { | 288 | enum { |
diff --git a/libbb/wfopen_input.c b/libbb/wfopen_input.c index 1b4928e1f..a7c1c32f5 100644 --- a/libbb/wfopen_input.c +++ b/libbb/wfopen_input.c | |||
@@ -10,8 +10,6 @@ | |||
10 | /* A number of applets need to open a file for reading, where the filename | 10 | /* A number of applets need to open a file for reading, where the filename |
11 | * is a command line arg. Since often that arg is '-' (meaning stdin), | 11 | * is a command line arg. Since often that arg is '-' (meaning stdin), |
12 | * we avoid testing everywhere by consolidating things in this routine. | 12 | * we avoid testing everywhere by consolidating things in this routine. |
13 | * | ||
14 | * Note: we also consider "" to mean stdin (for 'cmp' at least). | ||
15 | */ | 13 | */ |
16 | 14 | ||
17 | #include "libbb.h" | 15 | #include "libbb.h" |
@@ -21,11 +19,30 @@ FILE *fopen_or_warn_stdin(const char *filename) | |||
21 | FILE *fp = stdin; | 19 | FILE *fp = stdin; |
22 | 20 | ||
23 | if (filename != bb_msg_standard_input | 21 | if (filename != bb_msg_standard_input |
24 | && filename[0] | ||
25 | && NOT_LONE_DASH(filename) | 22 | && NOT_LONE_DASH(filename) |
26 | ) { | 23 | ) { |
27 | fp = fopen_or_warn(filename, "r"); | 24 | fp = fopen_or_warn(filename, "r"); |
28 | } | 25 | } |
29 | |||
30 | return fp; | 26 | return fp; |
31 | } | 27 | } |
28 | |||
29 | FILE *xfopen_stdin(const char *filename) | ||
30 | { | ||
31 | FILE *fp = fopen_or_warn_stdin(filename); | ||
32 | if (fp) | ||
33 | return fp; | ||
34 | xfunc_die(); /* We already output an error message. */ | ||
35 | } | ||
36 | |||
37 | int open_or_warn_stdin(const char *filename) | ||
38 | { | ||
39 | int fd = STDIN_FILENO; | ||
40 | |||
41 | if (filename != bb_msg_standard_input | ||
42 | && NOT_LONE_DASH(filename) | ||
43 | ) { | ||
44 | fd = open_or_warn(filename, O_RDONLY); | ||
45 | } | ||
46 | |||
47 | return fd; | ||
48 | } | ||