Page MenuHomeFreeBSD
Differential D59602

pciconf: Add -C option to filter by class name
Needs ReviewPublic
Actions

Authored by code_fedang.net on Fri, Sep 11, 10:16 PM.
Tags
None
Referenced Files
F172589286: D59602.id186494.diff
Sat, Sep 19, 12:24 PM
Unknown Object (File)
Sat, Sep 19, 3:21 AM
Unknown Object (File)
Sat, Sep 19, 1:00 AM
Unknown Object (File)
Sat, Sep 19, 12:29 AM
Unknown Object (File)
Fri, Sep 18, 8:51 PM
Unknown Object (File)
Fri, Sep 18, 3:40 AM
Unknown Object (File)
Thu, Sep 17, 6:52 PM
Unknown Object (File)
Thu, Sep 17, 10:40 AM
View All 27 Files
Subscribers
imp
ziaee

Details

Reviewers
bnovkov

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 76785
Build 73668: arc lint + arc unit

Event Timeline

code_fedang.net created this revision.Fri, Sep 11, 10:16 PM
Herald added subscribers: ziaee, imp. · View Herald TranscriptFri, Sep 11, 10:16 PM
code_fedang.net requested review of this revision.Fri, Sep 11, 10:16 PM
Harbormaster completed remote builds in B76785: Diff 186493.Fri, Sep 11, 10:16 PM
code_fedang.net updated this revision to Diff 186494.Fri, Sep 11, 10:27 PM

Use strcasecmp instead of strcmp to check class name.

Harbormaster completed remote builds in B76786: Diff 186494.Fri, Sep 11, 10:27 PM
code_fedang.net added a comment.Fri, Sep 11, 10:29 PM

Addresses Bug 289923

bnovkov added inline comments.Mon, Sep 14, 4:44 PM
usr.sbin/pciconf/pciconf.8
26

Please bump the date.

usr.sbin/pciconf/pciconf.c
1094

This function should ideally return a bool.

1097
1100

Please turn this into an explicit == 0 check

1101–1104
code_fedang.net updated this revision to Diff 186782.Tue, Sep 15, 3:46 PM

Address comment

Harbormaster completed remote builds in B76936: Diff 186782.Tue, Sep 15, 3:46 PM
code_fedang.net marked 5 inline comments as done.Tue, Sep 15, 4:18 PM
bnovkov added inline comments.Fri, Sep 18, 9:24 AM
usr.sbin/pciconf/pciconf.c
335

Passing -l without a class string will crash the program since there's no class != NULL check here or in validate_class.

I suggest adding a NULL check in validate_class.

code_fedang.net updated this revision to Diff 187182.Fri, Sep 18, 11:20 PM

Fix null check

Harbormaster completed remote builds in B77072: Diff 187182.Fri, Sep 18, 11:20 PM
code_fedang.net marked an inline comment as done.Fri, Sep 18, 11:21 PM
code_fedang.net added inline comments.
usr.sbin/pciconf/pciconf.c
335

oops, that slipped through

Revision Contents
Changeset List

PathSize
usr.sbin/
pciconf/
8 lines
39 lines

Diff 186493

View Options

usr.sbin/pciconf/pciconf.8

Loading...
View Options

usr.sbin/pciconf/pciconf.c

Loading...