diff options
| author | Florian Obser <florian@cvs.openbsd.org> | 2020-12-01 18:08:54 +0000 |
|---|---|---|
| committer | Florian Obser <florian@cvs.openbsd.org> | 2020-12-01 18:08:54 +0000 |
| commit | b93a1c994aea918478055f149be6440c65622c5a (patch) | |
| tree | 698aabba19924b2a3c3404ea4d8631d0e887d999 /sbin/slaacd/slaacd.c | |
| parent | b59e338847c508a8817e739546e2c142cfb75574 (diff) | |
Sync get_ifrdomain() to rad(8) version by removing a useles (void)
cast in front of a strlcpy(3).
tb@ had pointed out that the use in rad(8) was not consistent.
I consider the void cast a useless annotation, either the strl*
functions must have the canonical trucation check or it must be
obvious that truncation cannot happen or is not a problem.
While at it remove a bunch more casts that snuck in over time.
Diffstat (limited to 'sbin/slaacd/slaacd.c')
| -rw-r--r-- | sbin/slaacd/slaacd.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sbin/slaacd/slaacd.c b/sbin/slaacd/slaacd.c index 37234b43134..2504dcec326 100644 --- a/sbin/slaacd/slaacd.c +++ b/sbin/slaacd/slaacd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: slaacd.c,v 1.52 2020/09/14 09:40:28 florian Exp $ */ +/* $OpenBSD: slaacd.c,v 1.53 2020/12/01 18:08:53 florian Exp $ */ /* * Copyright (c) 2017 Florian Obser <florian@openbsd.org> @@ -675,7 +675,7 @@ configure_interface(struct imsg_configure_address *address) if (address->mtu) { struct ifreq ifr; - (void)strlcpy(ifr.ifr_name, in6_addreq.ifra_name, + strlcpy(ifr.ifr_name, in6_addreq.ifra_name, sizeof(ifr.ifr_name)); ifr.ifr_mtu = address->mtu; log_debug("Setting MTU to %d", ifr.ifr_mtu); |