andymeneely / chromium-history

Scripts and data related Chromium's history
11 stars 4 forks source link

Manually Inspect CVEs for Code Reviews #89

Closed bspates closed 10 years ago

bspates commented 10 years ago

Hey @andymeneely, @cck9672, @smt9020,

So in our shared folder in the Vulnerabilities spreadsheet I added a sheet called Manual Inspection. It should be the first one. In there is a list of CVEs with either no information at all, or a list of google code issue ids. The ones with google code ids might be easiest to tackle first. If people wanna claim a range in the table of entries with g ids and they can start inspecting. Go to https://code.google.com/p/chromium/issues/list, and enter the provided issue in the search bar to go to the corresponding issue page. That would be the best starting place. From there look for any links to the rietveld system, or the repo tree, as links to code reviews have been found in commits there.

andymeneely commented 10 years ago

Ok, I will buy lunch for anyone who does more of these than me. I've already done two.

Follow the link to the CVE and try to find the Rietveld id. It won't be obvious because Brian's scraper finds those.

If you find the commit and it does not mention a the code review, you can consider it "none", which counts as done.

bspates commented 10 years ago

Woops, I think i can remove some more of these automatically, I'll update it it later today, hold off on doing more till then.

bspates commented 10 years ago

Forgot to update this, but the list is ready.

bspates commented 10 years ago

Done