tftpd: Don't consume arbitrary requests when failing to fork.
ClosedPublic
Actions

Authored by des on Mar 7 2023, 11:04 PM.

Tags

None

Referenced Files

	F153760337: D38967.id.diff
	Thu, Apr 23, 12:20 PM

	F153716456: D38967.id118621.diff
	Thu, Apr 23, 4:20 AM

	F153668133: D38967.id118503.diff
	Wed, Apr 22, 7:09 PM

	Unknown Object (File)
	Tue, Apr 21, 5:34 AM

	Unknown Object (File)
	Thu, Apr 9, 5:06 PM

	Unknown Object (File)
	Thu, Apr 9, 10:06 AM

	Unknown Object (File)
	Thu, Apr 9, 6:28 AM

	Unknown Object (File)
	Mar 15 2026, 2:48 AM

View All Files

Subscribers

None

Details

Reviewers

imp
jhb
markj

Group Reviewers

Klara

Commits

rGb4736c90adb8: tftpd: Don't consume arbitrary requests when failing to fork.

Summary

We've already consumed one request, which is sufficient to prevent inetd from endlessly restarting us in this particular and extremely unlikely case.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des requested review of this revision.Mar 7 2023, 11:04 PM

des created this revision.

Harbormaster completed remote builds in B50214: Diff 118503.Mar 7 2023, 11:04 PM

des added a parent revision: D38966: tftpd: Use poll() instead of alarm() + setjmp()..Mar 7 2023, 11:04 PM

des added a child revision: D38968: tftpd: Don't forget to close stderr on startup..

markj accepted this revision as: markj.Mar 8 2023, 2:56 PM

This revision is now accepted and ready to land.Mar 8 2023, 2:56 PM

Closed by commit rGb4736c90adb8: tftpd: Don't consume arbitrary requests when failing to fork. (authored by des). · Explain WhyMar 10 2023, 1:29 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGb4736c90adb8: tftpd: Don't consume arbitrary requests when failing to fork..

Revision Contents
Changeset List

Path

Size

libexec/

tftpd/

tftpd.c

52 lines

Diff 118621

View Options

tftpd: Don't consume arbitrary requests when failing to fork.ClosedPublicActions