diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-03-17 09:09:09 +0000 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-03-17 09:09:09 +0000 |
commit | 1d42665b6b0571b9fa5d3b10fbf2dd03382f0ba2 (patch) | |
tree | 7ec8969fc2b07a2b7dc37f96435e5eddf7daf7fd /networking/tftp.c | |
parent | 62a90cdd7435f09f4bb8673e8b7b213067f9d5cc (diff) | |
download | busybox-w32-1d42665b6b0571b9fa5d3b10fbf2dd03382f0ba2.tar.gz busybox-w32-1d42665b6b0571b9fa5d3b10fbf2dd03382f0ba2.tar.bz2 busybox-w32-1d42665b6b0571b9fa5d3b10fbf2dd03382f0ba2.zip |
*: shrink: use Vladimir's "o+" specifier instead of xatou(opt_param)
function old new delta
getopt32 1370 1385 +15
sulogin_main 490 494 +4
realpath_main 84 86 +2
sleep_main 76 77 +1
mt_main 256 257 +1
printenv_main 75 74 -1
fdformat_main 546 545 -1
usleep_main 44 42 -2
setlogcons_main 77 75 -2
ed_main 2654 2649 -5
deallocvt_main 69 64 -5
addgroup_main 373 368 -5
mkfs_minix_main 2989 2982 -7
tail_main 1221 1213 -8
sv_main 1254 1241 -13
du_main 348 328 -20
tftp_main 325 302 -23
split_main 581 558 -23
nc_main 1000 977 -23
diff_main 891 868 -23
arping_main 1797 1770 -27
ls_main 893 847 -46
od_main 2797 2750 -47
readprofile_main 1944 1895 -49
tcpudpsvd_main 1973 1922 -51
udhcpc_main 2590 2513 -77
grep_main 824 722 -102
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 5/22 up/down: 23/-560) Total: -537 bytes
text data bss dec hex filename
796973 658 7428 805059 c48c3 busybox_old
796479 662 7420 804561 c46d1 busybox_unstripped
Diffstat (limited to 'networking/tftp.c')
-rw-r--r-- | networking/tftp.c | 16 |
1 files changed, 5 insertions, 11 deletions
diff --git a/networking/tftp.c b/networking/tftp.c index 14c340b22..187261568 100644 --- a/networking/tftp.c +++ b/networking/tftp.c | |||
@@ -397,9 +397,6 @@ int tftp_main(int argc ATTRIBUTE_UNUSED, char **argv) | |||
397 | len_and_sockaddr *peer_lsa; | 397 | len_and_sockaddr *peer_lsa; |
398 | const char *localfile = NULL; | 398 | const char *localfile = NULL; |
399 | const char *remotefile = NULL; | 399 | const char *remotefile = NULL; |
400 | #if ENABLE_FEATURE_TFTP_BLOCKSIZE | ||
401 | const char *sblocksize = NULL; | ||
402 | #endif | ||
403 | int port; | 400 | int port; |
404 | USE_GETPUT(int cmd;) | 401 | USE_GETPUT(int cmd;) |
405 | int fd = -1; | 402 | int fd = -1; |
@@ -409,13 +406,14 @@ int tftp_main(int argc ATTRIBUTE_UNUSED, char **argv) | |||
409 | 406 | ||
410 | /* -p or -g is mandatory, and they are mutually exclusive */ | 407 | /* -p or -g is mandatory, and they are mutually exclusive */ |
411 | opt_complementary = "" USE_FEATURE_TFTP_GET("g:") USE_FEATURE_TFTP_PUT("p:") | 408 | opt_complementary = "" USE_FEATURE_TFTP_GET("g:") USE_FEATURE_TFTP_PUT("p:") |
412 | USE_GETPUT("?g--p:p--g"); | 409 | USE_GETPUT("g--p:p--g:") |
410 | USE_FEATURE_TFTP_BLOCKSIZE("b+"); | ||
413 | 411 | ||
414 | USE_GETPUT(cmd =) getopt32(argv, | 412 | USE_GETPUT(cmd =) getopt32(argv, |
415 | USE_FEATURE_TFTP_GET("g") USE_FEATURE_TFTP_PUT("p") | 413 | USE_FEATURE_TFTP_GET("g") USE_FEATURE_TFTP_PUT("p") |
416 | "l:r:" USE_FEATURE_TFTP_BLOCKSIZE("b:"), | 414 | "l:r:" USE_FEATURE_TFTP_BLOCKSIZE("b:"), |
417 | &localfile, &remotefile | 415 | &localfile, &remotefile |
418 | USE_FEATURE_TFTP_BLOCKSIZE(, &sblocksize)); | 416 | USE_FEATURE_TFTP_BLOCKSIZE(, &blocksize)); |
419 | argv += optind; | 417 | argv += optind; |
420 | 418 | ||
421 | flags = O_RDONLY; | 419 | flags = O_RDONLY; |
@@ -423,12 +421,8 @@ int tftp_main(int argc ATTRIBUTE_UNUSED, char **argv) | |||
423 | flags = O_WRONLY | O_CREAT | O_TRUNC; | 421 | flags = O_WRONLY | O_CREAT | O_TRUNC; |
424 | 422 | ||
425 | #if ENABLE_FEATURE_TFTP_BLOCKSIZE | 423 | #if ENABLE_FEATURE_TFTP_BLOCKSIZE |
426 | if (sblocksize) { | 424 | if (!tftp_blocksize_check(blocksize, 0)) |
427 | blocksize = xatoi_u(sblocksize); | 425 | return EXIT_FAILURE; |
428 | if (!tftp_blocksize_check(blocksize, 0)) { | ||
429 | return EXIT_FAILURE; | ||
430 | } | ||
431 | } | ||
432 | #endif | 426 | #endif |
433 | 427 | ||
434 | if (!localfile) | 428 | if (!localfile) |