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

	F163444564: D45832.id140481.diff
	Thu, Jul 23, 6:35 AM

	Unknown Object (File)
	Tue, Jul 21, 5:03 PM

	Unknown Object (File)
	Mon, Jul 20, 12:19 AM

	Unknown Object (File)
	Sat, Jul 18, 7:48 PM

	Unknown Object (File)
	Tue, Jul 7, 3:57 AM

	Unknown Object (File)
	Sun, Jul 5, 12:17 PM

	Unknown Object (File)
	Sun, Jul 5, 3:33 AM

	Unknown Object (File)
	Sat, Jul 4, 6:32 AM

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