summaryrefslogtreecommitdiff
path: root/sbin/pfctl
diff options
context:
space:
mode:
Diffstat (limited to 'sbin/pfctl')
-rw-r--r--sbin/pfctl/pfctl.c68
-rw-r--r--sbin/pfctl/pfctl_table.c48
2 files changed, 72 insertions, 44 deletions
diff --git a/sbin/pfctl/pfctl.c b/sbin/pfctl/pfctl.c
index 50b3689dfd2..0b4a5ce6062 100644
--- a/sbin/pfctl/pfctl.c
+++ b/sbin/pfctl/pfctl.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: pfctl.c,v 1.129 2003/01/09 18:55:32 dhartmei Exp $ */
+/* $OpenBSD: pfctl.c,v 1.130 2003/01/10 14:21:21 cedric Exp $ */
/*
* Copyright (c) 2001 Daniel Hartmeier
@@ -49,6 +49,7 @@
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
+#include <assert.h>
#include "pfctl_parser.h"
#include "pfctl.h"
@@ -77,6 +78,7 @@ int pfctl_clear_rule_counters(int, int);
int pfctl_add_pool(struct pfctl *, struct pf_pool *, sa_family_t);
int pfctl_test_altqsupport(int, int);
int pfctl_show_anchors(int, int);
+char *pfctl_lookup_option(char *, char **);
char *clearopt;
char *rulesopt;
@@ -158,6 +160,25 @@ static const struct {
{ NULL, NULL }
};
+static char *clearopt_list[] = {
+ "nat", "queue", "rules", "state", "info", "Tables", "all", NULL
+};
+
+static char *showopt_list[] = {
+ "nat", "queue", "rules", "anchors", "state", "info", "labels",
+ "timeouts", "memory", "Tables", "all", NULL
+};
+
+static char *tblcmdopt_list[] = {
+ "create", "kill", "flush", "add", "delete", "replace", "show",
+ "test", "zero", NULL
+};
+
+static char *debugopt_list[] = {
+ "none", "urgent", "misc", NULL
+};
+
+
void
usage(void)
{
@@ -1211,6 +1232,18 @@ pfctl_show_anchors(int dev, int opts)
return (0);
}
+char *
+pfctl_lookup_option(char *cmd, char **list)
+{
+ if (cmd != NULL && *cmd)
+ for (; *list; list++)
+ if (!strncmp(cmd, *list, strlen(cmd)))
+ return (*list);
+ return (NULL);
+}
+
+
+
int
main(int argc, char *argv[])
{
@@ -1241,7 +1274,11 @@ main(int argc, char *argv[])
opts |= PF_OPT_QUIET;
break;
case 'F':
- clearopt = optarg;
+ clearopt = pfctl_lookup_option(optarg, clearopt_list);
+ if (clearopt == NULL) {
+ warnx("Unknown flush modifier '%s'", optarg);
+ usage();
+ }
mode = O_RDWR;
break;
case 'k':
@@ -1280,13 +1317,21 @@ main(int argc, char *argv[])
loadopt |= PFCTL_FLAG_OPTION;
break;
case 's':
- showopt = optarg;
+ showopt = pfctl_lookup_option(optarg, showopt_list);
+ if (showopt == NULL) {
+ warnx("Unknown show modifier '%s'", optarg);
+ usage();
+ }
break;
case 't':
tableopt = optarg;
break;
case 'T':
- tblcmdopt = optarg;
+ tblcmdopt = pfctl_lookup_option(optarg, tblcmdopt_list);
+ if (tblcmdopt == NULL) {
+ warnx("Unknown table command '%s'", optarg);
+ usage();
+ }
break;
case 'v':
if (opts & PF_OPT_VERBOSE)
@@ -1294,7 +1339,11 @@ main(int argc, char *argv[])
opts |= PF_OPT_VERBOSE;
break;
case 'x':
- debugopt = optarg;
+ debugopt = pfctl_lookup_option(optarg, debugopt_list);
+ if (debugopt == NULL) {
+ warnx("Unknown debug level '%s'", optarg);
+ usage();
+ }
mode = O_RDWR;
break;
case 'z':
@@ -1407,8 +1456,7 @@ main(int argc, char *argv[])
pfctl_clear_tables(opts);
break;
default:
- warnx("Unknown flush modifier '%s'", clearopt);
- error = 1;
+ assert(0);
}
}
if (state_killers)
@@ -1468,8 +1516,7 @@ main(int argc, char *argv[])
pfctl_show_tables(opts);
break;
default:
- warnx("Unknown show modifier '%s'", showopt);
- error = 1;
+ assert(0);
}
}
@@ -1489,8 +1536,7 @@ main(int argc, char *argv[])
pfctl_debug(dev, PF_DEBUG_MISC, opts);
break;
default:
- warnx("Unknown debug level '%s'", debugopt);
- error = 1;
+ assert(0);
}
}
diff --git a/sbin/pfctl/pfctl_table.c b/sbin/pfctl/pfctl_table.c
index 77a3bf5e9bd..dc1724e351f 100644
--- a/sbin/pfctl/pfctl_table.c
+++ b/sbin/pfctl/pfctl_table.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: pfctl_table.c,v 1.15 2003/01/09 18:59:02 dhartmei Exp $ */
+/* $OpenBSD: pfctl_table.c,v 1.16 2003/01/10 14:21:21 cedric Exp $ */
/*
* Copyright (c) 2002 Cedric Berger
@@ -81,21 +81,6 @@ static union {
static int size, msize, ticket, inactive;
extern char *__progname;
-static char *commands[] = {
- "-F", /* pfctl -F tables: flush all tables */
- "-s", /* pfctl -s tables: show all tables */
- "create", /* create a new table */
- "kill", /* kill a table */
- "flush", /* flush all addresses of a table */
- "add", /* add one or more addresses in a table */
- "delete", /* delete one or more addresses from a table */
- "replace", /* replace the addresses of the table */
- "show", /* show the content (addresses) of a table */
- "test", /* test if the given addresses match a table */
- "zero", /* clear all the statistics of a table */
- NULL
-};
-
static char *stats_text[PFR_DIR_MAX][PFR_OP_TABLE_MAX] = {
{ "In/Block:", "In/Pass:", "In/XPass:" },
{ "Out/Block:", "Out/Pass:", "Out/XPass:" }
@@ -137,14 +122,10 @@ pfctl_table(int argc, char *argv[], char *tname, char *command,
char *file, int opts)
{
struct pfr_table table;
- char **p;
int nadd = 0, ndel = 0, nchange = 0, nzero = 0;
int i, flags = 0, nmatch = 0;
- for (p = commands; *p != NULL; p++)
- if (!strncmp(command, *p, strlen(command)))
- break;
- if (*p == NULL)
+ if (command == NULL)
usage();
if (opts & PF_OPT_NOACTION)
flags |= PFR_FLAG_DUMMY;
@@ -156,14 +137,14 @@ pfctl_table(int argc, char *argv[], char *tname, char *command,
sizeof(table.pfrt_name)) >= sizeof(table.pfrt_name))
errx(1, "pfctl_table: strlcpy");
}
- if (!strcmp(*p, "-F")) {
+ if (!strcmp(command, "-F")) {
if (argc || file != NULL)
usage();
RVTEST(pfr_clr_tables(&ndel, flags));
if (!(opts & PF_OPT_QUIET))
fprintf(stderr, "%d tables deleted%s.\n", ndel,
DUMMY);
- } else if (!strcmp(*p, "-s")) {
+ } else if (!strcmp(command, "-s")) {
if (argc || file != NULL)
usage();
for (;;) {
@@ -188,27 +169,27 @@ pfctl_table(int argc, char *argv[], char *tname, char *command,
else
print_table(buffer.tables+i,
opts & PF_OPT_VERBOSE);
- } else if (!strcmp(*p, "create")) {
+ } else if (!strcmp(command, "create")) {
if (argc || file != NULL)
usage();
table.pfrt_flags = PFR_TFLAG_PERSIST;
RVTEST(pfr_add_tables(&table, 1, &nadd, flags));
if (!(opts & PF_OPT_QUIET))
fprintf(stderr, "%d table added%s.\n", nadd, DUMMY);
- } else if (!strcmp(*p, "kill")) {
+ } else if (!strcmp(command, "kill")) {
if (argc || file != NULL)
usage();
RVTEST(pfr_del_tables(&table, 1, &ndel, flags));
if (!(opts & PF_OPT_QUIET))
fprintf(stderr, "%d table deleted%s.\n", ndel, DUMMY);
- } else if (!strcmp(*p, "flush")) {
+ } else if (!strcmp(command, "flush")) {
if (argc || file != NULL)
usage();
RVTEST(pfr_clr_addrs(&table, &ndel, flags));
if (!(opts & PF_OPT_QUIET))
fprintf(stderr, "%d addresses deleted%s.\n", ndel,
DUMMY);
- } else if (!strcmp(*p, "add")) {
+ } else if (!strcmp(command, "add")) {
load_addr(argc, argv, file, 0);
if (opts & PF_OPT_VERBOSE)
flags |= PFR_FLAG_FEEDBACK;
@@ -223,7 +204,7 @@ pfctl_table(int argc, char *argv[], char *tname, char *command,
buffer.addrs[i].pfra_fback)
print_addrx(buffer.addrs+i, NULL,
opts & PF_OPT_USEDNS);
- } else if (!strcmp(*p, "delete")) {
+ } else if (!strcmp(command, "delete")) {
load_addr(argc, argv, file, 0);
if (opts & PF_OPT_VERBOSE)
flags |= PFR_FLAG_FEEDBACK;
@@ -238,7 +219,7 @@ pfctl_table(int argc, char *argv[], char *tname, char *command,
buffer.addrs[i].pfra_fback)
print_addrx(buffer.addrs+i, NULL,
opts & PF_OPT_USEDNS);
- } else if (!strcmp(*p, "replace")) {
+ } else if (!strcmp(command, "replace")) {
load_addr(argc, argv, file, 0);
if (opts & PF_OPT_VERBOSE)
flags |= PFR_FLAG_FEEDBACK;
@@ -272,7 +253,7 @@ pfctl_table(int argc, char *argv[], char *tname, char *command,
buffer.addrs[i].pfra_fback)
print_addrx(buffer.addrs+i, NULL,
opts & PF_OPT_USEDNS);
- } else if (!strcmp(*p, "show")) {
+ } else if (!strcmp(command, "show")) {
if (argc || file != NULL)
usage();
for (;;) {
@@ -298,7 +279,7 @@ pfctl_table(int argc, char *argv[], char *tname, char *command,
print_addrx(buffer.addrs+i, NULL,
opts & PF_OPT_USEDNS);
}
- } else if (!strcmp(*p, "test")) {
+ } else if (!strcmp(command, "test")) {
load_addr(argc, argv, file, 1);
if (opts & PF_OPT_VERBOSE2) {
flags |= PFR_FLAG_REPLACE;
@@ -325,7 +306,7 @@ pfctl_table(int argc, char *argv[], char *tname, char *command,
opts & PF_OPT_USEDNS);
if (nmatch < size)
return (2);
- } else if (!strcmp(*p, "zero")) {
+ } else if (!strcmp(command, "zero")) {
if (argc || file != NULL)
usage();
flags |= PFR_FLAG_ADDRSTOO;
@@ -333,7 +314,8 @@ pfctl_table(int argc, char *argv[], char *tname, char *command,
if (!(opts & PF_OPT_QUIET))
fprintf(stderr, "%d table/stats cleared%s.\n", nzero,
DUMMY);
- }
+ } else
+ assert(0);
return (0);
}