Fix pfsync breaking carp
ClosedPublic
Actions

Authored by smh on Jan 18 2019, 12:13 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mar 21 2024, 11:51 AM

	Unknown Object (File)
	Jan 2 2024, 3:29 PM

	Unknown Object (File)
	Dec 23 2023, 11:39 AM

	Unknown Object (File)
	Nov 15 2023, 4:55 AM

	Unknown Object (File)
	Oct 23 2023, 1:01 AM

	Unknown Object (File)
	Oct 14 2023, 2:12 AM

	Unknown Object (File)
	Oct 11 2023, 4:07 AM

	Unknown Object (File)
	Oct 11 2023, 4:07 AM

View All 23 Files

Subscribers

farrokhi

imp

Details

Reviewers

Commits

rS343130: pf: fix pfsync breaking carp

Summary

Fix missing initialisation of sc_flags into a valid sync state on clone which breaks carp in pfsync.

This regression was introduce by 342051

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

smh created this revision.Jan 18 2019, 12:13 AM

Herald added subscribers: farrokhi, imp. · View Herald TranscriptJan 18 2019, 12:14 AM

Harbormaster completed remote builds in B22013: Diff 52954.Jan 18 2019, 12:14 AM

smh added a reviewer: kp.Jan 18 2019, 12:16 AM

Yeah, looking at my original commit I did manage to lose that flag. I’ve not tested yet, but this patch is almost certainly correct.

I’ll take a deeper look and commit later today.

This revision was not accepted when it landed; it landed in state Needs Review.Jan 18 2019, 8:20 AM

Closed by commit rS343130: pf: fix pfsync breaking carp (authored by kp). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

netpfil/

pf/

if_pfsync.c

1 line

Diff 52965

View Options

head/sys/netpfil/pf/if_pfsync.c

Fix pfsync breaking carpClosedPublicActions