Remove debugging statement.

[oweals/busybox.git] / tftp.c

diff --git a/tftp.c b/tftp.c
index f56d6562e9917c3d3f9f3a4ae3a66b3e2b43bfbf..bb75c88ecf9e6d2af972aa75e5e1d217a78f8fbc 100644 (file)
--- a/tftp.c
+++ b/tftp.c
@@ -110,13 +110,13 @@ 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);
 
        memset(&sa, 0, len);
-       bind(socketfd, &sa, len);
+       bind(socketfd, (struct sockaddr *)&sa, len);
 
        sa.sin_family = host->h_addrtype;
        sa.sin_port = htons(port);
@@ -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)
@@ -391,19 +387,13 @@ int tftp_main(int argc, char **argv)
 
                serverfile = cp + 1;
 
-               if ((s = strdup(serverstr))) {
-                       s[cp - serverstr] = '\0';
+               s = xstrdup(serverstr);
+               s[cp - serverstr] = '\0';
 
-                       if ((host = gethostbyname(s))) {
-                               bad = 0;
-                       }
-               }
+               host = xgethostbyname(s);
 
                free(s);
        }
-       if (bad) {
-               perror_msg_and_die("bad \"server:file\" combination");
-       }
 
        if (BB_TFTP_DEBUG) {
                printf("using server \"%s\", serverfile \"%s\","