rename pfsync_up() to pfsync_is_up()
authordlg <dlg@openbsd.org>
Fri, 11 Nov 2022 11:47:12 +0000 (11:47 +0000)
committerdlg <dlg@openbsd.org>
Fri, 11 Nov 2022 11:47:12 +0000 (11:47 +0000)
foo_up() where foo is a network driver is usually a function that
configures and brings an interface up into a running state. this
small tweak just makes the code a bit easier for me to read.

sys/net/if_pfsync.c
sys/net/if_pfsync.h
sys/net/pf.c

index 64a2da1..f69790e 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: if_pfsync.c,v 1.310 2022/11/11 11:22:48 sashan Exp $  */
+/*     $OpenBSD: if_pfsync.c,v 1.311 2022/11/11 11:47:13 dlg Exp $     */
 
 /*
  * Copyright (c) 2002 Michael Shalayeff
@@ -2571,7 +2571,7 @@ pfsync_send_plus(void *plus, size_t pluslen)
 }
 
 int
-pfsync_up(void)
+pfsync_is_up(void)
 {
        struct pfsync_softc *sc = pfsyncif;
 
index 5447b82..ff26ac3 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: if_pfsync.h,v 1.58 2022/11/06 18:05:05 dlg Exp $      */
+/*     $OpenBSD: if_pfsync.h,v 1.59 2022/11/11 11:47:13 dlg Exp $      */
 
 /*
  * Copyright (c) 2001 Michael Shalayeff
@@ -341,7 +341,7 @@ int                 pfsync_defer(struct pf_state *, struct mbuf *,
                            struct pfsync_deferral **);
 void                   pfsync_undefer(struct pfsync_deferral *, int);
 
-int                    pfsync_up(void);
+int                    pfsync_is_up(void);
 int                    pfsync_state_in_use(struct pf_state *);
 
 void                   pfsync_iack(struct pf_state *);
index f61449a..d936112 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: pf.c,v 1.1149 2022/11/11 11:02:35 dlg Exp $ */
+/*     $OpenBSD: pf.c,v 1.1150 2022/11/11 11:47:12 dlg Exp $ */
 
 /*
  * Copyright (c) 2001 Daniel Hartmeier
@@ -4285,7 +4285,7 @@ pf_test_rule(struct pf_pdesc *pd, struct pf_rule **rm, struct pf_state **sm,
 
 #if NPFSYNC > 0
        if (*sm != NULL && !ISSET((*sm)->state_flags, PFSTATE_NOSYNC) &&
-           pd->dir == PF_OUT && pfsync_up()) {
+           pd->dir == PF_OUT && pfsync_is_up()) {
                /*
                 * We want the state created, but we dont
                 * want to send this in case a partner