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

	F146758874: D38967.diff
	Thu, Mar 5, 8:49 AM

	F146731532: D38967.diff
	Thu, Mar 5, 2:53 AM

	Unknown Object (File)
	Tue, Mar 3, 10:36 AM

	Unknown Object (File)
	Sat, Feb 28, 8:16 PM

	Unknown Object (File)
	Mon, Feb 23, 12:43 AM

	Unknown Object (File)
	Sun, Feb 8, 3:40 AM

	Unknown Object (File)
	Jan 31 2026, 5:09 AM

	Unknown Object (File)
	Jan 21 2026, 3:37 AM

View All 95 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