Joseph,

I guess there is also regression with pfsync printing. Didn't check with
real traffic, but:

# grep pfsync_if_print *
grep: cmake: Is a directory
grep: doc: Is a directory
grep: lbl: Is a directory
grep: missing: Is a directory
netdissect.h:extern u_int pfsync_if_print(netdissect_options *, const struct 
pcap_pkthdr *, const u_char *);
print-pfsync.c:pfsync_if_print(netdissect_options *ndo, const struct 
pcap_pkthdr *h,
#

Doesn't look used.

On Wed, Jun 14, 2023 at 03:59:04AM +0000, Gleb Smirnoff wrote:
T> The branch main has been updated by glebius:
T> 
T> URL: 
https://cgit.FreeBSD.org/src/commit/?id=171a7bbfc04885150401ab64d96793373a8b2061
T> 
T> commit 171a7bbfc04885150401ab64d96793373a8b2061
T> Author:     Gleb Smirnoff <[email protected]>
T> AuthorDate: 2023-06-14 03:58:36 +0000
T> Commit:     Gleb Smirnoff <[email protected]>
T> CommitDate: 2023-06-14 03:58:36 +0000
T> 
T>     tcpdump: fix build with WITHOUT_PF=1
T>     
T>     Fixes:  ee67461e56828dd1f8de165947ba83f6d9148a87
T> ---
T>  contrib/tcpdump/print.c | 2 +-
T>  1 file changed, 1 insertion(+), 1 deletion(-)
T> 
T> diff --git a/contrib/tcpdump/print.c b/contrib/tcpdump/print.c
T> index 9c0ab8622de1..42a4548e8689 100644
T> --- a/contrib/tcpdump/print.c
T> +++ b/contrib/tcpdump/print.c
T> @@ -185,7 +185,7 @@ static const struct printer printers[] = {
T>  #ifdef DLT_LOOP
T>      { null_if_print,        DLT_LOOP },
T>  #endif
T> -#ifdef DLT_PFLOG
T> +#if defined(DLT_PFLOG) && defined(HAVE_NET_IF_PFLOG_H)
T>      { pflog_if_print,       DLT_PFLOG },
T>  #endif
T>  #ifdef DLT_PKTAP

-- 
Gleb Smirnoff

Reply via email to