Skip to content

Commit 4f7d98a

Browse files
committed
Make implicit conversions explicite
1 parent 7964ca0 commit 4f7d98a

File tree

1 file changed

+21
-21
lines changed

1 file changed

+21
-21
lines changed

htpdate.c

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -198,15 +198,15 @@ static void splitURL(char **scheme, char **host, char **port, char **path) {
198198
}
199199

200200

201-
static void swuid(int id) {
201+
static void swuid(unsigned int id) {
202202
if (seteuid(id)) {
203203
printlog(1, "seteuid() %i", id);
204204
exit(1);
205205
}
206206
}
207207

208208

209-
static void swgid(int id) {
209+
static void swgid(unsigned int id) {
210210
if (setegid(id)) {
211211
printlog(1, "setegid() %i", id);
212212
exit(1);
@@ -231,7 +231,7 @@ static long long getoffset(char remote_time[25]) {
231231

232232

233233
static int sendHEAD(int server_s, char *headrequest, char *buffer) {
234-
int ret = send(server_s, headrequest, strlen(headrequest), 0);
234+
int ret = (int)send(server_s, headrequest, strlen(headrequest), 0);
235235

236236
if (ret < 0) {
237237
printlog(1, "Error sending");
@@ -397,7 +397,7 @@ static double getHTTPdate(
397397
#endif
398398

399399
long long offset = 0, first_offset = 0, prev_offset = 0;
400-
long nap = 1e9L;
400+
long nap = 1000000000;
401401
long latency = 0;
402402
long when = nap >> precision;
403403
do {
@@ -415,7 +415,7 @@ static double getHTTPdate(
415415
if (when - latency >= now.tv_nsec) {
416416
sleepspec.tv_nsec = when - now.tv_nsec - latency;
417417
} else {
418-
sleepspec.tv_nsec = 1e9 + when - now.tv_nsec - latency;
418+
sleepspec.tv_nsec = 1000000000 + when - now.tv_nsec - latency;
419419
rtt++;
420420
}
421421

@@ -437,7 +437,7 @@ static double getHTTPdate(
437437
clock_gettime(CLOCK_REALTIME, &now);
438438

439439
/* rtt contains round trip time in nanoseconds */
440-
rtt = (now.tv_sec - rtt) * 1e9 + now.tv_nsec - when + latency;
440+
rtt = (now.tv_sec - rtt) * 1000000000 + now.tv_nsec - when + latency;
441441

442442
/* Obtain rtt/latency first */
443443
if (latency == 0) {
@@ -487,15 +487,15 @@ static double getHTTPdate(
487487
/* Rounding */
488488
if (debug) printlog(0, "when: %ld, nap: %ld", when, nap);
489489
if (offset == LLONG_MAX) return(ERR_TIMESTAMP);
490-
if (when + nap == 1e9 && offset == 0) return 0;
490+
if (when + nap == 1000000000 && offset == 0) return 0;
491491

492492
/* Return the time delta between web server time (timevalue)
493493
and system time (now)
494494
*/
495495
if (first_offset < 0) {
496-
return(-first_offset + (1e9L-when)/(double)1e9L);
496+
return((double)-first_offset + (1000000000-(double)when)/1000000000);
497497
} else {
498-
return(-first_offset + 1 - when/(double)1e9L);
498+
return((double)-first_offset + 1 - ((double)when/1000000000));
499499
}
500500
}
501501

@@ -527,7 +527,7 @@ static int setclock(double timedelta, int setmode) {
527527
return(0);
528528
case 1: /* Adjust time smoothly */
529529
timeofday.tv_sec = (long)timedelta;
530-
timeofday.tv_usec = (long)((timedelta - timeofday.tv_sec) * 1e6);
530+
timeofday.tv_usec = (long)((timedelta - (double)timeofday.tv_sec) * 1e6);
531531

532532
printlog(0, "Adjusting %.1f ms", timedelta * 1e3);
533533

@@ -538,10 +538,10 @@ static int setclock(double timedelta, int setmode) {
538538
printlog(0, "Setting %.3f seconds", timedelta);
539539

540540
clock_gettime(CLOCK_REALTIME, &now);
541-
timedelta += (now.tv_sec + now.tv_nsec * 1e-9);
541+
timedelta += (double)now.tv_sec + (double)now.tv_nsec * 1e-9;
542542

543543
now.tv_sec = (long)timedelta;
544-
now.tv_nsec = (long)(timedelta - now.tv_sec) * 1e9;
544+
now.tv_nsec = (long)((timedelta - (double)now.tv_sec) * 1e9);
545545

546546
strftime(buffer, sizeof(buffer), "%c", localtime(&now.tv_sec));
547547
printlog(0, "Set time: %s", buffer);
@@ -597,7 +597,7 @@ static int htpdate_adjtimex(double drift, char *driftfile, float confidence) {
597597
adjtimex(&tmx);
598598

599599
/* Weighted average of current and new frequency */
600-
tmx.freq = tmx.freq + 65536e6 * drift * confidence;
600+
tmx.freq = tmx.freq + (long int)(65536e6 * drift * confidence);
601601
if ((tmx.freq < -MAX_DRIFT) || (tmx.freq > MAX_DRIFT))
602602
tmx.freq = sign(tmx.freq) * MAX_DRIFT;
603603

@@ -734,10 +734,10 @@ int main(int argc, char *argv[]) {
734734
int noproxyenv = 0;
735735
int ipversion = DEFAULT_IP_VERSION;
736736
long long timelimit = DEFAULT_TIME_LIMIT;
737-
int minsleep = DEFAULT_MIN_SLEEP;
738-
int maxsleep = DEFAULT_MAX_SLEEP;
739-
int sleeptime = minsleep;
740-
int sw_uid = 0, sw_gid = 0;
737+
unsigned int minsleep = DEFAULT_MIN_SLEEP;
738+
unsigned int maxsleep = DEFAULT_MAX_SLEEP;
739+
unsigned int sleeptime = minsleep;
740+
unsigned int sw_gid = 0, sw_uid = 0;
741741
time_t starttime = 0;
742742

743743
struct passwd *pw;
@@ -783,7 +783,7 @@ int main(int argc, char *argv[]) {
783783
logmode = 1;
784784
break;
785785
case 'm': /* minimum poll interval */
786-
if ((minsleep = atoi(optarg)) <= 0) {
786+
if ((minsleep = (unsigned int)atoi(optarg)) <= 0) {
787787
fputs("Invalid sleep time\n", stderr);
788788
exit(1);
789789
}
@@ -847,7 +847,7 @@ int main(int argc, char *argv[]) {
847847
logmode = 2;
848848
break;
849849
case 'M': /* maximum poll interval */
850-
if ((maxsleep = atoi(optarg)) <= 0) {
850+
if ((maxsleep = (unsigned int)atoi(optarg)) <= 0) {
851851
fputs("Invalid sleep time\n", stderr);
852852
exit(1);
853853
}
@@ -983,7 +983,7 @@ int main(int argc, char *argv[]) {
983983
if (daemonize || foreground) {
984984
if (starttime) {
985985
/* Calculate systematic clock drift */
986-
drift = timeavg / (time(NULL) - starttime);
986+
drift = timeavg / (double)(time(NULL) - starttime);
987987
printlog(0, "Drift %.2f PPM, %.2f s/day", drift*1e6, drift*86400);
988988

989989
/* Adjust system clock */
@@ -1003,7 +1003,7 @@ int main(int argc, char *argv[]) {
10031003
sleeptime = minsleep;
10041004

10051005
/* Sleep for some time after a time adjust or set */
1006-
sleep(fabs(timeavg*2000));
1006+
sleep((unsigned int)fabs(timeavg*2000));
10071007
}
10081008
} else {
10091009
/* Increase polling interval */

0 commit comments

Comments
 (0)