summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorkrw <krw@openbsd.org>2018-07-08 17:15:07 +0000
committerkrw <krw@openbsd.org>2018-07-08 17:15:07 +0000
commit3c12ada049ec8174546d7ebd54606e5a13c3e487 (patch)
tree624e17f8db98eb3aad46e23c1fe06b1fdd8649a2
parentdocument what happens when the @define-tag defining package gets removed (diff)
downloadwireguard-openbsd-3c12ada049ec8174546d7ebd54606e5a13c3e487.tar.xz
wireguard-openbsd-3c12ada049ec8174546d7ebd54606e5a13c3e487.zip
Be consistent in warn() and log_warn() usage when
running out of memory. Next step, be correct *and* consistent. ok dennis@ tb@ benno@ schwarze@
-rw-r--r--sbin/iked/parse.y6
-rw-r--r--sbin/ipsecctl/parse.y6
-rw-r--r--sbin/pfctl/parse.y8
-rw-r--r--usr.sbin/acme-client/parse.y4
-rw-r--r--usr.sbin/bgpd/parse.y6
-rw-r--r--usr.sbin/dvmrpd/parse.y6
-rw-r--r--usr.sbin/ifstated/parse.y4
-rw-r--r--usr.sbin/iscsictl/parse.y6
-rw-r--r--usr.sbin/ldapd/parse.y6
-rw-r--r--usr.sbin/npppd/npppd/parse.y6
-rw-r--r--usr.sbin/ntpd/parse.y6
-rw-r--r--usr.sbin/ospf6d/parse.y6
-rw-r--r--usr.sbin/ospfd/parse.y6
-rw-r--r--usr.sbin/radiusd/parse.y6
-rw-r--r--usr.sbin/ripd/parse.y6
-rw-r--r--usr.sbin/snmpd/parse.y6
-rw-r--r--usr.sbin/switchd/parse.y6
-rw-r--r--usr.sbin/vmd/parse.y6
-rw-r--r--usr.sbin/ypldap/parse.y8
19 files changed, 58 insertions, 56 deletions
diff --git a/sbin/iked/parse.y b/sbin/iked/parse.y
index 8a817d163b7..25c08cd766b 100644
--- a/sbin/iked/parse.y
+++ b/sbin/iked/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.72 2018/06/11 10:05:17 denis Exp $ */
+/* $OpenBSD: parse.y,v 1.73 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2010-2013 Reyk Floeter <reyk@openbsd.org>
@@ -1503,11 +1503,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- warn("malloc");
+ warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- warn("malloc");
+ warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/sbin/ipsecctl/parse.y b/sbin/ipsecctl/parse.y
index d9739724d32..e022bf55dd7 100644
--- a/sbin/ipsecctl/parse.y
+++ b/sbin/ipsecctl/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.170 2018/04/26 14:12:19 krw Exp $ */
+/* $OpenBSD: parse.y,v 1.171 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2002, 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -1300,11 +1300,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- warn("malloc");
+ warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- warn("malloc");
+ warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/sbin/pfctl/parse.y b/sbin/pfctl/parse.y
index 3ce2fda94c1..98e4854dbfe 100644
--- a/sbin/pfctl/parse.y
+++ b/sbin/pfctl/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.673 2018/05/18 13:39:49 benno Exp $ */
+/* $OpenBSD: parse.y,v 1.674 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2001 Markus Friedl. All rights reserved.
@@ -5549,9 +5549,11 @@ pushfile(const char *name, int secret)
if ((nfile = calloc(1, sizeof(struct file))) == NULL ||
(nfile->name = strdup(name)) == NULL) {
- if (nfile)
+ if (nfile) {
+ warn("strdup");
free(nfile);
- warn("malloc");
+ } else
+ warn("calloc");
return (NULL);
}
if (TAILQ_FIRST(&files) == NULL && strcmp(nfile->name, "-") == 0) {
diff --git a/usr.sbin/acme-client/parse.y b/usr.sbin/acme-client/parse.y
index 2c14ca31bcf..d5666bd2123 100644
--- a/usr.sbin/acme-client/parse.y
+++ b/usr.sbin/acme-client/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.22 2018/06/17 19:28:36 denis Exp $ */
+/* $OpenBSD: parse.y,v 1.23 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2016 Kristaps Dzonsons <kristaps@bsd.lv>
@@ -703,7 +703,7 @@ pushfile(const char *name)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- warn("malloc");
+ warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
diff --git a/usr.sbin/bgpd/parse.y b/usr.sbin/bgpd/parse.y
index ff1bf077c11..7377123f04d 100644
--- a/usr.sbin/bgpd/parse.y
+++ b/usr.sbin/bgpd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.323 2018/06/13 09:44:59 job Exp $ */
+/* $OpenBSD: parse.y,v 1.324 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2002, 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -2894,11 +2894,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/dvmrpd/parse.y b/usr.sbin/dvmrpd/parse.y
index a0586ef5620..d0fd56e4a14 100644
--- a/usr.sbin/dvmrpd/parse.y
+++ b/usr.sbin/dvmrpd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.33 2018/04/26 14:12:19 krw Exp $ */
+/* $OpenBSD: parse.y,v 1.34 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2004, 2005, 2006 Esben Norby <norby@openbsd.org>
@@ -688,11 +688,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/ifstated/parse.y b/usr.sbin/ifstated/parse.y
index 12adbfa66f6..00047cc50fe 100644
--- a/usr.sbin/ifstated/parse.y
+++ b/usr.sbin/ifstated/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.49 2018/04/26 14:12:19 krw Exp $ */
+/* $OpenBSD: parse.y,v 1.50 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2004 Ryan McBride <mcbride@openbsd.org>
@@ -682,7 +682,7 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- warn("malloc");
+ warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
diff --git a/usr.sbin/iscsictl/parse.y b/usr.sbin/iscsictl/parse.y
index 4964afe3a0d..fc9b294b366 100644
--- a/usr.sbin/iscsictl/parse.y
+++ b/usr.sbin/iscsictl/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.12 2018/06/11 09:39:58 denis Exp $ */
+/* $OpenBSD: parse.y,v 1.13 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2010 David Gwynne <dlg@openbsd.org>
@@ -652,11 +652,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- warn("malloc");
+ warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- warn("malloc");
+ warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/ldapd/parse.y b/usr.sbin/ldapd/parse.y
index 33da592ad4d..509e5ed30a5 100644
--- a/usr.sbin/ldapd/parse.y
+++ b/usr.sbin/ldapd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.29 2018/06/11 18:19:03 denis Exp $ */
+/* $OpenBSD: parse.y,v 1.30 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2009, 2010 Martin Hedenfalk <martinh@openbsd.org>
@@ -769,11 +769,11 @@ pushfile(const char *name, int secret)
log_debug("parsing config %s", name);
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/npppd/npppd/parse.y b/usr.sbin/npppd/npppd/parse.y
index 2872c71f3b0..1c7042f3cae 100644
--- a/usr.sbin/npppd/npppd/parse.y
+++ b/usr.sbin/npppd/npppd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.19 2017/08/12 11:20:34 goda Exp $ */
+/* $OpenBSD: parse.y,v 1.20 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2002, 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -1329,11 +1329,11 @@ pushfile(const char *name)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/ntpd/parse.y b/usr.sbin/ntpd/parse.y
index 41fa9ba1fc6..52e30138311 100644
--- a/usr.sbin/ntpd/parse.y
+++ b/usr.sbin/ntpd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.67 2017/08/11 19:12:21 naddy Exp $ */
+/* $OpenBSD: parse.y,v 1.68 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2002, 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -733,11 +733,11 @@ pushfile(const char *name)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/ospf6d/parse.y b/usr.sbin/ospf6d/parse.y
index 41fa137b483..33a3ca8f1fd 100644
--- a/usr.sbin/ospf6d/parse.y
+++ b/usr.sbin/ospf6d/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.32 2018/06/11 18:22:56 denis Exp $ */
+/* $OpenBSD: parse.y,v 1.33 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2004, 2005 Esben Norby <norby@openbsd.org>
@@ -882,11 +882,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/ospfd/parse.y b/usr.sbin/ospfd/parse.y
index ee05e2ea71b..2f1def07ef6 100644
--- a/usr.sbin/ospfd/parse.y
+++ b/usr.sbin/ospfd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.88 2018/06/11 18:21:47 denis Exp $ */
+/* $OpenBSD: parse.y,v 1.89 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2004, 2005 Esben Norby <norby@openbsd.org>
@@ -1127,11 +1127,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/radiusd/parse.y b/usr.sbin/radiusd/parse.y
index 131473b4be1..20caf10ec23 100644
--- a/usr.sbin/radiusd/parse.y
+++ b/usr.sbin/radiusd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.4 2016/08/27 09:04:20 yasuoka Exp $ */
+/* $OpenBSD: parse.y,v 1.5 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2002, 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -661,11 +661,11 @@ pushfile(const char *name)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/ripd/parse.y b/usr.sbin/ripd/parse.y
index 5c9ebec2542..ef6e4e95e8b 100644
--- a/usr.sbin/ripd/parse.y
+++ b/usr.sbin/ripd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.38 2018/04/26 14:12:19 krw Exp $ */
+/* $OpenBSD: parse.y,v 1.39 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2006 Michele Marchetto <mydecay@openbeer.it>
@@ -713,11 +713,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/snmpd/parse.y b/usr.sbin/snmpd/parse.y
index 2c9c7edb09b..35e6c62bcba 100644
--- a/usr.sbin/snmpd/parse.y
+++ b/usr.sbin/snmpd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.48 2018/06/11 18:16:45 denis Exp $ */
+/* $OpenBSD: parse.y,v 1.49 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2007, 2008, 2012 Reyk Floeter <reyk@openbsd.org>
@@ -953,11 +953,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/switchd/parse.y b/usr.sbin/switchd/parse.y
index 8d751baf223..9128fcb6859 100644
--- a/usr.sbin/switchd/parse.y
+++ b/usr.sbin/switchd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.7 2018/06/11 09:57:01 denis Exp $ */
+/* $OpenBSD: parse.y,v 1.8 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2007-2016 Reyk Floeter <reyk@openbsd.org>
@@ -573,11 +573,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/vmd/parse.y b/usr.sbin/vmd/parse.y
index 52759a7ae4f..db6f834dc5c 100644
--- a/usr.sbin/vmd/parse.y
+++ b/usr.sbin/vmd/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.36 2018/06/26 10:00:08 reyk Exp $ */
+/* $OpenBSD: parse.y,v 1.37 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2007-2016 Reyk Floeter <reyk@openbsd.org>
@@ -972,11 +972,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
diff --git a/usr.sbin/ypldap/parse.y b/usr.sbin/ypldap/parse.y
index 5fdcc850f79..63735147890 100644
--- a/usr.sbin/ypldap/parse.y
+++ b/usr.sbin/ypldap/parse.y
@@ -1,4 +1,4 @@
-/* $OpenBSD: parse.y,v 1.25 2018/06/11 09:36:49 denis Exp $ */
+/* $OpenBSD: parse.y,v 1.26 2018/07/08 17:15:07 krw Exp $ */
/*
* Copyright (c) 2008 Pierre-Yves Ritschard <pyr@openbsd.org>
@@ -781,11 +781,11 @@ pushfile(const char *name, int secret)
struct file *nfile;
if ((nfile = calloc(1, sizeof(struct file))) == NULL) {
- log_warn("malloc");
+ log_warn("calloc");
return (NULL);
}
if ((nfile->name = strdup(name)) == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
free(nfile);
return (NULL);
}
@@ -845,7 +845,7 @@ parse_config(struct env *x_conf, const char *filename, int opts)
conf->sc_conf_tv.tv_usec = 0;
conf->sc_cafile = strdup(YPLDAP_CERT_FILE);
if (conf->sc_cafile == NULL) {
- log_warn("malloc");
+ log_warn("strdup");
return (-1);
}