Page MenuHomeFreeBSD
Differential D5157

Patch for bug 100219
ClosedPublic
Actions

Authored by joe-thestackframe.org on Feb 1 2016, 7:30 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 17, 4:16 AM
Unknown Object (File)
Thu, Oct 16, 7:47 AM
Unknown Object (File)
Wed, Oct 8, 9:39 AM
Unknown Object (File)
Thu, Sep 25, 11:36 PM
Unknown Object (File)
Thu, Sep 25, 11:34 PM
Unknown Object (File)
Thu, Sep 25, 6:39 AM
Unknown Object (File)
Aug 26 2025, 6:48 PM
Unknown Object (File)
Jul 26 2025, 8:08 PM
View All Files
Subscribers
None

Details

Reviewers
gnn
bz
rstone
Commits
rS307541: Limit the number of mbufs that can be allocated for IPV6_2292PKTOPTIONS
Summary

Limit the number of mbufs that can be allocated for IPV6_2292PKTOPTIONS (and IPV6_PKTOPTIONS).

Patch for bug 100219:

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=100219

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joe-thestackframe.org updated this revision to Diff 12921.Feb 1 2016, 7:30 AM
joe-thestackframe.org retitled this revision from to Patch for bug 100219.
joe-thestackframe.org updated this object.
joe-thestackframe.org edited the test plan for this revision. (Show Details)
joe-thestackframe.org added reviewers: gnn, bz.
joe-thestackframe.org added a reviewer: rstone.Jul 29 2016, 9:01 PM
rstone requested changes to this revision.Jul 30 2016, 2:33 AM
rstone edited edge metadata.
rstone added inline comments.
sys/netinet6/ip6_output.c
1382

Should probably wrap the value in () to prevent weird order-of-operation issues from ever cropping up in the future. There isn't any in the current code but who knows how this could be used in the future.

This revision now requires changes to proceed.Jul 30 2016, 2:33 AM
joe-thestackframe.org updated this revision to Diff 21439.Oct 17 2016, 7:40 AM
joe-thestackframe.org edited edge metadata.

Add brackets around value.

joe-thestackframe.org marked an inline comment as done.Oct 17 2016, 7:42 AM
gnn accepted this revision.Oct 17 2016, 12:16 PM
gnn edited edge metadata.

Looks good to me. I can commit this if you like.

joe-thestackframe.org added a comment.Oct 17 2016, 6:09 PM
In D5157#171934, @gnn wrote:

Looks good to me. I can commit this if you like.

Yeah, that would be great. Thanks.

Closed by commit rS307541: Limit the number of mbufs that can be allocated for IPV6_2292PKTOPTIONS (authored by gnn). · Explain WhyOct 17 2016, 11:25 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
netinet6/
ip6_output.c
ip6_output.c.orig
15 lines

Diff 12921

View Options

sys/netinet6/ip6_output.c

Loading...