diff options
| author |   florian <florian@openbsd.org> | 2018-07-18 15:46:49 +0000 |
|---|---|---|
| committer | florian <florian@openbsd.org> | 2018-07-18 15:46:49 +0000 |
| commit | 39e11f4ea3d15dcd98f7fc5e12ef4d96186afa28 (patch) | |
| tree | 8de40db8b171185d70a8e444898f502f6beab7ff | |
| parent | Stop our own router advertisements from looping back to us. (diff) | |
| download | wireguard-openbsd-39e11f4ea3d15dcd98f7fc5e12ef4d96186afa28.tar.xz wireguard-openbsd-39e11f4ea3d15dcd98f7fc5e12ef4d96186afa28.zip | |
replace manual zero initialization of various fields with memset;
makes the code shorter and easier to read.
suggested by & OK claudio
| -rw-r--r-- | sbin/ping/ping.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/sbin/ping/ping.c b/sbin/ping/ping.c index 520f3226910..8abf4f61b2b 100644 --- a/sbin/ping/ping.c +++ b/sbin/ping/ping.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ping.c,v 1.226 2018/07/18 13:55:39 florian Exp $ */ +/* $OpenBSD: ping.c,v 1.227 2018/07/18 15:46:49 florian Exp $ */ /* * Copyright (C) 1995, 1996, 1997, and 1998 WIDE Project. @@ -820,6 +820,7 @@ main(int argc, char *argv[]) if (pinger(s) != 0) { (void)signal(SIGALRM, onsignal); timeout = INFTIM; + memset(&itimer, 0, sizeof(itimer)); if (nreceived) { itimer.it_value.tv_sec = 2 * tmax / 1000; @@ -827,9 +828,6 @@ main(int argc, char *argv[]) itimer.it_value.tv_sec = 1; } else itimer.it_value.tv_sec = maxwait; - itimer.it_interval.tv_sec = 0; - itimer.it_interval.tv_usec = 0; - itimer.it_value.tv_usec = 0; (void)setitimer(ITIMER_REAL, &itimer, NULL); /* When the alarm goes off we are done. */ @@ -1008,15 +1006,13 @@ retransmit(int s) * to wait two round-trip times if we've received any packets or * maxwait seconds if we haven't. */ + memset(&itimer, 0, sizeof(itimer)); if (nreceived) { itimer.it_value.tv_sec = 2 * tmax / 1000; if (itimer.it_value.tv_sec == 0) itimer.it_value.tv_sec = 1; } else itimer.it_value.tv_sec = maxwait; - itimer.it_interval.tv_sec = 0; - itimer.it_interval.tv_usec = 0; - itimer.it_value.tv_usec = 0; (void)setitimer(ITIMER_REAL, &itimer, NULL); /* When the alarm goes off we are done. */ |