Page MenuHomeFreeBSD

kp (Kristof Provost)
Troubleshooter

Projects (6)

User Details

User Since
Sep 28 2014, 7:22 PM (561 w, 1 d)

Recent Activity

Yesterday

kp committed rG5c0eb439cfad: pfctl: Move AF-specific mask logic from callers into set_ipmask() (authored by kp).
pfctl: Move AF-specific mask logic from callers into set_ipmask()
Mon, Jun 30, 3:01 PM
kp committed rGe790bcd784c9: pfctl: Use error label in host_if() (authored by kp).
pfctl: Use error label in host_if()
Mon, Jun 30, 3:01 PM
kp committed rGeb6c2218fe64: pfctl: Zap bits in host_v4(), use mask parameter (authored by kp).
pfctl: Zap bits in host_v4(), use mask parameter
Mon, Jun 30, 3:01 PM
kp committed rGa59e7964d58a: pfctl: Zap v4mask and v6mask in host() (authored by kp).
pfctl: Zap v4mask and v6mask in host()
Mon, Jun 30, 3:01 PM
kp committed rG5d9877ba8f5a: pfctl: Simplify host() (authored by kp).
pfctl: Simplify host()
Mon, Jun 30, 3:01 PM
kp committed rG7b82e3620367: pfctl: Use strtonum in host() (authored by kp).
pfctl: Use strtonum in host()
Mon, Jun 30, 3:01 PM
kp committed rG466ac79e47fe: pfctl: Move duplicate code into new helper print_addr_str() (authored by kp).
pfctl: Move duplicate code into new helper print_addr_str()
Mon, Jun 30, 3:01 PM
kp committed rGb728aaa12cbe: pfctl: Simplify getaddrinfo() error handling (authored by kp).
pfctl: Simplify getaddrinfo() error handling
Mon, Jun 30, 3:01 PM
kp committed rGd1ac3e245f08: wg: fix LINT-NOIP build (authored by kp).
wg: fix LINT-NOIP build
Mon, Jun 30, 9:56 AM
kp committed rGe56247e0debc: pfctl: Stop checking table commands for `create' (authored by kp).
pfctl: Stop checking table commands for `create'
Mon, Jun 30, 9:56 AM
kp committed rGd8d7bd50cb88: pfctl: simplify FOM_PRIO handling (authored by kp).
pfctl: simplify FOM_PRIO handling
Mon, Jun 30, 9:56 AM
kp committed rGd18ed24a9e9b: pf: trade few 'goto unlock: for 'break' in pf_test() (authored by kp).
pf: trade few 'goto unlock: for 'break' in pf_test()
Mon, Jun 30, 9:56 AM
kp committed rG8572367b6814: pf: remove STATE_LOOKUP (authored by kp).
pf: remove STATE_LOOKUP
Mon, Jun 30, 9:56 AM
kp committed rGe7be8e0026ad: pfctl: use __func__ rather than hardcoding function names for errors (authored by kp).
pfctl: use __func__ rather than hardcoding function names for errors
Mon, Jun 30, 9:56 AM
kp committed rG512f136f39c4: pf.conf.5: reflect the new state limit (authored by kp).
pf.conf.5: reflect the new state limit
Mon, Jun 30, 9:56 AM
kp committed rGe03181510de2: pfctl: Add "listenrepv2" for MLDv2 Listener Reports from RFC3810 (authored by kp).
pfctl: Add "listenrepv2" for MLDv2 Listener Reports from RFC3810
Mon, Jun 30, 9:55 AM
kp committed rG23c17bd60904: pf tests: verify max-pkt-rate on anchors (authored by kp).
pf tests: verify max-pkt-rate on anchors
Mon, Jun 30, 9:55 AM
kp committed rG13358e47edbc: pfctl: fix anchor rules with filter opts, introduce filteropts_to_rule() (authored by kp).
pfctl: fix anchor rules with filter opts, introduce filteropts_to_rule()
Mon, Jun 30, 9:55 AM

Fri, Jun 27

kp committed rG5f739302ba0a: pfctl: Zap dead code (authored by kp).
pfctl: Zap dead code
Fri, Jun 27, 3:17 PM
kp committed rG6ccd82f8deb5: pfctl: clean up allocation warnings (authored by kp).
pfctl: clean up allocation warnings
Fri, Jun 27, 3:17 PM
kp committed rGc31621c125ee: pfctl: fix memory leak (authored by kp).
pfctl: fix memory leak
Fri, Jun 27, 3:17 PM
kp committed rG753da351598a: pfctl: plug some memory leaks (authored by kp).
pfctl: plug some memory leaks
Fri, Jun 27, 3:17 PM
kp committed rGc114db294d5d: pf: Refactor the six ways to find TCP options into one new function. (authored by kp).
pf: Refactor the six ways to find TCP options into one new function.
Fri, Jun 27, 3:17 PM
kp committed rGe2d2aadc292a: pfctl tests: test line number reporting in include files (authored by kp).
pfctl tests: test line number reporting in include files
Fri, Jun 27, 3:17 PM
kp committed rGd40166400955: pfctl: fix reporting of line numbers for included files (authored by kp).
pfctl: fix reporting of line numbers for included files
Fri, Jun 27, 3:17 PM
kp committed rG41fd03c08f67: pf: add 'max-pkt-size' (authored by kp).
pf: add 'max-pkt-size'
Fri, Jun 27, 3:17 PM
kp committed rG0c273335b2de: pf: ensure max-pkt-size works on match rules (authored by kp).
pf: ensure max-pkt-size works on match rules
Fri, Jun 27, 3:17 PM
kp accepted D51064: pfsync: Allocate and initialize buckets before attaching the interface.
Fri, Jun 27, 11:50 AM
kp accepted D51063: pfsync: Destroy mutexes on clone destroying interface.
Fri, Jun 27, 11:49 AM
kp committed rGe7abf8829d8d: pf: fix ICMP ECHO handling of ID conflicts (authored by Damir Bikmuhametov <boco@ufanet.ru>).
pf: fix ICMP ECHO handling of ID conflicts
Fri, Jun 27, 11:15 AM
kp committed rG32f793e22976: pf tests: test handling of ICMP echo requests with the same ID (authored by kp).
pf tests: test handling of ICMP echo requests with the same ID
Fri, Jun 27, 11:15 AM

Thu, Jun 26

kp committed rG5c2d3093b27f: pfctl: Plug leak in error case of the common 'varset' implementations. (authored by kp).
pfctl: Plug leak in error case of the common 'varset' implementations.
Thu, Jun 26, 1:12 PM
kp committed rGfed2e93b0a00: pf: fix regression in pflog output (authored by kp).
pf: fix regression in pflog output
Thu, Jun 26, 1:12 PM
kp committed rGad591caf2a70: pf: decrement TTL in pf_route(6)() (authored by kp).
pf: decrement TTL in pf_route(6)()
Thu, Jun 26, 1:12 PM
kp committed rG22e8dc82ff0c: pf: Use pf_send_icmp() consistently in pf_route() (authored by kp).
pf: Use pf_send_icmp() consistently in pf_route()
Thu, Jun 26, 1:12 PM
kp committed rG9ab84b78caaf: pf: disallow IPv6 routing header by default (authored by kp).
pf: disallow IPv6 routing header by default
Thu, Jun 26, 1:12 PM
kp committed rG230ae7a67665: pfctl: fails to handle nested 'load anchor' properly (authored by kp).
pfctl: fails to handle nested 'load anchor' properly
Thu, Jun 26, 1:12 PM
kp committed rGa62c14538100: pf: drop neighbor discovery packets with the wrong hop limit (authored by kp).
pf: drop neighbor discovery packets with the wrong hop limit
Thu, Jun 26, 1:12 PM

Wed, Jun 25

kp committed rGcd0169c9379c: pf: limit extra SCTP states (authored by kp).
pf: limit extra SCTP states
Wed, Jun 25, 8:05 PM
kp committed rGa7d631f69d3f: pfctl: fix use-after-free and memory leak in pfctl_optimzie.c (authored by kp).
pfctl: fix use-after-free and memory leak in pfctl_optimzie.c
Wed, Jun 25, 8:05 PM
kp committed rG809ba93c689f: pfctl: rule optimizer: anchor name vs. anchor path mix up (authored by kp).
pfctl: rule optimizer: anchor name vs. anchor path mix up
Wed, Jun 25, 8:05 PM
kp committed rGc5f21023771a: pf tests: basic max-pkt-rate test (authored by kp).
pf tests: basic max-pkt-rate test
Wed, Jun 25, 8:05 PM
kp committed rGc7adc9ad973e: pfctl tests: max-pkt-rate test (authored by kp).
pfctl tests: max-pkt-rate test
Wed, Jun 25, 8:05 PM
kp committed rG87adb6ab7660: pfctl: nested anchors vs. pfctl/parse.y (authored by kp).
pfctl: nested anchors vs. pfctl/parse.y
Wed, Jun 25, 8:05 PM
kp committed rGeb01c0dfe275: pf.conf.5: tweak max-pkt-rate (authored by kp).
pf.conf.5: tweak max-pkt-rate
Wed, Jun 25, 8:05 PM
kp committed rGff11f1c8c76c: pf: add a generic packet rate matching filter (authored by kp).
pf: add a generic packet rate matching filter
Wed, Jun 25, 5:57 PM
kp committed rG53a341d0e445: pf: use counter_rate() for rate checking (authored by kp).
pf: use counter_rate() for rate checking
Wed, Jun 25, 5:57 PM
kp committed rG1cd5c35d136e: counter(9): rate limit periods may be more than 1 second (authored by kp).
counter(9): rate limit periods may be more than 1 second
Wed, Jun 25, 5:57 PM
kp closed D50797: pf: use counter_rate() for rate checking.
Wed, Jun 25, 5:57 PM
kp closed D50796: counter(9): rate limit periods may be more than 1 second.
Wed, Jun 25, 5:57 PM

Mon, Jun 23

kp added a comment to D50796: counter(9): rate limit periods may be more than 1 second.
In D50796#1162649, @kp wrote:

Unless Jonathan remembers something I'd like to go with this version.

Mon, Jun 23, 1:50 PM

Sat, Jun 21

kp requested review of D50968: pf: limit extra SCTP states.
Sat, Jun 21, 7:19 PM
kp committed rG2fe78d74faa5: pf tests: do not shutdown the sctp server in sctp:related_icmp (authored by kp).
pf tests: do not shutdown the sctp server in sctp:related_icmp
Sat, Jun 21, 12:07 PM

Thu, Jun 19

kp accepted D50938: sys: Correct kern.osreldate description.
Thu, Jun 19, 2:29 PM
kp added a comment to D50796: counter(9): rate limit periods may be more than 1 second.

I'm not sure about making the structure opaque and thus only dynamically allocated. In general I prefer hiding internals, but for some reason when we did this, we decided to make it exposed. There definitely was some reason for that, otherwise I would make it opaque. Adding Jonathan, maybe he can recall.

Thu, Jun 19, 8:49 AM
kp updated the diff for D50796: counter(9): rate limit periods may be more than 1 second.

man page

Thu, Jun 19, 8:46 AM

Mon, Jun 16

kp added a comment to D50859: pf.4/pf.conf.5: Remove unused sysctl.

Oh sure, blame me for the commit message ;)

Mon, Jun 16, 12:05 PM
kp accepted D50856: pf.4/pfsync.4: Separate sysctl/tunables >> SYNOPSIS.
Mon, Jun 16, 11:44 AM
kp accepted D50859: pf.4/pf.conf.5: Remove unused sysctl.

The change is good, but I'd rephrase the commit message slightly. We never actually added the sysctl. The limit was implemented, and the sysctl to tune it was documented but never added (so the limit is there, but is always 16 and cannot be changed).

Mon, Jun 16, 11:42 AM

Sat, Jun 14

kp added inline comments to D50856: pf.4/pfsync.4: Separate sysctl/tunables >> SYNOPSIS.
Sat, Jun 14, 7:34 PM
kp added inline comments to D50856: pf.4/pfsync.4: Separate sysctl/tunables >> SYNOPSIS.
Sat, Jun 14, 6:35 PM
kp added a comment to D50825: libexec/kgdb: Add a new VNET function and add more scaffolding.

While testing I found that this has issues with vnet variables in kernel modules.

Sat, Jun 14, 2:30 AM

Fri, Jun 13

kp closed D50819: qlnx: fix panic at startup.
Fri, Jun 13, 2:50 PM
kp committed rG4f7473d18a62: qlnx: fix panic at startup (authored by kp).
qlnx: fix panic at startup
Fri, Jun 13, 2:50 PM

Thu, Jun 12

kp requested review of D50819: qlnx: fix panic at startup.
Thu, Jun 12, 5:00 PM
kp committed R11:d59c86f1ef2e: net/libpfctl: add 14.3 version (authored by kp).
net/libpfctl: add 14.3 version
Thu, Jun 12, 1:09 PM

Wed, Jun 11

kp added a comment to D50796: counter(9): rate limit periods may be more than 1 second.

May I ask to wait for my review until next week?

Wed, Jun 11, 8:53 PM
kp requested review of D50798: pf: add a generic packet rate matching filter.
Wed, Jun 11, 7:40 PM
kp requested review of D50797: pf: use counter_rate() for rate checking.
Wed, Jun 11, 7:40 PM
kp requested review of D50796: counter(9): rate limit periods may be more than 1 second.
Wed, Jun 11, 7:40 PM
kp accepted D50577: if_bridge_test: add a test for QinQ (802.1ad).
Wed, Jun 11, 6:04 PM
kp added a comment to D50781: WIP: pf: Add RFC5549 support for route-to.

I'm going to need to take a deeper look later (probably when I'm back home from BSDCan). First impressions are that the implementation is probably good, but I really dislike the 'RFC5549' naming of the flag and variables. Unfortunately I don't immediately have a better suggestion.

Wed, Jun 11, 2:27 PM
kp accepted D50779: pf: Prevent infinite looping over tables in round-robin pools.

Approved.

Wed, Jun 11, 1:59 PM

Tue, Jun 10

kp accepted D50768: pf: Don't access sk and nk before they are allocated.

Approved.

Tue, Jun 10, 7:57 PM
kp committed rG79005ea537d2: sys: add LINT-NOVIMAGE (authored by kp).
sys: add LINT-NOVIMAGE
Tue, Jun 10, 7:53 PM
kp closed D50780: sys: add LINT-NOVIMAGE.
Tue, Jun 10, 7:53 PM
kp accepted D50764: pf tests: Add jail configuration for route_to and rfc5549 tests.

Approved, but please only commit it along with a test case that uses this setup code.

Tue, Jun 10, 6:47 PM
kp requested review of D50780: sys: add LINT-NOVIMAGE.
Tue, Jun 10, 6:33 PM
kp added inline comments to D50763: pf: Fix error handling when pf_map_addr() fails.
Tue, Jun 10, 5:12 PM
kp accepted D50762: pf: Don't return src node and hash from pf_map_addr_sn.

Approved.

Tue, Jun 10, 5:07 PM

Mon, Jun 9

kp committed rGf96f838114a2: pf tests: frag6.py:TestFrag6_RouteTo::test_too_big requires scapy (authored by kp).
pf tests: frag6.py:TestFrag6_RouteTo::test_too_big requires scapy
Mon, Jun 9, 9:07 PM
kp committed rG54a547fcb47c: pf: split pf_find_or_create_ruleset() into smaller chunks (authored by kp).
pf: split pf_find_or_create_ruleset() into smaller chunks
Mon, Jun 9, 9:07 PM
kp committed rGecc175e984d9: pf tests: killstate:key requires scapy (authored by kp).
pf tests: killstate:key requires scapy
Mon, Jun 9, 9:07 PM
kp committed rGdefc181278cc: pf: reorganise fragment reassembly (authored by kp).
pf: reorganise fragment reassembly
Mon, Jun 9, 9:07 PM
kp committed rG4ace4ea9ca6e: pfctl: add option -S (no domain resolution) (authored by kp).
pfctl: add option -S (no domain resolution)
Mon, Jun 9, 9:07 PM
kp committed rG67d1ea0cfeb4: pf: use time_uptime rather than time_seconds (authored by kp).
pf: use time_uptime rather than time_seconds
Mon, Jun 9, 9:07 PM
kp closed D50725: pf: split pf_find_or_create_ruleset() into smaller chunks.
Mon, Jun 9, 9:07 PM
kp closed D50724: pfctl: add option -S (no domain resolution).
Mon, Jun 9, 9:07 PM
kp closed D50723: pf: reorganise fragment reassembly.
Mon, Jun 9, 9:07 PM
kp closed D50722: pf: use time_uptime rather than time_seconds.
Mon, Jun 9, 9:07 PM

Fri, Jun 6

kp committed rG06664c3b0ec2: pf: fix panic in pf_return() (authored by kp).
pf: fix panic in pf_return()
Fri, Jun 6, 9:23 PM
kp committed rG6f6ca5288060: pf: fix panic in pf_return() (authored by kp).
pf: fix panic in pf_return()
Fri, Jun 6, 9:21 PM
kp requested review of D50725: pf: split pf_find_or_create_ruleset() into smaller chunks.
Fri, Jun 6, 2:25 PM
kp requested review of D50724: pfctl: add option -S (no domain resolution).
Fri, Jun 6, 2:25 PM
kp requested review of D50723: pf: reorganise fragment reassembly.
Fri, Jun 6, 2:25 PM
kp requested review of D50722: pf: use time_uptime rather than time_seconds.
Fri, Jun 6, 2:25 PM
kp committed rGfeac17714fdf: pfctl: void functions and exit(3) on error (authored by kp).
pfctl: void functions and exit(3) on error
Fri, Jun 6, 1:00 PM
kp committed rG6ea237c31eff: pf.conf.5: clarify filter evaluation and anchor loading (authored by kp).
pf.conf.5: clarify filter evaluation and anchor loading
Fri, Jun 6, 1:00 PM
kp committed rG834d72b555b5: pf.conf.5: clarify set prio (authored by kp).
pf.conf.5: clarify set prio
Fri, Jun 6, 1:00 PM
kp committed rG311ad5bc811d: UPDATING: document recent pf changes (authored by kp).
UPDATING: document recent pf changes
Fri, Jun 6, 11:18 AM