libsa: validate tftp_makereq() after we did reset the read
ClosedPublic
Actions

Authored by tsoome on Sep 8 2018, 7:29 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, May 6, 2:39 AM

	Unknown Object (File)
	Thu, Apr 30, 5:14 PM

	Unknown Object (File)
	Tue, Apr 28, 5:20 PM

	Unknown Object (File)
	Tue, Apr 28, 7:32 AM

	Unknown Object (File)
	Mon, Apr 20, 4:13 PM

	Unknown Object (File)
	Apr 15 2026, 1:53 PM

	Unknown Object (File)
	Apr 15 2026, 8:22 AM

	Unknown Object (File)
	Apr 11 2026, 2:54 AM

View All 89 Files

Subscribers

imp

Details

Reviewers

allanjude

Commits

rS338540: libsa: validate tftp_makereq() after we did reset the read

Summary

The name check referred in the comment is not the only possible error source,
we need to validate the result.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tsoome created this revision.Sep 8 2018, 7:29 PM

Herald added a subscriber: imp. · View Herald TranscriptSep 8 2018, 7:29 PM

Harbormaster completed remote builds in B19465: Diff 47819.Sep 8 2018, 7:29 PM

allanjude accepted this revision.Sep 8 2018, 7:31 PM

This revision is now accepted and ready to land.Sep 8 2018, 7:31 PM

Closed by commit rS338540: libsa: validate tftp_makereq() after we did reset the read (authored by tsoome). · Explain WhySep 9 2018, 6:30 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

stand/

libsa/

tftp.c

17 lines

Diff 47828

View Options

libsa: validate tftp_makereq() after we did reset the readClosedPublicActions