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

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

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Sep 19, 10:09 AM

	Unknown Object (File)
	Thu, Sep 19, 5:34 AM

	Unknown Object (File)
	Wed, Sep 18, 4:37 PM

	Unknown Object (File)
	Sun, Sep 15, 9:07 PM

	Unknown Object (File)
	Tue, Sep 10, 9:10 PM

	Unknown Object (File)
	Mon, Sep 9, 10:59 AM

	Unknown Object (File)
	Mon, Sep 9, 10:59 AM

	Unknown Object (File)
	Sun, Sep 8, 8:28 AM

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.Wed, Sep 4, 12:56 PM

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

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

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

Probably for network related code ENOBUFS is better than ENOMEM.

ENOMEM -> ENOBUFS

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

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

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

Closed by commit rG5644e2c6d47c: if_ovpn: ensure it's safe to modify the mbuf (authored by kp). · Explain WhyThu, Sep 5, 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.Thu, Sep 12, 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