Page MenuHomeFreeBSD
Differential D44957

tftpd: Immediately reject any request shorter than 4 bytes.
ClosedPublic
Actions

Authored by des on Thu, Apr 25, 5:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 8, 1:08 PM
Unknown Object (File)
Sun, Apr 28, 5:31 PM
Unknown Object (File)
Sat, Apr 27, 7:27 PM
Unknown Object (File)
Sat, Apr 27, 5:42 AM
Unknown Object (File)
Fri, Apr 26, 7:32 PM
Unknown Object (File)
Fri, Apr 26, 7:31 PM
Unknown Object (File)
Fri, Apr 26, 7:31 PM
Unknown Object (File)
Fri, Apr 26, 7:31 PM
View All 12 Files
Subscribers
imp

Details

Reviewers
kevans
Group Reviewers
Klara
Commits
rG9f231af307b8: tftpd: Immediately reject any request shorter than 4 bytes.
Summary

MFC after: 1 week
Sponsored by: Klara, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des created this revision.Thu, Apr 25, 5:24 PM
Herald added a subscriber: imp. · View Herald TranscriptThu, Apr 25, 5:24 PM
des requested review of this revision.Thu, Apr 25, 5:24 PM
Harbormaster completed remote builds in B57368: Diff 137679.Thu, Apr 25, 5:24 PM
des added a parent revision: D44956: tftpd: Check the server status after each test..Thu, Apr 25, 5:24 PM
kevans accepted this revision.Thu, Apr 25, 5:47 PM
This revision is now accepted and ready to land.Thu, Apr 25, 5:47 PM
des updated this revision to Diff 137683.Thu, Apr 25, 5:56 PM

forgot to include include

This revision now requires review to proceed.Thu, Apr 25, 5:56 PM
Harbormaster completed remote builds in B57371: Diff 137683.Thu, Apr 25, 5:56 PM
kevans accepted this revision.Thu, Apr 25, 6:00 PM
This revision is now accepted and ready to land.Thu, Apr 25, 6:00 PM
des updated this revision to Diff 137685.Thu, Apr 25, 6:16 PM

revert header change and hardcode 4

This revision now requires review to proceed.Thu, Apr 25, 6:16 PM
Harbormaster completed remote builds in B57373: Diff 137685.Thu, Apr 25, 6:16 PM
kevans accepted this revision.Thu, Apr 25, 6:33 PM
kevans added inline comments.
libexec/tftpd/tftpd.c
231

I'd maybe just toss in a quick /* tftphdr */ to draw the connection and make it seem less arbitrary, but I do not insist

This revision is now accepted and ready to land.Thu, Apr 25, 6:33 PM
Closed by commit rG9f231af307b8: tftpd: Immediately reject any request shorter than 4 bytes. (authored by des). · Explain WhyThu, Apr 25, 6:36 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG9f231af307b8: tftpd: Immediately reject any request shorter than 4 bytes..
des mentioned this in rG986cceda16d8: tftpd: Use `size_t` where appropriate..Thu, May 2, 9:06 AM

Revision Contents
Changeset List

PathSize
libexec/
tftpd/
tests/
19 lines
5 lines

Diff 137690

View Options

libexec/tftpd/tests/functional.c

Loading...
View Options

libexec/tftpd/tftpd.c

Loading...