diff options
| author |   krw <krw@openbsd.org> | 2013-01-13 04:51:28 +0000 |
|---|---|---|
| committer | krw <krw@openbsd.org> | 2013-01-13 04:51:28 +0000 |
| commit | 9863900ab34255fc6752320b4b2bc06db6418bd1 (patch) | |
| tree | 84ad827f98b4dc27207ad99303f582f01aaefaa3 | |
| parent | Destatic. (diff) | |
| download | wireguard-openbsd-9863900ab34255fc6752320b4b2bc06db6418bd1.tar.xz wireguard-openbsd-9863900ab34255fc6752320b4b2bc06db6418bd1.zip | |
Check results of strdup() of domainname and nameservers options.
Original diff from lteo@, tweaked by me. ok lteo@
| -rw-r--r-- | sbin/dhclient/dhclient.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/sbin/dhclient/dhclient.c b/sbin/dhclient/dhclient.c index 4b553293bce..41ffd12055c 100644 --- a/sbin/dhclient/dhclient.c +++ b/sbin/dhclient/dhclient.c @@ -1,4 +1,4 @@ -/* $OpenBSD: dhclient.c,v 1.202 2013/01/06 15:33:12 krw Exp $ */ +/* $OpenBSD: dhclient.c,v 1.203 2013/01/13 04:51:28 krw Exp $ */ /* * Copyright 2004 Henning Brauer <henning@openbsd.org> @@ -718,12 +718,17 @@ bind_lease(void) DHO_DOMAIN_NAME, &options[DHO_DOMAIN_NAME], 0)); else domainname = strdup(""); + if (domainname == NULL) + error("no memory for domainname"); + if (options[DHO_DOMAIN_NAME_SERVERS].len) { nameservers = strdup(pretty_print_option( DHO_DOMAIN_NAME_SERVERS, &options[DHO_DOMAIN_NAME_SERVERS], 0)); } else nameservers = strdup(""); + if (nameservers == NULL) + error("no memory for nameservers"); new_resolv_conf(ifi->name, domainname, nameservers); |