if_ovpn: ensure it's safe to modify the mbuf
ClosedPublic
Actions

Authored by kp on Sep 4 2024, 12:56 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Aug 15, 11:09 AM

	Unknown Object (File)
	Sat, Aug 15, 11:09 AM

	Unknown Object (File)
	Sat, Aug 15, 11:09 AM

	Unknown Object (File)
	Sat, Aug 15, 11:09 AM

	Unknown Object (File)
	Sat, Aug 15, 11:08 AM

	Unknown Object (File)
	Fri, Aug 7, 10:06 PM

	Unknown Object (File)
	Fri, Aug 7, 2:47 PM

	Unknown Object (File)
	Fri, Aug 7, 2:47 PM

Subscribers

Details

Reviewers

Group Reviewers

network
pfsense

Commits

rG8f2519378501: if_ovpn: ensure it's safe to modify the mbuf
rG5644e2c6d47c: if_ovpn: ensure it's safe to modify the mbuf

Summary

PR: 280036
MFC after: 1 week
Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kp created this revision.Sep 4 2024, 12:56 PM

Herald added subscribers: glebius, melifaro, ae, imp. · View Herald TranscriptSep 4 2024, 12:56 PM

kp requested review of this revision.Sep 4 2024, 12:56 PM

Harbormaster completed remote builds in B59296: Diff 142853.Sep 4 2024, 12:56 PM

Probably for network related code ENOBUFS is better than ENOMEM.

ENOMEM -> ENOBUFS

Harbormaster completed remote builds in B59297: Diff 142854.Sep 4 2024, 1:47 PM

ae accepted this revision.Sep 5 2024, 7:56 AM

This revision is now accepted and ready to land.Sep 5 2024, 7:56 AM

Closed by commit rG5644e2c6d47c: if_ovpn: ensure it's safe to modify the mbuf (authored by kp). · Explain WhySep 5 2024, 9:00 AM

This revision was automatically updated to reflect the committed changes.

kp added a commit: rG5644e2c6d47c: if_ovpn: ensure it's safe to modify the mbuf.

kp added a commit: rG8f2519378501: if_ovpn: ensure it's safe to modify the mbuf.Sep 12 2024, 11:53 AM

Revision Contents
Changeset List

Path

Size

sys/

net/

if_ovpn.c

12 lines

Diff 142959

View Options

if_ovpn: ensure it's safe to modify the mbufClosedPublicActions