pfctl(8): Fix two wrong conditions.
ClosedPublic
Actions

Authored by pfg on Jan 27 2018, 10:34 PM.

Tags

None

Referenced Files

	F174847386: D14083.diff
	Tue, Oct 6, 11:58 AM

	Unknown Object (File)
	Mon, Sep 21, 10:03 PM

	Unknown Object (File)
	Fri, Sep 18, 1:41 AM

	Unknown Object (File)
	Fri, Sep 18, 1:36 AM

	Unknown Object (File)
	Fri, Sep 18, 1:35 AM

	Unknown Object (File)
	Fri, Sep 18, 1:26 AM

	Unknown Object (File)
	Sat, Sep 12, 4:32 PM

	Unknown Object (File)
	Fri, Sep 11, 2:14 PM

View All Files

Subscribers

imp

Details

Reviewers

kp
glebius

Commits

rS328497: pfctl(8): Fix two wrong conditions.

Summary

Caught by gcc80's -Wtautological-compare option.

Obtained from: DragonFlyBSD (git e3cdbf6c)

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

pfg created this revision.Jan 27 2018, 10:34 PM

Herald added a subscriber: imp. · View Herald TranscriptJan 27 2018, 10:34 PM

Good catch.

This revision is now accepted and ready to land.Jan 27 2018, 10:46 PM

Closed by commit rS328497: pfctl(8): Fix two wrong conditions. (authored by pfg). · Explain WhyJan 27 2018, 10:57 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sbin/

pfctl/

pfctl_optimize.c

4 lines

Diff 38551

View Options

head/sbin/pfctl/pfctl_optimize.c

pfctl(8): Fix two wrong conditions.ClosedPublicActions