diff options
| author |   krw <krw@openbsd.org> | 2020-11-21 20:36:39 +0000 |
|---|---|---|
| committer | krw <krw@openbsd.org> | 2020-11-21 20:36:39 +0000 |
| commit | b2c5fc662a8828f082a7e5717b7689f0f95dd32a (patch) | |
| tree | 060143e49fa172f05cc9c23e6b23ee5feea13d94 /sbin/dhclient | |
| parent | Fix ikev2_nat_detection(). (diff) | |
| download | wireguard-openbsd-b2c5fc662a8828f082a7e5717b7689f0f95dd32a.tar.xz wireguard-openbsd-b2c5fc662a8828f082a7e5717b7689f0f95dd32a.zip | |
No need to update ifi->state just before setting quit to TERMINATE.
Check quit != TERMINATE before trying to open leases file.
Diffstat (limited to 'sbin/dhclient')
| -rw-r--r-- | sbin/dhclient/dhclient.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/sbin/dhclient/dhclient.c b/sbin/dhclient/dhclient.c index beb5d65459f..8ad849951a1 100644 --- a/sbin/dhclient/dhclient.c +++ b/sbin/dhclient/dhclient.c @@ -1,4 +1,4 @@ -/* $OpenBSD: dhclient.c,v 1.685 2020/11/21 18:34:25 krw Exp $ */ +/* $OpenBSD: dhclient.c,v 1.686 2020/11/21 20:36:39 krw Exp $ */ /* * Copyright 2004 Henning Brauer <henning@openbsd.org> @@ -367,12 +367,10 @@ rtm_dispatch(struct interface_info *ifi, struct rt_msghdr *rtm) } else if ((ifi->flags & IFI_IN_CHARGE) != 0) { log_debug("%s: yielding responsibility", log_procname); - ifi->state = S_PREBOOT; quit = TERMINATE; } } else if ((rtm->rtm_flags & RTF_PROTO2) != 0) { release_lease(ifi); /* OK even if we sent it. */ - ifi->state = S_PREBOOT; quit = TERMINATE; } else return; /* Ignore tell_unwind() proposals. */ @@ -2539,7 +2537,7 @@ take_charge(struct interface_info *ifi, int routefd, char *leasespath) if (nfds == 1 && (fds[0].revents & POLLIN) == POLLIN) routefd_handler(ifi, routefd); - if ((ifi->flags & IFI_IN_CHARGE) == IFI_IN_CHARGE) { + if (quit != TERMINATE && (ifi->flags & IFI_IN_CHARGE) == IFI_IN_CHARGE) { fd = open(leasespath, O_NONBLOCK | O_RDONLY|O_EXLOCK|O_CREAT|O_NOFOLLOW, 0640); if (fd == -1 && errno != EWOULDBLOCK) |