Page MenuHomeFreeBSD
Differential D6651

keyserv(1): Wrong comparison in load_des()
ClosedPublic
Actions

Authored by pfg on May 30 2016, 7:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 8 2024, 5:31 AM
Unknown Object (File)
Mar 8 2024, 5:31 AM
Unknown Object (File)
Mar 8 2024, 5:17 AM
Unknown Object (File)
Mar 1 2024, 8:25 AM
Unknown Object (File)
Jan 10 2024, 9:15 AM
Unknown Object (File)
Dec 22 2023, 11:21 AM
Unknown Object (File)
Dec 20 2023, 12:54 AM
Unknown Object (File)
Nov 19 2023, 2:18 PM
View All 31 Files
Subscribers
imp

Details

Reviewers
truckman
ngie
Summary

According to Coverity (CID 1008422), as dlpath is a character array
so the omparson to NULL makes no sense. It appears it is always
assigned to something in the previous "if" so perhaps we should
get rid of it completely.

While here remove redundant bracket as it appears to be the style
exception wrt the rest of the code.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4039
Build 4082: arc lint + arc unit

Event Timeline

pfg updated this revision to Diff 17122.May 30 2016, 7:55 PM
pfg retitled this revision from to keyserv(1): Wrong comparison in load_des().
pfg updated this object.
pfg edited the test plan for this revision. (Show Details)
pfg added reviewers: truckman, ngie.
Herald added a subscriber: imp. · View Herald TranscriptMay 30 2016, 7:55 PM
ngie accepted this revision.May 30 2016, 7:57 PM
ngie edited edge metadata.
This revision is now accepted and ready to land.May 30 2016, 7:57 PM
truckman accepted this revision.Jun 1 2016, 6:21 PM
truckman edited edge metadata.
pfg closed this revision.Jun 1 2016, 6:50 PM

Committed as r301005.
Thanks to the reviewers!

Revision Contents
Changeset List

PathSize
usr.sbin/
keyserv/
9 lines

Diff 17122

View Options

usr.sbin/keyserv/crypt_server.c

Loading...