devmatch(8): Respect mask field when matching strings.
ClosedPublic
Actions

Authored by wulf on Oct 17 2020, 12:19 AM.

Details

Reviewers

Commits

rS367676: MFC r367237:
rS367237: devmatch(8): Respect mask field when matching strings of Z type.

Summary

Currently, only integer fields of descriptor string can be masked out for matching. This patch adds support for masking of 'Z' type field.

While here, add debug output for this action.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

wulf requested review of this revision.Oct 17 2020, 12:19 AM

wulf created this revision.

imp added inline comments.Oct 17 2020, 12:24 AM

sbin/devmatch/devmatch.c
352	When would bit by < 0?

wulf added inline comments.Oct 17 2020, 12:32 AM

sbin/devmatch/devmatch.c
352	It starts iterating from -1. May be the first field is dedicated to mask itself?

imp added inline comments.Oct 17 2020, 12:36 AM

sbin/devmatch/devmatch.c
352	Yes.... I don't think the check is needed, but I'm not sure

wulf added inline comments.Oct 17 2020, 12:39 AM

sbin/devmatch/devmatch.c
352	I borrowed the check from integer case to not introduce new bugs.

imp accepted this revision.Oct 17 2020, 12:53 AM

imp added inline comments.

sbin/devmatch/devmatch.c
352	I borrowed the check from integer case to not introduce new bugs. Fair enough. Keep it...

This revision is now accepted and ready to land.Oct 17 2020, 12:53 AM

Closed by commit rS367237: devmatch(8): Respect mask field when matching strings of Z type. (authored by wulf). · Explain WhyOct 31 2020, 10:04 PM

This revision was automatically updated to reflect the committed changes.

wulf added a commit: rS367237: devmatch(8): Respect mask field when matching strings of Z type..

wulf added a commit: rS367676: MFC r367237:.Nov 14 2020, 11:52 AM

Revision Contents
Changeset List

Path

Size

sbin/

devmatch/

devmatch.c

5 lines

Diff 78338

View Options

devmatch(8): Respect mask field when matching strings.ClosedPublicActions