Added the Bluez checker #232
Conversation
FReeshabh
commented
Oct 18, 2019
FReeshabh
commented
modified: checkers/__init__.py new file: checkers/bluez.py
|
Thank you! This is looking pretty good, but I currently get the following error when I run it: "ValueError: Docstring of 'bluez' checker does not define a vendor package pair" It looks like you're missing the VPkg definition in the docstring. It should look like this: VPkg: bluez, bluez There's some instructions about it here: |
cve_bin_tool/checkers/bluez.py
Outdated
|
|
||
| # def get_version(lines, filename): | ||
| def get_version(lines, filename): | ||
| """Bluetoothctl will work for Version 5.0+""" |
There was a problem hiding this comment.
You'll need to include the
Vpkg: bluez, bluez
string in the bottom of this docstring so that the checker can do the magic of associating it with the database entries.
There was a problem hiding this comment.
I added the Vpkg string at the bottom of the doc string. I might have made an unneeded comment, so you might need to squash the commits. Thanks for the guidance!
modified: bluez.py
modified: bluez.py
|
Sorry about the delay in getting this merged; I've been travelling. I'm just trying to get my remote setup working properly to run the tests since travis isn't behaving, but I'm expecting to get this merged today. |
|
Got it working; thanks! I'll file a bug so we don't forget to add a test for bluez, too. |
2 participants
