diff options
author | krw <krw@openbsd.org> | 2018-07-08 17:15:07 +0000 |
---|---|---|
committer | krw <krw@openbsd.org> | 2018-07-08 17:15:07 +0000 |
commit | 3c12ada049ec8174546d7ebd54606e5a13c3e487 (patch) | |
tree | 624e17f8db98eb3aad46e23c1fe06b1fdd8649a2 | |
parent | document what happens when the @define-tag defining package gets removed (diff) | |
download | wireguard-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.y | 6 | ||||
-rw-r--r-- | sbin/ipsecctl/parse.y | 6 | ||||
-rw-r--r-- | sbin/pfctl/parse.y | 8 | ||||
-rw-r--r-- | usr.sbin/acme-client/parse.y | 4 | ||||
-rw-r--r-- | usr.sbin/bgpd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/dvmrpd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/ifstated/parse.y | 4 | ||||
-rw-r--r-- | usr.sbin/iscsictl/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/ldapd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/npppd/npppd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/ntpd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/ospf6d/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/ospfd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/radiusd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/ripd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/snmpd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/switchd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/vmd/parse.y | 6 | ||||
-rw-r--r-- | usr.sbin/ypldap/parse.y | 8 |
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); } |