Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: java parser failing to match vendor on product without '-' #2960

Closed
bcieszko opened this issue Apr 28, 2023 · 0 comments · Fixed by #2961
Closed

fix: java parser failing to match vendor on product without '-' #2960

bcieszko opened this issue Apr 28, 2023 · 0 comments · Fixed by #2961

Comments

@bcieszko
Copy link
Contributor

bcieszko commented Apr 28, 2023

during rpm parsing development #2916 I've found that the find_vendor would return None for product without "-"

def find_vendor(self, product, version):

This is because the conversion and return of info is only done inside the if for "-"

bcieszko added a commit to bcieszko/cve-bin-tool that referenced this issue Apr 28, 2023
if the product did not have - in name and the vendor was found
it would return None even though the match was found

* fixes intel#2960

Signed-off-by: Bartlomiej Cieszkowski <[email protected]>
bcieszko added a commit to bcieszko/cve-bin-tool that referenced this issue May 1, 2023
if the product did not have - in name and the vendor was found
it would return None even though the match was found

* fixes intel#2960

Signed-off-by: Bartlomiej Cieszkowski <[email protected]>
bcieszko added a commit to bcieszko/cve-bin-tool that referenced this issue May 18, 2023
if the product did not have - in name and the vendor was found
it would return None even though the match was found

* fixes intel#2960

Signed-off-by: Bartlomiej Cieszkowski <[email protected]>
terriko pushed a commit that referenced this issue May 22, 2023
if the product did not have - in name and the vendor was found
it would return None even though the match was found

* fixes #2960

Signed-off-by: Bartlomiej Cieszkowski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant