Paths

Table of Contentst

pf: Migrate to asserted mtod_()
AbandonedPublic
Actions

Authored by igoro on Sep 16 2024, 6:47 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, May 9, 3:25 PM

	Unknown Object (File)
	Sat, May 9, 3:25 PM

	Unknown Object (File)
	Sat, May 9, 3:25 PM

	Unknown Object (File)
	Fri, May 8, 12:44 AM

	Unknown Object (File)
	Wed, May 6, 1:25 PM

	Unknown Object (File)
	Mon, Apr 27, 7:29 AM

	Unknown Object (File)
	Mar 28 2026, 9:08 AM

	Unknown Object (File)
	Mar 27 2026, 10:47 PM

Subscribers

vegeta_tuxpowered.net

Details

Reviewers

kp
markj

Summary

This is an example of https://reviews.freebsd.org/D46684 usage. Only part of pf is changed for demo purposes.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 59506
Build 56393: arc lint + arc unit

Event Timeline

igoro created this revision.Sep 16 2024, 6:47 PM

Herald added subscribers: vegeta_tuxpowered.net, glebius, melifaro and 2 others. · View Herald TranscriptSep 16 2024, 6:47 PM

igoro requested review of this revision.Sep 16 2024, 6:47 PM

Harbormaster completed remote builds in B59506: Diff 143393.Sep 16 2024, 6:47 PM

igoro mentioned this in D46684: mbuf: Add m_len assertion to mtod() and mtodo().Sep 16 2024, 6:47 PM

igoro edited the summary of this revision. (Show Details)

According to the https://reviews.freebsd.org/D46684 it's agreed to work towards the global change of existing mtod() and mtodo().

Revision Contents
Changeset List

Path

Size

sys/

netpfil/

pf/

pf.c

32 lines

Diff 143393

View Options

pf: Migrate to asserted mtod_()AbandonedPublicActions