git: da8d8b22e69c - main - pf: fix ABI breakage

[Kristof Provost]

The branch main has been updated by kp:

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

commit da8d8b22e69c72f1d001d55b5eff5acbb0265cfc
Author:     Kristof Provost <kp at FreeBSD.org>
AuthorDate: 2021-07-28 09:24:44 +0000
Commit:     Kristof Provost <kp at FreeBSD.org>
CommitDate: 2021-07-28 10:16:33 +0000

    pf: fix ABI breakage
    
    The introduction of synproxy support changed the size of struct
    pf_status, which in turn broke the userspace ABI.
    
    Revert the relevant change. More work is needed on the synproxy code to
    keep and expose the counters, but in the mean time this restores the
    ABI.
    
    PR:             257469
    MFC after:      3 days
    Sponsored by:   Modirum MDPay
---
 sys/netpfil/pf/pf.h            | 3 +--
 sys/netpfil/pf/pf_syncookies.c | 1 -
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/sys/netpfil/pf/pf.h b/sys/netpfil/pf/pf.h
index c8bf932af9c3..d8ee43506d99 100644
--- a/sys/netpfil/pf/pf.h
+++ b/sys/netpfil/pf/pf.h
@@ -160,8 +160,7 @@ enum	{ PF_ADDR_ADDRMASK, PF_ADDR_NOROUTE, PF_ADDR_DYNIFTL,
 #define LCNT_SRCCONNRATE	4	/* max-src-conn-rate */
 #define LCNT_OVERLOAD_TABLE	5	/* entry added to overload table */
 #define LCNT_OVERLOAD_FLUSH	6	/* state entries flushed */
-#define LCNT_SYNCOOKIES_VALID	7	/* syncookies validated */ /* XXX TODO: Ensure no API breakage! */
-#define LCNT_MAX		8	/* total+1 */
+#define LCNT_MAX		7	/* total+1 */
 
 #define LCNT_NAMES { \
 	"max states per rule", \
diff --git a/sys/netpfil/pf/pf_syncookies.c b/sys/netpfil/pf/pf_syncookies.c
index 8699ae855ec8..744a12e4e21d 100644
--- a/sys/netpfil/pf/pf_syncookies.c
+++ b/sys/netpfil/pf/pf_syncookies.c
@@ -275,7 +275,6 @@ pf_syncookie_validate(struct pf_pdesc *pd)
 	if ((ack & ~0xff) != (hash & ~0xff))
 		return (0);
 
-	counter_u64_add(V_pf_status.lcounters[LCNT_SYNCOOKIES_VALID], 1);
 	return (1);
 }