Modified crash display query to properly group based on verification status

stephenfewer / grinder

Grinder is a system to automate the fuzzing of web browsers and the management of a large number of crashes.

BSD 3-Clause "New" or "Revised" License

414 stars 131 forks source link

Modified crash display query to properly group based on verification status #26

Closed pyoor closed 10 years ago

pyoor commented 10 years ago

Modified 'verified' property so that it lists verification status based on importance incrementally.

$verified_unknown = 0; $verified_uninteresting = 1; $verified_interesting = 2; $verified_exploitable = 3;

Then modified the crash display query to use groupwise max in order to only display top level (highest) value assigned to 'verified' when displaying unique crashes only. This ensures that when crashes are deleted from the database that they will be displayed properly in the UI.

stephenfewer commented 10 years ago

Hi,

Thanks for the patch!! :) I added in a migration in user.php:update_db() to update an old version of the DB to account for the change in the interesting/uninteresting values and I also updated the node side (webstats.rb), which I think covers everything.

Thanks again!