Allocate path only when needed, use __func__
authorkn <kn@openbsd.org>
Sat, 8 Sep 2018 14:45:55 +0000 (14:45 +0000)
committerkn <kn@openbsd.org>
Sat, 8 Sep 2018 14:45:55 +0000 (14:45 +0000)
sbin/pfctl/pfctl.c

index f65ebb7..b08f730 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: pfctl.c,v 1.358 2018/09/08 14:12:57 kn Exp $ */
+/*     $OpenBSD: pfctl.c,v 1.359 2018/09/08 14:45:55 kn Exp $ */
 
 /*
  * Copyright (c) 2001 Daniel Hartmeier
@@ -2496,8 +2496,6 @@ main(int argc, char *argv[])
                /* NOTREACHED */
        }
 
-       if ((path = calloc(1, PATH_MAX)) == NULL)
-               errx(1, "pfctl: calloc");
        memset(anchorname, 0, sizeof(anchorname));
        if (anchoropt != NULL) {
                if (mode == O_RDONLY && showopt == NULL) {
@@ -2541,6 +2539,9 @@ main(int argc, char *argv[])
                if (pfctl_disable(dev, opts))
                        error = 1;
 
+       if ((path = calloc(1, PATH_MAX)) == NULL)
+               errx(1, "%s: calloc", __func__);
+
        if (showopt != NULL) {
                switch (*showopt) {
                case 'A':