Adapt to <limits.h> universe.
authorderaadt <deraadt@openbsd.org>
Fri, 16 Jan 2015 15:06:40 +0000 (15:06 +0000)
committerderaadt <deraadt@openbsd.org>
Fri, 16 Jan 2015 15:06:40 +0000 (15:06 +0000)
ok millert

17 files changed:
usr.sbin/relayd/ca.c
usr.sbin/relayd/carp.c
usr.sbin/relayd/check_icmp.c
usr.sbin/relayd/check_script.c
usr.sbin/relayd/check_tcp.c
usr.sbin/relayd/control.c
usr.sbin/relayd/hce.c
usr.sbin/relayd/parse.y
usr.sbin/relayd/pfe.c
usr.sbin/relayd/pfe_filter.c
usr.sbin/relayd/pfe_route.c
usr.sbin/relayd/relay.c
usr.sbin/relayd/relay_http.c
usr.sbin/relayd/relay_udp.c
usr.sbin/relayd/relayd.c
usr.sbin/relayd/relayd.h
usr.sbin/relayd/ssl.c

index 137d503..6964a0b 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: ca.c,v 1.10 2014/12/12 10:05:09 reyk Exp $    */
+/*     $OpenBSD: ca.c,v 1.11 2015/01/16 15:06:40 deraadt Exp $ */
 
 /*
  * Copyright (c) 2014 Reyk Floeter <reyk@openbsd.org>
@@ -16,7 +16,6 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <sys/param.h>
 #include <sys/queue.h>
 #include <sys/socket.h>
 #include <sys/uio.h>
index 536fed2..ba5836b 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: carp.c,v 1.9 2012/11/27 05:00:28 guenther Exp $ */
+/*     $OpenBSD: carp.c,v 1.10 2015/01/16 15:06:40 deraadt Exp $ */
 
 /*
  * Copyright (c) 2006 Henning Brauer <henning@openbsd.org>
@@ -16,7 +16,6 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <sys/param.h>
 #include <sys/socket.h>
 #include <sys/ioctl.h>
 
index a32d930..3fefb8a 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: check_icmp.c,v 1.38 2014/12/21 00:54:49 guenther Exp $        */
+/*     $OpenBSD: check_icmp.c,v 1.39 2015/01/16 15:06:40 deraadt Exp $ */
 
 /*
  * Copyright (c) 2006 Pierre-Yves Ritschard <pyr@openbsd.org>
@@ -16,7 +16,6 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <sys/param.h>
 #include <sys/types.h>
 #include <sys/queue.h>
 #include <sys/socket.h>
index adf9340..0e22176 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: check_script.c,v 1.16 2014/06/25 11:05:15 reyk Exp $  */
+/*     $OpenBSD: check_script.c,v 1.17 2015/01/16 15:06:40 deraadt Exp $       */
 
 /*
  * Copyright (c) 2007 - 2014 Reyk Floeter <reyk@openbsd.org>
@@ -16,7 +16,6 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <sys/param.h>
 #include <sys/queue.h>
 #include <sys/socket.h>
 #include <sys/wait.h>
index 4e2553e..d439471 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: check_tcp.c,v 1.44 2014/12/12 10:05:09 reyk Exp $     */
+/*     $OpenBSD: check_tcp.c,v 1.45 2015/01/16 15:06:40 deraadt Exp $  */
 
 /*
  * Copyright (c) 2006 Pierre-Yves Ritschard <pyr@openbsd.org>
@@ -16,7 +16,6 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <sys/param.h>
 #include <sys/queue.h>
 #include <sys/socket.h>
 
index e9c7fb2..2272e34 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: control.c,v 1.46 2014/07/11 16:39:10 krw Exp $        */
+/*     $OpenBSD: control.c,v 1.47 2015/01/16 15:06:40 deraadt Exp $    */
 
 /*
  * Copyright (c) 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -17,7 +17,6 @@
  */
 
 #include <sys/queue.h>
-#include <sys/param.h>
 #include <sys/stat.h>
 #include <sys/socket.h>
 #include <sys/un.h>
index 5459e83..70748dd 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: hce.c,v 1.67 2014/12/12 10:05:09 reyk Exp $   */
+/*     $OpenBSD: hce.c,v 1.68 2015/01/16 15:06:40 deraadt Exp $        */
 
 /*
  * Copyright (c) 2006 Pierre-Yves Ritschard <pyr@openbsd.org>
@@ -16,7 +16,6 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <sys/param.h>
 #include <sys/queue.h>
 #include <sys/time.h>
 #include <sys/stat.h>
index 7b46db8..1c692c2 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: parse.y,v 1.199 2014/12/23 13:18:23 reyk Exp $        */
+/*     $OpenBSD: parse.y,v 1.200 2015/01/16 15:06:40 deraadt Exp $     */
 
 /*
  * Copyright (c) 2007 - 2014 Reyk Floeter <reyk@openbsd.org>
@@ -150,7 +150,7 @@ typedef struct {
                enum direction           dir;
                struct {
                        struct sockaddr_storage  ss;
-                       char                     name[MAXHOSTNAMELEN];
+                       char                     name[HOST_NAME_MAX+1];
                }                        addr;
                struct {
                        enum digest_type type;
index 150642a..c7ddde9 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: pfe.c,v 1.76 2014/11/19 10:24:40 blambert Exp $       */
+/*     $OpenBSD: pfe.c,v 1.77 2015/01/16 15:06:40 deraadt Exp $        */
 
 /*
  * Copyright (c) 2006 Pierre-Yves Ritschard <pyr@openbsd.org>
@@ -16,7 +16,6 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <sys/param.h>
 #include <sys/stat.h>
 #include <sys/socket.h>
 #include <sys/un.h>
index d828f8e..19cb4d8 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: pfe_filter.c,v 1.54 2014/12/23 13:18:23 reyk Exp $    */
+/*     $OpenBSD: pfe_filter.c,v 1.55 2015/01/16 15:06:40 deraadt Exp $ */
 
 /*
  * Copyright (c) 2006 Pierre-Yves Ritschard <pyr@openbsd.org>
@@ -38,6 +38,8 @@
 
 #include <openssl/ssl.h>
 
+#define MINIMUM(a, b)  (((a) < (b)) ? (a) : (b))
+
 #include "relayd.h"
 
 struct pfdata {
@@ -443,7 +445,7 @@ sync_ruleset(struct relayd *env, struct rdr *rdr, int enable)
 
                if (rio.rule.proto == IPPROTO_TCP)
                        rio.rule.timeout[PFTM_TCP_ESTABLISHED] =
-                           (u_int32_t)MIN(rdr->conf.timeout.tv_sec, INT_MAX);
+                           (u_int32_t)MINIMUM(rdr->conf.timeout.tv_sec, INT_MAX);
 
                if (strlen(rdr->conf.tag))
                        (void)strlcpy(rio.rule.tagname, rdr->conf.tag,
index 715bad8..135b935 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: pfe_route.c,v 1.7 2014/12/21 00:54:49 guenther Exp $  */
+/*     $OpenBSD: pfe_route.c,v 1.8 2015/01/16 15:06:40 deraadt Exp $   */
 
 /*
  * Copyright (c) 2009 - 2011 Reyk Floeter <reyk@openbsd.org>
@@ -75,7 +75,7 @@ sync_routes(struct relayd *env, struct router *rt)
 {
        struct netroute         *nr;
        struct host             *host;
-       char                     buf[MAXHOSTNAMELEN];
+       char                     buf[HOST_NAME_MAX+1];
        struct ctl_netroute      crt;
 
        if (!(env->sc_flags & F_NEEDRT))
index 894c26d..eff2ee2 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: relay.c,v 1.185 2015/01/16 14:34:51 reyk Exp $        */
+/*     $OpenBSD: relay.c,v 1.186 2015/01/16 15:06:40 deraadt Exp $     */
 
 /*
  * Copyright (c) 2006 - 2014 Reyk Floeter <reyk@openbsd.org>
@@ -45,6 +45,8 @@
 
 #include "relayd.h"
 
+#define MINIMUM(a, b)  (((a) < (b)) ? (a) : (b))
+
 void            relay_statistics(int, short, void *);
 int             relay_dispatch_parent(int, struct privsep_proc *,
                    struct imsg *);
@@ -1964,7 +1966,7 @@ relay_tls_callback_dh(SSL *ssl, int export, int keylen)
        }
 
        /* get built-in params based on the shorter key length */
-       dh = relay_tls_get_dhparams(MIN(keylen, maxlen));
+       dh = relay_tls_get_dhparams(MINIMUM(keylen, maxlen));
 
        return (dh);
 }
@@ -1994,7 +1996,7 @@ relay_tls_ctx_create(struct relay *rlay)
 
        /* Modify session timeout and cache size*/
        SSL_CTX_set_timeout(ctx,
-           (long)MIN(rlay->rl_conf.timeout.tv_sec, LONG_MAX));
+           (long)MINIMUM(rlay->rl_conf.timeout.tv_sec, LONG_MAX));
        if (proto->cache < -1) {
                SSL_CTX_set_session_cache_mode(ctx, SSL_SESS_CACHE_OFF);
        } else if (proto->cache >= -1) {
@@ -2338,7 +2340,7 @@ relay_tls_readcb(int fd, short event, void *arg)
        }
 
        if (bufev->wm_read.high != 0)
-               howmuch = MIN(sizeof(rbuf), bufev->wm_read.high);
+               howmuch = MINIMUM(sizeof(rbuf), bufev->wm_read.high);
 
        ret = SSL_read(cre->ssl, rbuf, howmuch);
        if (ret <= 0) {
index b3038de..99b804d 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: relay_http.c,v 1.40 2015/01/13 09:24:20 reyk Exp $    */
+/*     $OpenBSD: relay_http.c,v 1.41 2015/01/16 15:06:40 deraadt Exp $ */
 
 /*
  * Copyright (c) 2006 - 2015 Reyk Floeter <reyk@openbsd.org>
@@ -693,7 +693,7 @@ relay_lookup_url(struct ctl_relay_event *cre, const char *host, struct kv *kv)
        struct http_descriptor  *desc = (struct http_descriptor *)cre->desc;
        int                      i, j, dots;
        char                    *hi[RELAY_MAXLOOKUPLEVELS], *p, *pp, *c, ch;
-       char                     ph[MAXHOSTNAMELEN];
+       char                     ph[HOST_NAME_MAX+1];
        int                      ret;
 
        if (desc->http_path == NULL)
index df19122..ccd8567 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: relay_udp.c,v 1.37 2014/12/21 00:54:49 guenther Exp $ */
+/*     $OpenBSD: relay_udp.c,v 1.38 2015/01/16 15:06:40 deraadt Exp $  */
 
 /*
  * Copyright (c) 2007 - 2013 Reyk Floeter <reyk@openbsd.org>
@@ -29,6 +29,7 @@
 #include <netinet/ip.h>
 #include <netinet/tcp.h>
 
+#include <signal.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <stdlib.h>
index eceeb2e..b5fdf91 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: relayd.c,v 1.135 2014/12/18 20:55:01 reyk Exp $       */
+/*     $OpenBSD: relayd.c,v 1.136 2015/01/16 15:06:40 deraadt Exp $    */
 
 /*
  * Copyright (c) 2007 - 2014 Reyk Floeter <reyk@openbsd.org>
@@ -27,6 +27,7 @@
 #include <netinet/in.h>
 #include <arpa/inet.h>
 
+#include <signal.h>
 #include <string.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -46,6 +47,8 @@
 
 #include "relayd.h"
 
+#define MAXIMUM(a, b)  (((a) > (b)) ? (a) : (b))
+
 __dead void     usage(void);
 
 int             parent_configure(struct relayd *);
@@ -1483,7 +1486,7 @@ socket_rlimit(int maxfd)
        if (maxfd == -1)
                rl.rlim_cur = rl.rlim_max;
        else
-               rl.rlim_cur = MAX(rl.rlim_max, (rlim_t)maxfd);
+               rl.rlim_cur = MAXIMUM(rl.rlim_max, (rlim_t)maxfd);
        if (setrlimit(RLIMIT_NOFILE, &rl) == -1)
                fatal("socket_rlimit: failed to set resource limit");
 }
index 1da7e9d..1a450d2 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: relayd.h,v 1.203 2015/01/13 09:24:21 reyk Exp $       */
+/*     $OpenBSD: relayd.h,v 1.204 2015/01/16 15:06:41 deraadt Exp $    */
 
 /*
  * Copyright (c) 2006 - 2015 Reyk Floeter <reyk@openbsd.org>
@@ -23,7 +23,6 @@
 
 #include <sys/tree.h>
 
-#include <sys/param.h>         /* MAXHOSTNAMELEN */
 #include <netinet/in.h>
 #include <limits.h>
 #include <imsg.h>
@@ -126,8 +125,8 @@ struct ctl_script {
        objid_t          host;
        int              retval;
        struct timeval   timeout;
-       char             name[MAXHOSTNAMELEN];
-       char             path[MAXPATHLEN];
+       char             name[HOST_NAME_MAX+1];
+       char             path[PATH_MAX];
 };
 
 struct ctl_demote {
@@ -387,7 +386,7 @@ struct host_config {
        objid_t                  parentid;
        objid_t                  tableid;
        int                      retry;
-       char                     name[MAXHOSTNAMELEN];
+       char                     name[HOST_NAME_MAX+1];
        struct sockaddr_storage  ss;
        int                      ttl;
        int                      priority;
@@ -469,7 +468,7 @@ struct table_config {
        int                      skip_cnt;
        char                     name[TABLE_NAME_SIZE];
        size_t                   name_len;
-       char                     path[MAXPATHLEN];
+       char                     path[PATH_MAX];
        char                     exbuf[64];
        char                     digest[41]; /* length of sha1 digest * 2 */
        u_int8_t                 digest_type;
@@ -683,9 +682,9 @@ struct protocol {
        char                     tlsciphers[768];
        int                      tlsdhparams;
        int                      tlsecdhcurve;
-       char                     tlsca[MAXPATHLEN];
-       char                     tlscacert[MAXPATHLEN];
-       char                     tlscakey[MAXPATHLEN];
+       char                     tlsca[PATH_MAX];
+       char                     tlscacert[PATH_MAX];
+       char                     tlscakey[PATH_MAX];
        char                    *tlscapass;
        char                     name[MAX_NAME_SIZE];
        int                      cache;
@@ -728,7 +727,7 @@ struct relay_config {
        objid_t                  id;
        u_int32_t                flags;
        objid_t                  proto;
-       char                     name[MAXHOSTNAMELEN];
+       char                     name[HOST_NAME_MAX+1];
        in_port_t                port;
        in_port_t                dstport;
        int                      dstretry;
@@ -813,7 +812,7 @@ TAILQ_HEAD(netroutelist, netroute);
 struct router_config {
        objid_t                  id;
        u_int32_t                flags;
-       char                     name[MAXHOSTNAMELEN];
+       char                     name[HOST_NAME_MAX+1];
        char                     label[RT_LABEL_SIZE];
        int                      nroutes;
        objid_t                  gwtable;
index 93f7b1e..b813329 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: ssl.c,v 1.26 2014/12/12 10:05:09 reyk Exp $   */
+/*     $OpenBSD: ssl.c,v 1.27 2015/01/16 15:06:41 deraadt Exp $        */
 
 /*
  * Copyright (c) 2007 - 2014 Reyk Floeter <reyk@openbsd.org>
@@ -17,7 +17,6 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <sys/param.h>
 #include <sys/queue.h>
 #include <sys/socket.h>