diff options
author | Henning Brauer <henning@cvs.openbsd.org> | 2003-01-11 21:54:44 +0000 |
---|---|---|
committer | Henning Brauer <henning@cvs.openbsd.org> | 2003-01-11 21:54:44 +0000 |
commit | c176bee6065ef2d7fb475bd4964bdbd95fd096a4 (patch) | |
tree | 03e6ea16276c3226eb6fc7e72ca89eaf8093556a /sbin | |
parent | 9c7f650a338de7bb6c190a6a3ab5de59c43b0c17 (diff) |
use errx instead of handcrufting the same with fprintf and exit
Diffstat (limited to 'sbin')
-rw-r--r-- | sbin/pfctl/pfctl_table.c | 37 |
1 files changed, 11 insertions, 26 deletions
diff --git a/sbin/pfctl/pfctl_table.c b/sbin/pfctl/pfctl_table.c index 50ca1e7a62d..8d0077fa9e4 100644 --- a/sbin/pfctl/pfctl_table.c +++ b/sbin/pfctl/pfctl_table.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pfctl_table.c,v 1.18 2003/01/11 21:50:57 henning Exp $ */ +/* $OpenBSD: pfctl_table.c,v 1.19 2003/01/11 21:54:43 henning Exp $ */ /* * Copyright (c) 2002 Cedric Berger @@ -436,11 +436,8 @@ next_token(char buf[BUF_SIZE], FILE *fp) buf[i++] = next_ch; next_ch = fgetc(fp); } while (!feof(fp) && !isspace(next_ch)); - if (i >= BUF_SIZE) { - fprintf(stderr, "%s: address too long (%d bytes)\n", - __progname, i); - exit(1); - } + if (i >= BUF_SIZE) + errx(1, "address too long (%d bytes)", i); buf[i] = '\0'; return (1); } @@ -477,30 +474,21 @@ append_addr(char *s, int test) if (p) { net = strtol(p+1, &q, 0); - if (!q || *q) { - fprintf(stderr, "%s: illegal network: \"%s\"\n", - __progname, p+1); - exit(1); - } + if (!q || *q) + errx(1, "illegal network: \"%s\"", p+1); *p++ = '\0'; } bzero(&hints, sizeof(hints)); hints.ai_socktype = SOCK_DGRAM; rv = getaddrinfo(buf, NULL, &hints, &res); - if (rv) { - fprintf(stderr, "%s: illegal address: \"%s\"\n", __progname, - buf); - exit(1); - } + if (rv) + errx(1, "illegal address: \"%s\"", buf); for (ai = res; ai; ai = ai->ai_next) { switch (ai->ai_family) { case AF_INET: - if (net > 32) { - fprintf(stderr, "%s: network too big: %d\n", - __progname, net); - exit(1); - } + if (net > 32) + errx(1, "invalid netmask %d", net); if (size >= msize) grow_buffer(sizeof(struct pfr_addr), 0); buffer.addrs[size].pfra_ip4addr = @@ -511,11 +499,8 @@ append_addr(char *s, int test) size++; break; case AF_INET6: - if (net > 128) { - fprintf(stderr, "%s: network too big: %d\n", - __progname, net); - exit(1); - } + if (net > 128) + errx(1, "invalid netmask %d", net); if (size >= msize) grow_buffer(sizeof(struct pfr_addr), 0); buffer.addrs[size].pfra_ip6addr = |