Page MenuHomeFreeBSD
Differential D53308

ipfilter: Calculate the number of elements in ipf_errors
ClosedPublic
Actions

Authored by cy on Oct 23 2025, 11:01 PM.
Tags
None
Referenced Files
F165085564: D53308.diff
Wed, Aug 5, 9:56 PM
Unknown Object (File)
Sat, Jul 25, 6:28 PM
Unknown Object (File)
Fri, Jul 24, 8:54 PM
Unknown Object (File)
Fri, Jul 24, 5:23 PM
Unknown Object (File)
Wed, Jul 22, 4:59 PM
Unknown Object (File)
Wed, Jul 22, 11:41 AM
Unknown Object (File)
Tue, Jul 21, 11:22 AM
Unknown Object (File)
Tue, Jul 21, 8:12 AM
View All 66 Files
Subscribers
imp

Details

Reviewers
emaste
markj
Commits
rGeb411cd796fe: ipfilter: Calculate the number of elements in ipf_errors
rGe73527950d5d: ipfilter: Calculate the number of elements in ipf_errors
rG7051ddc5c703: ipfilter: Calculate the number of elements in ipf_errors
rGab3c9853285b: ipfilter: Calculate the number of elements in ipf_errors
Summary

It serves no purpose to manually manage the IPF_NUM_ERRORS count.
Calculate it instead.

MFC after: 1 day

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cy created this revision.Oct 23 2025, 11:01 PM
Herald added a subscriber: imp. · View Herald TranscriptOct 23 2025, 11:01 PM
cy requested review of this revision.Oct 23 2025, 11:01 PM
Harbormaster completed remote builds in B68061: Diff 164914.Oct 23 2025, 11:01 PM
markj accepted this revision.Oct 24 2025, 12:51 PM
This revision is now accepted and ready to land.Oct 24 2025, 12:51 PM
emaste accepted this revision.Oct 24 2025, 1:28 PM

Or if you wanted you could just use nitems() in find_error instead of IPF_NUM_ERRORS

cy added a comment.Oct 24 2025, 10:36 PM
In D53308#1217827, @emaste wrote:

Or if you wanted you could just use nitems() in find_error instead of IPF_NUM_ERRORS

That would require #include <sys/param.h> in interror.c. We don't need the extra definitions (namespace pollution) from param.h.

Closed by commit rGab3c9853285b: ipfilter: Calculate the number of elements in ipf_errors (authored by cy). · Explain WhyNov 5 2025, 3:34 PM
This revision was automatically updated to reflect the committed changes.
cy added a commit: rGab3c9853285b: ipfilter: Calculate the number of elements in ipf_errors.
cy added a commit: rG7051ddc5c703: ipfilter: Calculate the number of elements in ipf_errors.Nov 14 2025, 4:10 AM
cy added a commit: rGe73527950d5d: ipfilter: Calculate the number of elements in ipf_errors.
cy added a commit: rGeb411cd796fe: ipfilter: Calculate the number of elements in ipf_errors.

Revision Contents
Changeset List

PathSize
sbin/
ipf/
libipf/
4 lines

Diff 165875

View Options

sbin/ipf/libipf/interror.c

Loading...