Differential D57071

tftp: Close files when we're done with them
ClosedPublic
Actions

Authored by des on May 18 2026, 9:34 PM.

Tags

None

Referenced Files

	F170768501: D57071.id178039.diff
	Sun, Sep 6, 12:52 PM

	F170767903: D57071.id.diff
	Sun, Sep 6, 12:46 PM

	Unknown Object (File)
	Sun, Sep 6, 12:34 AM

	Unknown Object (File)
	Sat, Sep 5, 6:56 PM

	Unknown Object (File)
	Sat, Sep 5, 7:18 AM

	Unknown Object (File)
	Sat, Sep 5, 12:39 AM

	Unknown Object (File)
	Thu, Sep 3, 5:00 PM

	Unknown Object (File)
	Thu, Sep 3, 8:24 AM

View All 75 Files

Subscribers

Details

Reviewers

Commits

rGddd16aef6b03: tftp: Close files when we're done with them
rG66a6b7e7e92f: tftp: Close files when we're done with them
rG29351335704e: tftp: Close files when we're done with them

Summary

Also, delete the file we created if receiving it failed.

MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.May 18 2026, 9:34 PM

Herald added a subscriber: imp. · View Herald TranscriptMay 18 2026, 9:34 PM

des requested review of this revision.May 18 2026, 9:34 PM

Harbormaster completed remote builds in B73201: Diff 178039.May 18 2026, 9:34 PM

des added a parent revision: D57070: tftp: Simplify URI handling.May 18 2026, 9:34 PM

des added a child revision: D57072: tftp: Replace fgets with getline.

markj accepted this revision.May 19 2026, 1:05 PM

This revision is now accepted and ready to land.May 19 2026, 1:05 PM

des removed a child revision: D57072: tftp: Replace fgets with getline.May 19 2026, 9:58 PM

des added a child revision: D57105: tftp: Fix handling of port name or number.

Closed by commit rG29351335704e: tftp: Close files when we're done with them (authored by des). · Explain WhyMay 22 2026, 5:58 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rG29351335704e: tftp: Close files when we're done with them.

des added a commit: rG66a6b7e7e92f: tftp: Close files when we're done with them.May 27 2026, 9:03 AM

des added a commit: rGddd16aef6b03: tftp: Close files when we're done with them.

Revision Contents
Changeset List

Path

Size

usr.bin/

tftp/

10 lines

Diff 178405

usr.bin/tftp/main.c

Loading...