On Sat, Aug 20, 2022 at 01:35:38PM -0400, Charlie Li wrote:
C> > -#define   PR_SLOWHZ       2               /* 2 slow timeouts per second */
C> > -#define   PR_FASTHZ       5               /* 5 fast timeouts per second */
C> > -
C> >   /*
C> >    * This number should be defined again within each protocol family to 
avoid
C> >    * confusion.
C> > 
C> > 
C> This broke net-mgmt/net-snmp, which uses PR_SLOWHZ directly. Upstream 
C> issue filed at https://github.com/net-snmp/net-snmp/issues/445

Thanks for report! Looking into mibII/tcp.c code, I'm 99% sure it
wasn't correct on FreeBSD for at least 10-15 years if not always.

The next failure is:

mibgroup/ip-mib/data_access/scalars_sysctl.c:285:38: error: use of undeclared 
identifier 'PR_SLOWHZ'
    ipReasmTimeout_val = IPFRAGTTL / PR_SLOWHZ;

And this code was actually constructing correct value.

I created pull requests upstream.

-- 
Gleb Smirnoff

Reply via email to