From 40eaa9f0bb2c689552e39890592ed773d870c1fa Mon Sep 17 00:00:00 2001 From: Eric Andersen Date: Mon, 19 Mar 2001 19:41:54 +0000 Subject: [PATCH] Size optimization for rdate from Vladimir --- rdate.c | 56 ++++++++++++++++++---------------------------- util-linux/rdate.c | 56 ++++++++++++++++++---------------------------- 2 files changed, 44 insertions(+), 68 deletions(-) diff --git a/rdate.c b/rdate.c index 28e87323d..5f31282fc 100644 --- a/rdate.c +++ b/rdate.c @@ -33,16 +33,11 @@ #include #include #include "busybox.h" -#define BB_DECLARE_EXTERN -#include "messages.c" static const int RFC_868_BIAS = 2208988800UL; -static int setdate= 0; -static int printdate= 0; - -static time_t askremotedate(char *host) +static time_t askremotedate(const char *host) { struct hostent *h; struct sockaddr_in sin; @@ -50,32 +45,25 @@ static time_t askremotedate(char *host) unsigned long int nett, localt; int fd; - if (!(h = gethostbyname(host))) { /* get the IP addr */ - perror_msg("%s", host); - return(-1); - } - if ((tserv = getservbyname("time", "tcp")) == NULL) { /* find port # */ - perror_msg("%s", "time"); - return(-1); - } - if ((fd = socket(AF_INET, SOCK_STREAM, 0)) < 0) { /* get net connection */ - perror_msg("%s", "socket"); - return(-1); - } + if (!(h = gethostbyname(host))) /* get the IP addr */ + perror_msg_and_die("%s", host); + + if ((tserv = getservbyname("time", "tcp")) == NULL) /* find port # */ + perror_msg_and_die("%s", "time"); + + if ((fd = socket(AF_INET, SOCK_STREAM, 0)) < 0) /* get net connection */ + perror_msg_and_die("%s", "socket"); memcpy(&sin.sin_addr, h->h_addr, sizeof(sin.sin_addr)); sin.sin_port= tserv->s_port; sin.sin_family = AF_INET; - if (connect(fd, (struct sockaddr *)&sin, sizeof(sin)) < 0) { /* connect to time server */ - perror_msg("%s", host); - close(fd); - return(-1); - } - if (read(fd, (void *)&nett, 4) != 4) { /* read time from server */ - close(fd); - error_msg("%s did not send the complete time", host); - } + if (connect(fd, (struct sockaddr *)&sin, sizeof(sin)) < 0) /* connect to time server */ + perror_msg_and_die("%s", host); + + if (read(fd, (void *)&nett, 4) != 4) /* read time from server */ + error_msg_and_die("%s did not send the complete time", host); + close(fd); /* convert from network byte order to local byte order. @@ -93,6 +81,8 @@ int rdate_main(int argc, char **argv) { time_t time; int opt; + int setdate = 0; + int printdate= 0; /* Interpret command line args */ /* do special-case option parsing */ @@ -118,20 +108,18 @@ int rdate_main(int argc, char **argv) /* the default action is to set the date */ if (printdate==0 && setdate==0) setdate++; - if (optind == argc) { + if (optind == argc) show_usage(); - } - if ((time= askremotedate(argv[optind])) == (time_t)-1) { - return EXIT_FAILURE; - } + time = askremotedate(argv[optind]); + if (setdate) { if (stime(&time) < 0) perror_msg_and_die("Could not set time of day"); } - if (printdate) { + + if (printdate) printf("%s", ctime(&time)); - } return EXIT_SUCCESS; } diff --git a/util-linux/rdate.c b/util-linux/rdate.c index 28e87323d..5f31282fc 100644 --- a/util-linux/rdate.c +++ b/util-linux/rdate.c @@ -33,16 +33,11 @@ #include #include #include "busybox.h" -#define BB_DECLARE_EXTERN -#include "messages.c" static const int RFC_868_BIAS = 2208988800UL; -static int setdate= 0; -static int printdate= 0; - -static time_t askremotedate(char *host) +static time_t askremotedate(const char *host) { struct hostent *h; struct sockaddr_in sin; @@ -50,32 +45,25 @@ static time_t askremotedate(char *host) unsigned long int nett, localt; int fd; - if (!(h = gethostbyname(host))) { /* get the IP addr */ - perror_msg("%s", host); - return(-1); - } - if ((tserv = getservbyname("time", "tcp")) == NULL) { /* find port # */ - perror_msg("%s", "time"); - return(-1); - } - if ((fd = socket(AF_INET, SOCK_STREAM, 0)) < 0) { /* get net connection */ - perror_msg("%s", "socket"); - return(-1); - } + if (!(h = gethostbyname(host))) /* get the IP addr */ + perror_msg_and_die("%s", host); + + if ((tserv = getservbyname("time", "tcp")) == NULL) /* find port # */ + perror_msg_and_die("%s", "time"); + + if ((fd = socket(AF_INET, SOCK_STREAM, 0)) < 0) /* get net connection */ + perror_msg_and_die("%s", "socket"); memcpy(&sin.sin_addr, h->h_addr, sizeof(sin.sin_addr)); sin.sin_port= tserv->s_port; sin.sin_family = AF_INET; - if (connect(fd, (struct sockaddr *)&sin, sizeof(sin)) < 0) { /* connect to time server */ - perror_msg("%s", host); - close(fd); - return(-1); - } - if (read(fd, (void *)&nett, 4) != 4) { /* read time from server */ - close(fd); - error_msg("%s did not send the complete time", host); - } + if (connect(fd, (struct sockaddr *)&sin, sizeof(sin)) < 0) /* connect to time server */ + perror_msg_and_die("%s", host); + + if (read(fd, (void *)&nett, 4) != 4) /* read time from server */ + error_msg_and_die("%s did not send the complete time", host); + close(fd); /* convert from network byte order to local byte order. @@ -93,6 +81,8 @@ int rdate_main(int argc, char **argv) { time_t time; int opt; + int setdate = 0; + int printdate= 0; /* Interpret command line args */ /* do special-case option parsing */ @@ -118,20 +108,18 @@ int rdate_main(int argc, char **argv) /* the default action is to set the date */ if (printdate==0 && setdate==0) setdate++; - if (optind == argc) { + if (optind == argc) show_usage(); - } - if ((time= askremotedate(argv[optind])) == (time_t)-1) { - return EXIT_FAILURE; - } + time = askremotedate(argv[optind]); + if (setdate) { if (stime(&time) < 0) perror_msg_and_die("Could not set time of day"); } - if (printdate) { + + if (printdate) printf("%s", ctime(&time)); - } return EXIT_SUCCESS; }