Page MenuHomeFreeBSD
Differential D58555

ppp: Avoid overflow when formatting endpoint discriminator options
ClosedPublic
Actions

Authored by markj on Jul 30 2026, 3:03 PM.
Tags
None
Referenced Files
F173203320: D58555.id183111.diff
Thu, Sep 24, 6:53 AM
F173189716: D58555.diff
Thu, Sep 24, 4:44 AM
Unknown Object (File)
Mon, Sep 21, 8:39 AM
Unknown Object (File)
Thu, Sep 17, 5:15 PM
Unknown Object (File)
Fri, Sep 11, 3:56 AM
Unknown Object (File)
Fri, Sep 11, 2:42 AM
Unknown Object (File)
Tue, Sep 8, 8:24 PM
Unknown Object (File)
Tue, Sep 8, 12:31 PM
View All 94 Files
Subscribers
imp

Details

Reviewers
jhb
Group Reviewers
secteam
Commits
rG23deefa40b8d: ppp: Avoid overflow when formatting endpoint discriminator options
rG0cd984b41c2f: ppp: Avoid overflow when formatting endpoint discriminator options
rG561c1fc29b52: ppp: Avoid overflow when formatting endpoint discriminator options
rG281fb7d8a20b: ppp: Avoid overflow when formatting endpoint discriminator options
rG607c41d8f869: ppp: Avoid overflow when formatting endpoint discriminator options
rG78dfadaa0618: ppp: Avoid overflow when formatting endpoint discriminator options
rGe004ff15f87e: ppp: Avoid overflow when formatting endpoint discriminator options
Summary

Each byte of the address is represented by a pair of characters, so we
should be multiplying len by 2 when figuring out how much buffer space
we have.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 75294
Build 72177: arc lint + arc unit

Event Timeline

markj created this revision.Jul 30 2026, 3:03 PM
Herald added a subscriber: imp. · View Herald TranscriptJul 30 2026, 3:03 PM
markj requested review of this revision.Jul 30 2026, 3:03 PM
Harbormaster completed remote builds in B75294: Diff 183111.Jul 30 2026, 3:04 PM
markj added a parent revision: D58554: ppp: Reject invalid endpoint discriminator options.Jul 30 2026, 3:04 PM
markj updated this revision to Diff 183131.Jul 30 2026, 7:31 PM

Cast bytes to unsigned char, otherwise a negative byte will be sign
extended and occupy more than two bytes of the buffer.

Harbormaster completed remote builds in B75302: Diff 183131.Jul 30 2026, 7:31 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 3 2026, 3:23 PM
Closed by commit rGe004ff15f87e: ppp: Avoid overflow when formatting endpoint discriminator options (authored by markj). · Explain Why
This revision was automatically updated to reflect the committed changes.
markj added a commit: rGe004ff15f87e: ppp: Avoid overflow when formatting endpoint discriminator options.
markj added a commit: rG78dfadaa0618: ppp: Avoid overflow when formatting endpoint discriminator options.Aug 6 2026, 3:02 PM
markj added a commit: rG607c41d8f869: ppp: Avoid overflow when formatting endpoint discriminator options.Aug 7 2026, 12:42 PM
cperciva added a commit: rG281fb7d8a20b: ppp: Avoid overflow when formatting endpoint discriminator options.Aug 20 2026, 5:54 PM
Herald added a reviewer: scheduler. · View Herald TranscriptAug 20 2026, 5:54 PM
lwhsu removed a reviewer: scheduler.Aug 20 2026, 8:12 PM
markj added a commit: rG561c1fc29b52: ppp: Avoid overflow when formatting endpoint discriminator options.Aug 25 2026, 4:01 PM
markj added a commit: rG0cd984b41c2f: ppp: Avoid overflow when formatting endpoint discriminator options.
markj added a commit: rG23deefa40b8d: ppp: Avoid overflow when formatting endpoint discriminator options.

Revision Contents
Changeset List

PathSize
usr.sbin/
ppp/
8 lines

Diff 183111

View Options

usr.sbin/ppp/mp.c

Loading...