tftp: fix two minor Coverity CIDs
ClosedPublic
Actions

Authored by asomers on Sep 17 2019, 11:18 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Nov 2, 12:28 AM

	Unknown Object (File)
	Fri, Oct 31, 12:09 AM

	Unknown Object (File)
	Thu, Oct 23, 3:28 AM

	Unknown Object (File)
	Wed, Oct 22, 11:23 AM

	Unknown Object (File)
	Thu, Oct 16, 5:36 AM

	Unknown Object (File)
	Thu, Oct 16, 5:36 AM

	Unknown Object (File)
	Thu, Oct 16, 5:36 AM

	Unknown Object (File)
	Wed, Oct 15, 6:24 PM

View All 90 Files

Subscribers

imp

koobs

Details

Reviewers

cem

Commits

rS354163: MFC r353068:
rS353068: tftp: fix two minor Coverity CIDs

Summary

tftp: fix two minor Coverity CIDs

CID 1394842: file descriptor leak in an error path
CID 1007603: single byte array overflow

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

asomers created this revision.Sep 17 2019, 11:18 PM

Herald added a subscriber: imp. · View Herald TranscriptSep 17 2019, 11:18 PM

Harbormaster completed remote builds in B26540: Diff 62242.Sep 17 2019, 11:18 PM

I'm not C savvy, so probably not appropriate for review (but thanks for adding me so I was aware of the change)

koobs removed a reviewer: koobs.Sep 18 2019, 3:43 AM

koobs added a subscriber: koobs.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 3 2019, 8:22 PM

Closed by commit rS353068: tftp: fix two minor Coverity CIDs (authored by asomers). · Explain Why

This revision was automatically updated to reflect the committed changes.

asomers added a commit: rS353068: tftp: fix two minor Coverity CIDs.

asomers added a commit: rS354163: MFC r353068:.Oct 30 2019, 1:41 AM

Revision Contents
Changeset List

Path

Size

head/

usr.bin/

tftp/

main.c

3 lines

Diff 62883

View Options

tftp: fix two minor Coverity CIDsClosedPublicActions