Page MenuHomeFreeBSD
Differential D24602

Remove erroneous spacing in SCTP_ABORT_ASSOCIATION case statement comment in send_reject6()
ClosedPublic
Actions

Authored by nc on Apr 28 2020, 2:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 8, 5:48 PM
Unknown Object (File)
Sun, Dec 7, 7:45 AM
Unknown Object (File)
Sat, Dec 6, 7:34 AM
Unknown Object (File)
Thu, Dec 4, 9:43 AM
Unknown Object (File)
Tue, Dec 2, 8:41 AM
Unknown Object (File)
Sun, Nov 30, 5:26 PM
Unknown Object (File)
Sat, Nov 29, 3:39 PM
Unknown Object (File)
Sat, Nov 29, 11:34 AM
View All Files
Subscribers
ae
imp
melifaro
Contributor Reviews (src)

Details

Reviewers
emaste
rgrimes
tuexen
Commits
rS360594: ipfw: whitespace fix in SCTP_ABORT_ASSOCIATION case statement comment
Summary

Remove erroneous spacing in SCTP_ABORT_ASSOCIATION case statement comment in send_reject6(). This ensures consistency with send_reject().

While we're here, fix the grammar in both SCTP_ABORT_ASSOCIATION case comments.

Submitted by: Neel Chauhan <neel AT neelc DOT org>

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

nc created this revision.Apr 28 2020, 2:57 AM
Herald added subscribers: Contributor Reviews (src), melifaro, ae, imp. · View Herald TranscriptApr 28 2020, 2:57 AM
nc requested review of this revision.Apr 28 2020, 2:57 AM
nc added reviewers: emaste, rgrimes.May 1 2020, 6:38 PM
emaste added a reviewer: tuexen.May 1 2020, 6:43 PM
tuexen accepted this revision.May 1 2020, 7:01 PM

I put is in the way it is now. I agree, this is not consistent with the rest of the file.

This revision is now accepted and ready to land.May 1 2020, 7:01 PM
rgrimes added inline comments.May 2 2020, 5:44 AM
sys/netpfil/ipfw/ip_fw2.c
940

don't -> do not, and.. do not do what??? This is not a valid sentence, subject of verb do is missing.

tuexen added inline comments.May 2 2020, 8:24 AM
sys/netpfil/ipfw/ip_fw2.c
940

it should read "but do not do that to avoid attacks". And I guess the same text appears twice in this file. So please fix it twice.

nc updated this revision to Diff 71288.May 2 2020, 4:49 PM
nc edited the summary of this revision. (Show Details)

Thanks for the suggestions. I have made the requested changes here.

This revision now requires review to proceed.May 2 2020, 4:49 PM
nc marked 2 inline comments as done.May 2 2020, 4:50 PM
tuexen accepted this revision.May 2 2020, 5:36 PM
This revision is now accepted and ready to land.May 2 2020, 5:36 PM
rgrimes accepted this revision.May 3 2020, 2:19 AM
Closed by commit rS360594: ipfw: whitespace fix in SCTP_ABORT_ASSOCIATION case statement comment (authored by emaste). · Explain WhyMay 3 2020, 3:44 AM
This revision was automatically updated to reflect the committed changes.
emaste added a commit: rS360594: ipfw: whitespace fix in SCTP_ABORT_ASSOCIATION case statement comment.

Revision Contents
Changeset List

PathSize
sys/
netpfil/
ipfw/
2 lines

Diff 71091

View Options

sys/netpfil/ipfw/ip_fw2.c

Loading...