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

	F102094236: D46685.diff
	Thu, Nov 7, 1:24 PM

	Unknown Object (File)
	Tue, Nov 5, 3:48 AM

	Unknown Object (File)
	Sat, Nov 2, 1:35 PM

	Unknown Object (File)
	Wed, Oct 30, 3:06 PM

	Unknown Object (File)
	Sat, Oct 19, 7:36 AM

	Unknown Object (File)
	Tue, Oct 15, 5:03 AM

	Unknown Object (File)
	Mon, Oct 14, 9:04 PM

	Unknown Object (File)
	Sep 24 2024, 4:54 AM

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