blacklist: Fix use of uninitialized variable
AbandonedPublic
Actions

Authored by jlduran on Tue, Mar 25, 10:08 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 3, 2:31 AM

	Unknown Object (File)
	Thu, Apr 3, 2:16 AM

	Unknown Object (File)
	Wed, Apr 2, 10:14 PM

	Unknown Object (File)
	Tue, Apr 1, 7:58 AM

	Unknown Object (File)
	Tue, Apr 1, 7:37 AM

	Unknown Object (File)
	Mon, Mar 31, 7:16 PM

	Unknown Object (File)
	Mon, Mar 31, 3:12 PM

	Unknown Object (File)
	Wed, Mar 26, 8:00 AM

Subscribers

imp

Details

Reviewers

emaste
dch

Summary

The variables c.c_nfail and dbi.last can be used uninitialized when
getremoteaddress() fails while debugging.

This issue has been reported upstream, however this fix was not
proposed, because there are other ways to fix it. Once upstream
implements a fix, it can be safely reverted, if needed.

Coverity ID: 1473591
Coverity ID: 1473593

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 63124
Build 60008: arc lint + arc unit

Event Timeline

jlduran created this revision.Tue, Mar 25, 10:08 PM

Herald added a subscriber: imp. · View Herald TranscriptTue, Mar 25, 10:08 PM

jlduran requested review of this revision.Tue, Mar 25, 10:08 PM

Harbormaster completed remote builds in B63124: Diff 152676.Tue, Mar 25, 10:08 PM

jlduran added a parent revision: D49507: blacklist: ftdp: Add a permission denied probe.Tue, Mar 25, 10:08 PM

jlduran added a child revision: D49509: ftpd: Add an extra blacklist probe.

A fix has already been committed upstream.

Revision Contents
Changeset List

Path

Size

contrib/

blocklist/

bin/

blacklistd.c

7 lines

Diff 152676

View Options

contrib/blocklist/bin/blacklistd.c

blacklist: Fix use of uninitialized variableAbandonedPublicActions