search

fedora-static-analysis / firehose

Interchange format for results for static analysis tools
63 stars 18 forks source link

Add parser for flawfinder #31

Closed davidCarlos closed 7 years ago

davidCarlos commented 7 years ago

I'm still working on this pull request (missing tests). soon i will remove the 'WIP' tag.

davidmalcolm commented 7 years ago

As noted in #33 we don't have a good design yet for how to handle multiple CWEs in one issue.

For now, maybe have the parser capture the first CWE (if any), and note the limitation as a bug, marking it as a dependency of #33.

Thanks.

davidCarlos commented 7 years ago

Hey @davidmalcolm , i have updated this PR. I added some tests, and have included flawfinder in the documentation. Now the parser capture the first CWE, as we had discussed. I created the issue #35, to map the multiple cwes problem.

davidCarlos commented 7 years ago

Hey @davidmalcolm , i updated this PR with the last revision.

davidCarlos commented 7 years ago

@davidmalcolm thanks for the great revision, i had updated the PR.

davidCarlos commented 7 years ago

@davidmalcolm I had updated the PR

davidCarlos commented 7 years ago

@davidmalcolm I had updated the PR