diff options
| author | Mark Whitley <markw@lineo.com> | 2001-03-06 20:58:48 +0000 |
|---|---|---|
| committer | Mark Whitley <markw@lineo.com> | 2001-03-06 20:58:48 +0000 |
| commit | 8bb7df49c24de71f28b47eb2e3b86fb5b2e9d42a (patch) | |
| tree | 2bf5d94414a8e6d546514931d25787a4cc9b3df7 | |
| parent | 0f430e34af3471b5267cd6b00a803559ab7fb110 (diff) | |
| download | busybox-8bb7df49c24de71f28b47eb2e3b86fb5b2e9d42a.tar.gz | |
Applied patch from Vladimir Oleynik via Magnus Damm that removes newlines from
error_msg() calls and uses 'return EXIT_SUCCESS' instead of return 0.
| -rw-r--r-- | networking/tftp.c | 20 | ||||
| -rw-r--r-- | tftp.c | 20 |
2 files changed, 16 insertions, 24 deletions
diff --git a/networking/tftp.c b/networking/tftp.c index f56d6562e..e4e01b9a2 100644 --- a/networking/tftp.c +++ b/networking/tftp.c @@ -110,7 +110,7 @@ static inline int tftp(int cmd, struct hostent *host, if ((socketfd = socket(PF_INET, SOCK_DGRAM, 0)) < 0) { perror_msg("socket"); - return 1; + return EXIT_FAILURE; } len = sizeof(sa); @@ -157,7 +157,7 @@ static inline int tftp(int cmd, struct hostent *host, } if ((*cp != '\0') || (&buf[BUFSIZE - 1] - cp) < 7) { - error_msg("too long server-filename.\n"); + error_msg("too long server-filename"); break; } @@ -212,7 +212,7 @@ static inline int tftp(int cmd, struct hostent *host, if (sendto(socketfd, buf, len, 0, (struct sockaddr *) &sa, sizeof(sa)) < 0) { - perror_msg("send()"); + perror_msg("send"); len = -1; break; } @@ -256,7 +256,7 @@ static inline int tftp(int cmd, struct hostent *host, /* discard the packet - treat as timeout */ case 0: - error_msg("timeout.\n"); + error_msg("timeout"); if (!timeout) { timeout = BB_TFTP_NO_RETRIES; @@ -266,7 +266,7 @@ static inline int tftp(int cmd, struct hostent *host, if (!timeout) { len = -1; - error_msg("last timeout!\n"); + error_msg("last timeout"); } break; @@ -333,7 +333,7 @@ static inline int tftp(int cmd, struct hostent *host, } if (msg) { - error_msg("server says: %s.\n", msg); + error_msg("server says: %s", msg); } break; @@ -342,11 +342,7 @@ static inline int tftp(int cmd, struct hostent *host, close(socketfd); - if (finished) { - return 0; - } - - return 1; + return finished ? EXIT_SUCCESS : EXIT_FAILURE; } int tftp_main(int argc, char **argv) @@ -402,7 +398,7 @@ int tftp_main(int argc, char **argv) free(s); } if (bad) { - perror_msg_and_die("bad \"server:file\" combination"); + error_msg_and_die("bad \"server:file\" combination"); } if (BB_TFTP_DEBUG) { @@ -110,7 +110,7 @@ static inline int tftp(int cmd, struct hostent *host, if ((socketfd = socket(PF_INET, SOCK_DGRAM, 0)) < 0) { perror_msg("socket"); - return 1; + return EXIT_FAILURE; } len = sizeof(sa); @@ -157,7 +157,7 @@ static inline int tftp(int cmd, struct hostent *host, } if ((*cp != '\0') || (&buf[BUFSIZE - 1] - cp) < 7) { - error_msg("too long server-filename.\n"); + error_msg("too long server-filename"); break; } @@ -212,7 +212,7 @@ static inline int tftp(int cmd, struct hostent *host, if (sendto(socketfd, buf, len, 0, (struct sockaddr *) &sa, sizeof(sa)) < 0) { - perror_msg("send()"); + perror_msg("send"); len = -1; break; } @@ -256,7 +256,7 @@ static inline int tftp(int cmd, struct hostent *host, /* discard the packet - treat as timeout */ case 0: - error_msg("timeout.\n"); + error_msg("timeout"); if (!timeout) { timeout = BB_TFTP_NO_RETRIES; @@ -266,7 +266,7 @@ static inline int tftp(int cmd, struct hostent *host, if (!timeout) { len = -1; - error_msg("last timeout!\n"); + error_msg("last timeout"); } break; @@ -333,7 +333,7 @@ static inline int tftp(int cmd, struct hostent *host, } if (msg) { - error_msg("server says: %s.\n", msg); + error_msg("server says: %s", msg); } break; @@ -342,11 +342,7 @@ static inline int tftp(int cmd, struct hostent *host, close(socketfd); - if (finished) { - return 0; - } - - return 1; + return finished ? EXIT_SUCCESS : EXIT_FAILURE; } int tftp_main(int argc, char **argv) @@ -402,7 +398,7 @@ int tftp_main(int argc, char **argv) free(s); } if (bad) { - perror_msg_and_die("bad \"server:file\" combination"); + error_msg_and_die("bad \"server:file\" combination"); } if (BB_TFTP_DEBUG) { |