diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-03-17 09:07:36 +0000 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-03-17 09:07:36 +0000 |
commit | 62a90cdd7435f09f4bb8673e8b7b213067f9d5cc (patch) | |
tree | ac7f86c4de0ce0c095b59820c7ba238369c65f71 /coreutils/md5_sha1_sum.c | |
parent | d02db892440499a072ea8c83c2370e9d9c103098 (diff) | |
download | busybox-w32-62a90cdd7435f09f4bb8673e8b7b213067f9d5cc.tar.gz busybox-w32-62a90cdd7435f09f4bb8673e8b7b213067f9d5cc.tar.bz2 busybox-w32-62a90cdd7435f09f4bb8673e8b7b213067f9d5cc.zip |
*: shrink by using [f]open_or_warn_stdin where appropriate
function old new delta
lsattr_main 62 143 +81
open_or_warn_stdin - 36 +36
fclose_if_not_stdin 20 47 +27
xfopen_stdin - 20 +20
tac_main 336 356 +20
cksum_main 249 259 +10
bb_argv_dash - 8 +8
su_main 448 455 +7
cmp_main 630 633 +3
passwd_main 1072 1074 +2
uudecode_main 317 315 -2
text_yank 110 108 -2
handle_incoming_and_exit 2653 2651 -2
flags 5 1 -4
write_leases 235 230 -5
fopen_or_warn_stdin 48 42 -6
fold_main 648 642 -6
static.argv_dash 8 - -8
sum_main 142 128 -14
tail_main 1237 1221 -16
sed_main 711 695 -16
cmp_xfopen_input 17 - -17
bb_cat 113 96 -17
catv_main 328 306 -22
strings_main 457 434 -23
hash_file 298 274 -24
sum_file 353 325 -28
sort_main 904 859 -45
expand_main 736 686 -50
cut_main 1116 1065 -51
md5_sha1_sum_main 549 493 -56
lsattr_args 90 - -90
read_stduu 408 255 -153
------------------------------------------------------------------------------
(add/remove: 3/3 grow/shrink: 7/20 up/down: 214/-657) Total: -443 bytes
text data bss dec hex filename
797417 658 7428 805503 c4a7f busybox_old
796973 658 7428 805059 c48c3 busybox_unstripped
Diffstat (limited to 'coreutils/md5_sha1_sum.c')
-rw-r--r-- | coreutils/md5_sha1_sum.c | 37 |
1 files changed, 14 insertions, 23 deletions
diff --git a/coreutils/md5_sha1_sum.c b/coreutils/md5_sha1_sum.c index e94f2ceb1..080eac5f4 100644 --- a/coreutils/md5_sha1_sum.c +++ b/coreutils/md5_sha1_sum.c | |||
@@ -36,12 +36,9 @@ static uint8_t *hash_file(const char *filename, hash_algo_t hash_algo) | |||
36 | void (*update)(const void*, size_t, void*); | 36 | void (*update)(const void*, size_t, void*); |
37 | void (*final)(void*, void*); | 37 | void (*final)(void*, void*); |
38 | 38 | ||
39 | src_fd = STDIN_FILENO; | 39 | src_fd = open_or_warn_stdin(filename); |
40 | if (NOT_LONE_DASH(filename)) { | 40 | if (src_fd < 0) { |
41 | src_fd = open_or_warn(filename, O_RDONLY); | 41 | return NULL; |
42 | if (src_fd < 0) { | ||
43 | return NULL; | ||
44 | } | ||
45 | } | 42 | } |
46 | 43 | ||
47 | /* figure specific hash algorithims */ | 44 | /* figure specific hash algorithims */ |
@@ -78,7 +75,7 @@ static uint8_t *hash_file(const char *filename, hash_algo_t hash_algo) | |||
78 | } | 75 | } |
79 | 76 | ||
80 | int md5_sha1_sum_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; | 77 | int md5_sha1_sum_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; |
81 | int md5_sha1_sum_main(int argc, char **argv) | 78 | int md5_sha1_sum_main(int argc ATTRIBUTE_UNUSED, char **argv) |
82 | { | 79 | { |
83 | int return_value = EXIT_SUCCESS; | 80 | int return_value = EXIT_SUCCESS; |
84 | uint8_t *hash_value; | 81 | uint8_t *hash_value; |
@@ -90,6 +87,10 @@ int md5_sha1_sum_main(int argc, char **argv) | |||
90 | if (ENABLE_FEATURE_MD5_SHA1_SUM_CHECK) | 87 | if (ENABLE_FEATURE_MD5_SHA1_SUM_CHECK) |
91 | flags = getopt32(argv, "scw"); | 88 | flags = getopt32(argv, "scw"); |
92 | else optind = 1; | 89 | else optind = 1; |
90 | argv += optind; | ||
91 | //argc -= optind; | ||
92 | if (!*argv) | ||
93 | *--argv = (char*)"-"; | ||
93 | 94 | ||
94 | if (ENABLE_FEATURE_MD5_SHA1_SUM_CHECK && !(flags & FLAG_CHECK)) { | 95 | if (ENABLE_FEATURE_MD5_SHA1_SUM_CHECK && !(flags & FLAG_CHECK)) { |
95 | if (flags & FLAG_SILENT) { | 96 | if (flags & FLAG_SILENT) { |
@@ -101,26 +102,18 @@ int md5_sha1_sum_main(int argc, char **argv) | |||
101 | } | 102 | } |
102 | } | 103 | } |
103 | 104 | ||
104 | if (argc == optind) { | ||
105 | argv[argc++] = (char*)"-"; | ||
106 | } | ||
107 | |||
108 | if (ENABLE_FEATURE_MD5_SHA1_SUM_CHECK && (flags & FLAG_CHECK)) { | 105 | if (ENABLE_FEATURE_MD5_SHA1_SUM_CHECK && (flags & FLAG_CHECK)) { |
109 | FILE *pre_computed_stream; | 106 | FILE *pre_computed_stream; |
110 | int count_total = 0; | 107 | int count_total = 0; |
111 | int count_failed = 0; | 108 | int count_failed = 0; |
112 | char *file_ptr = argv[optind]; | ||
113 | char *line; | 109 | char *line; |
114 | 110 | ||
115 | if (optind + 1 != argc) { | 111 | if (argv[1]) { |
116 | bb_error_msg_and_die | 112 | bb_error_msg_and_die |
117 | ("only one argument may be specified when using -c"); | 113 | ("only one argument may be specified when using -c"); |
118 | } | 114 | } |
119 | 115 | ||
120 | pre_computed_stream = stdin; | 116 | pre_computed_stream = xfopen_stdin(argv[0]); |
121 | if (NOT_LONE_DASH(file_ptr)) { | ||
122 | pre_computed_stream = xfopen(file_ptr, "r"); | ||
123 | } | ||
124 | 117 | ||
125 | while ((line = xmalloc_getline(pre_computed_stream)) != NULL) { | 118 | while ((line = xmalloc_getline(pre_computed_stream)) != NULL) { |
126 | char *filename_ptr; | 119 | char *filename_ptr; |
@@ -168,17 +161,15 @@ int md5_sha1_sum_main(int argc, char **argv) | |||
168 | } | 161 | } |
169 | */ | 162 | */ |
170 | } else { | 163 | } else { |
171 | while (optind < argc) { | 164 | do { |
172 | char *file_ptr = argv[optind++]; | 165 | hash_value = hash_file(*argv, hash_algo); |
173 | |||
174 | hash_value = hash_file(file_ptr, hash_algo); | ||
175 | if (hash_value == NULL) { | 166 | if (hash_value == NULL) { |
176 | return_value = EXIT_FAILURE; | 167 | return_value = EXIT_FAILURE; |
177 | } else { | 168 | } else { |
178 | printf("%s %s\n", hash_value, file_ptr); | 169 | printf("%s %s\n", hash_value, *argv); |
179 | free(hash_value); | 170 | free(hash_value); |
180 | } | 171 | } |
181 | } | 172 | } while (*++argv); |
182 | } | 173 | } |
183 | return return_value; | 174 | return return_value; |
184 | } | 175 | } |