Fix auto-enabling of Capsicum on FreeBSD with Autoconf. [skip appveyor]
authorDenis Ovsienko <denis@ovsienko.info>
Tue, 3 Aug 2021 23:20:36 +0000 (00:20 +0100)
committerDenis Ovsienko <denis@ovsienko.info>
Tue, 3 Aug 2021 23:45:45 +0000 (00:45 +0100)
Even after commit 6393bb6 --with-sandbox-capsicum didn't work entirely
as documented, as it defaulted to disabled:

checking whether to sandbox using capsicum... no
checking whether to sandbox using Casper library... no

Get the test condition right so it does what it says:

checking sys/capsicum.h usability... yes
checking sys/capsicum.h presence... yes
checking for sys/capsicum.h... yes
checking for cap_enter... yes
checking for cap_rights_limit... yes
checking for cap_ioctls_limit... yes
checking for openat... yes
checking for cap_init in -lcasper... yes
checking for cap_gethostbyaddr in -lcap_dns... yes
checking whether to sandbox using capsicum... yes
checking whether to sandbox using Casper library... yes

CHANGES
configure
configure.ac

diff --git a/CHANGES b/CHANGES
index 435b9e7b93df3eb60083dfeb6ee16d2e8194b640..010fe469a68096a2040de17d6a38b09c954f32e4 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -25,6 +25,7 @@ Monthday, Month DD, YYYY by gharris and denis
       Fix "make releasetar" on AIX and Solaris.
       Mend "make check" on Solaris 9 with Autoconf.
       Address assorted compiler warnings.
+      Fix auto-enabling of Capsicum on FreeBSD with Autoconf.
 
 Wednesday, June 9, 2021 by gharris
   Summary for 4.99.1 tcpdump release
index 017eb0ad280f28a59c239a0b759eb9ded6f6e4bb..91291f3888b25d7d62f2907055957851e9b0d911 100755 (executable)
--- a/configure
+++ b/configure
@@ -4283,7 +4283,7 @@ fi
 # XXX - do we need to check for all of them, or are there some that, if
 # present, imply others are present?
 #
-if test ! -z "$with_sandbox_capsicum" && test "$with_sandbox_capsicum" != "no" ; then
+if test -z "$with_sandbox_capsicum" || test "$with_sandbox_capsicum" != "no" ; then
        #
        # First, make sure we have the required header.
        #
index 9567d51cfea178cffc89d280d841d347abb6c766..3e6e47e148e987b21a2ec3588238415fc3243f30 100644 (file)
@@ -198,7 +198,7 @@ AC_ARG_WITH(sandbox-capsicum,
 # XXX - do we need to check for all of them, or are there some that, if
 # present, imply others are present?
 #
-if test ! -z "$with_sandbox_capsicum" && test "$with_sandbox_capsicum" != "no" ; then
+if test -z "$with_sandbox_capsicum" || test "$with_sandbox_capsicum" != "no" ; then
        #
        # First, make sure we have the required header.
        #