Paths

Table of Contentst

Differential D59602

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

Authored by code_fedang.net on Fri, Sep 11, 10:16 PM.

Tags

None

Referenced Files

	F175295234: D59602.id187352.diff
	Fri, Oct 9, 6:11 PM

	F175225975: D59602.id186493.diff
	Fri, Oct 9, 6:16 AM

	F175223718: D59602.id187182.diff
	Fri, Oct 9, 5:52 AM

	F175222745: D59602.id186494.diff
	Fri, Oct 9, 5:40 AM

	Unknown Object (File)
	Thu, Oct 8, 12:13 PM

	Unknown Object (File)
	Thu, Oct 8, 3:59 AM

	Unknown Object (File)
	Fri, Oct 2, 7:03 PM

	Unknown Object (File)
	Fri, Oct 2, 6:54 PM

View All 78 Files

Subscribers

Details

Reviewers

Commits

rG46d383af0dbd: pciconf: Add -C option to filter by class name

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 76936
Build 73819: 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

Use strcasecmp instead of strcmp to check class name.

Harbormaster completed remote builds in B76786: Diff 186494.Fri, Sep 11, 10:27 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

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`.

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

This revision was not accepted when it landed; it landed in state Needs Review.Mon, Sep 21, 6:39 PM

Closed by commit rG46d383af0dbd: pciconf: Add -C option to filter by class name (authored by code_fedang.net, committed by bnovkov). · Explain Why

This revision was automatically updated to reflect the committed changes.

bnovkov added a commit: rG46d383af0dbd: pciconf: Add -C option to filter by class name.

Revision Contents
Changeset List

Path

Size

usr.sbin/

pciconf/

10 lines

40 lines

Diff 186782

usr.sbin/pciconf/pciconf.8

Loading...

usr.sbin/pciconf/pciconf.c

Loading...