pf tests: Fix missing pf enabling
AbandonedPublic
Actions

Authored by kp on Oct 25 2023, 5:41 PM.

Tags

None

Referenced Files

	F102085452: D42363.diff
	Thu, Nov 7, 10:40 AM

	Unknown Object (File)
	Sep 9 2024, 9:00 AM

	Unknown Object (File)
	Sep 8 2024, 10:26 PM

	Unknown Object (File)
	Sep 8 2024, 3:43 AM

	Unknown Object (File)
	Sep 7 2024, 11:19 AM

	Unknown Object (File)
	Aug 8 2024, 8:33 AM

	Unknown Object (File)
	Jul 25 2024, 7:57 PM

	Unknown Object (File)
	Jul 16 2024, 11:57 AM

Subscribers

Details

Reviewers

vegeta_tuxpowered.net

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

vegeta_tuxpowered.net created this revision.Oct 25 2023, 5:41 PM

Herald added subscribers: glebius, melifaro, farrokhi, imp. · View Herald TranscriptOct 25 2023, 5:41 PM

vegeta_tuxpowered.net requested review of this revision.Oct 25 2023, 5:41 PM

vegeta_tuxpowered.net mentioned this in D42355: pf: Fix packet reassembly.Oct 25 2023, 5:48 PM

The bug is in D42355, where we remove part of this test case, including the pfctl -e.
This fixes it, but I'm going to squash this together with the other review to keep the history simpler.

In D42363#966766, @kp wrote:

The bug is in D42355, where we remove part of this test case, including the pfctl -e.
This fixes it, but I'm going to squash this together with the other review to keep the history simpler.

Good catch! I thought it's something totally separate, my bad.

Commandeering to abandon. We included this in D42355.

kp abandoned this revision.Oct 29 2023, 9:12 AM

Revision Contents
Changeset List

Path

Size

tests/

sys/

netpfil/

pf/

fragmentation_compat.sh

1 line

Diff 129385

View Options

tests/sys/netpfil/pf/fragmentation_compat.sh

pf tests: Fix missing pf enablingAbandonedPublicActions