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)
	Dec 9 2024, 2:47 AM

	Unknown Object (File)
	Nov 25 2024, 12:21 PM

	Unknown Object (File)
	Nov 23 2024, 2:10 PM

	Unknown Object (File)
	Nov 21 2024, 3:02 AM

	Unknown Object (File)
	Nov 20 2024, 11:41 AM

	Unknown Object (File)
	Nov 19 2024, 10:06 AM

	Unknown Object (File)
	Nov 17 2024, 7:55 AM

	Unknown Object (File)
	Nov 5 2024, 9:08 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