Page MenuHomeFreeBSD
Differential D58555

ppp: Avoid overflow when formatting endpoint discriminator options
ClosedPublic
Actions

Authored by markj on Thu, Jul 30, 3:03 PM.
Tags
None
Referenced Files
F165322319: D58555.id183317.diff
Fri, Aug 7, 3:34 PM
F165286998: D58555.id183317.diff
Fri, Aug 7, 10:33 AM
F165286984: D58555.id183131.diff
Fri, Aug 7, 10:33 AM
F165286982: D58555.id183111.diff
Fri, Aug 7, 10:33 AM
F165286981: D58555.id.diff
Fri, Aug 7, 10:33 AM
F165286902: D58555.diff
Fri, Aug 7, 10:32 AM
F165229325: D58555.id183317.diff
Fri, Aug 7, 12:36 AM
Unknown Object (File)
Thu, Aug 6, 5:45 PM
View All 19 Files
Subscribers
imp

Details

Reviewers
jhb
Group Reviewers
secteam
Commits
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 Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj created this revision.Thu, Jul 30, 3:03 PM
Herald added a subscriber: imp. · View Herald TranscriptThu, Jul 30, 3:03 PM
markj requested review of this revision.Thu, Jul 30, 3:03 PM
Harbormaster completed remote builds in B75294: Diff 183111.Thu, Jul 30, 3:04 PM
markj added a parent revision: D58554: ppp: Reject invalid endpoint discriminator options.Thu, Jul 30, 3:04 PM
markj updated this revision to Diff 183131.Thu, Jul 30, 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.Thu, Jul 30, 7:31 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mon, Aug 3, 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.Thu, Aug 6, 3:02 PM
markj added a commit: rG607c41d8f869: ppp: Avoid overflow when formatting endpoint discriminator options.Fri, Aug 7, 12:42 PM

Revision Contents
Changeset List

PathSize
usr.sbin/
ppp/
12 lines

Diff 183317

View Options

usr.sbin/ppp/mp.c

Loading...