pf: Fix NULL deref in m_pullup failure case
ClosedPublic
Actions

Authored by igoro on Jul 2 2024, 4:29 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Nov 25, 11:18 AM

	Unknown Object (File)
	Mon, Nov 17, 9:04 AM

	Unknown Object (File)
	Mon, Nov 17, 2:52 AM

	Unknown Object (File)
	Thu, Oct 30, 3:18 AM

	Unknown Object (File)
	Oct 24 2025, 10:22 PM

	Unknown Object (File)
	Oct 20 2025, 1:25 AM

	Unknown Object (File)
	Oct 13 2025, 10:35 PM

	Unknown Object (File)
	Oct 13 2025, 10:35 PM

Subscribers

vegeta_tuxpowered.net

Details

Reviewers

kp
glebius

Commits

rG78918c9b271c: pf: Fix NULL deref in m_pullup failure case

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

igoro created this revision.Jul 2 2024, 4:29 PM

Herald added subscribers: vegeta_tuxpowered.net, glebius, melifaro and 2 others. · View Herald TranscriptJul 2 2024, 4:29 PM

igoro requested review of this revision.Jul 2 2024, 4:29 PM

After the main change landing brain starts thinking outside of the box. We could save the m_len before the m_pullup for the sake of a detailed error report, but it's not felt to be very important. Let me know if it is.

I don't think there is any value in saving and printing m_len.

This revision is now accepted and ready to land.Jul 2 2024, 4:38 PM

Closed by commit rG78918c9b271c: pf: Fix NULL deref in m_pullup failure case (authored by igoro, committed by kp). · Explain WhyJul 2 2024, 7:59 PM

This revision was automatically updated to reflect the committed changes.

kp added a commit: rG78918c9b271c: pf: Fix NULL deref in m_pullup failure case.

Revision Contents
Changeset List

Path

Size

sys/

netpfil/

pf/

pf.c

3 lines

Diff 140481

View Options

pf: Fix NULL deref in m_pullup failure caseClosedPublicActions