git: fc391b7a56d9 - stable/13 - pf: Introduce pf_nvbool()

[Kristof Provost]

The branch stable/13 has been updated by kp:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=fc391b7a56d9b8eec3f1585397c8263923153442

commit fc391b7a56d9b8eec3f1585397c8263923153442
Author:     Kristof Provost <k...@freebsd.org>
AuthorDate: 2021-10-13 09:00:46 +0000
Commit:     Kristof Provost <k...@freebsd.org>
CommitDate: 2021-10-20 07:51:34 +0000

    pf: Introduce pf_nvbool()
    
    Similar to the existing functions for strings and ints, this lets us
    simplify some of the nvlist conversion code.
    
    MFC after:      1 week
    Sponsored by:   Rubicon Communications, LLC ("Netgate")
    
    (cherry picked from commit 776df104fa54bb581e1fb88ac44af4fa7fd4052b)
---
 sys/netpfil/pf/pf_nv.c | 14 ++++++++++++--
 sys/netpfil/pf/pf_nv.h |  1 +
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/sys/netpfil/pf/pf_nv.c b/sys/netpfil/pf/pf_nv.c
index d1eca90e0ee5..5eda6b3917f4 100644
--- a/sys/netpfil/pf/pf_nv.c
+++ b/sys/netpfil/pf/pf_nv.c
@@ -99,6 +99,17 @@ __FBSDID("$FreeBSD$");
                }                                                               
\
        }
 
+int
+pf_nvbool(const nvlist_t *nvl, const char *name, bool *val)
+{
+       if (! nvlist_exists_bool(nvl, name))
+               return (EINVAL);
+
+       *val = nvlist_get_bool(nvl, name);
+
+       return (0);
+}
+
 int
 pf_nvbinary(const nvlist_t *nvl, const char *name, void *data,
     size_t expected_size)
@@ -840,8 +851,7 @@ pf_nvstate_kill_to_kstate_kill(const nvlist_t *nvl,
            sizeof(kill->psk_ifname)));
        PFNV_CHK(pf_nvstring(nvl, "label", kill->psk_label,
            sizeof(kill->psk_label)));
-       if (nvlist_exists_bool(nvl, "kill_match"))
-               kill->psk_kill_match = nvlist_get_bool(nvl, "kill_match");
+       PFNV_CHK(pf_nvbool(nvl, "kill_match", &kill->psk_kill_match));
 
 errout:
        return (error);
diff --git a/sys/netpfil/pf/pf_nv.h b/sys/netpfil/pf/pf_nv.h
index f1cdc52aad6b..7ac3bc4153ee 100644
--- a/sys/netpfil/pf/pf_nv.h
+++ b/sys/netpfil/pf/pf_nv.h
@@ -70,6 +70,7 @@ PF_NV_DEF_UINT(uint16, uint16_t, UINT16_MAX);
 PF_NV_DEF_UINT(uint32, uint32_t, UINT32_MAX);
 PF_NV_DEF_UINT(uint64, uint64_t, UINT64_MAX);
 
+int    pf_nvbool(const nvlist_t *, const char *, bool *);
 int    pf_nvbinary(const nvlist_t *, const char *, void *, size_t);
 int    pf_nvint(const nvlist_t *, const char *, int *);
 int    pf_nvstring(const nvlist_t *, const char *, char *, size_t);