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

Possible OWASP scoring issue #1

Closed
nojeffrey opened this issue Feb 26, 2014 · 7 comments
Closed

Possible OWASP scoring issue #1

nojeffrey opened this issue Feb 26, 2014 · 7 comments

Comments

@nojeffrey
Copy link

When submitting a risk and using the OWASP option under Threat Agent Factors/Skill Level, the rating is 1=No technical skills, and 9=Security penetration skills.

But on the OWASP page : https://www.owasp.org/index.php/OWASP_Risk_Rating_Methodology
Under skill level, the values are around the other way, 1=Security penetration skills, and 9=No technical skills.

This is the difference between a risk having a score of 10, and now that I've switched it around 1.1

@nojeffrey
Copy link
Author

Actually I think there is a bug here, my first 3 risks I've added all had a score of 10 using this OWASP scoring system, if i go into and edit a risk, and change one value from this OWASP score, it changes the score dramatically, for eg
screenshot from 2014-02-26 11 49 33
I changed Motive from 7 to 6, and the score went from 10, to 2.8, which does seem accurate now.

If i change it back to 7 it goes up to 2.9.

@jsokol
Copy link
Member

jsokol commented Feb 26, 2014

Thanks Jeffrey.  I will take a look into this and get back with you.  Can you please provide me with what version of SimpleRisk you are using?

Sincerely, 

Josh Sokol
Creator of SimpleRisk
http://www.simplerisk.org

-------- Original message --------
From: jeffrey32 notifications@github.com
Date: 02/25/2014 6:54 PM (GMT-06:00)
To: simplerisk/bundles bundles@noreply.github.com
Subject: Re: [bundles] Possible OWASP scoring issue (#1)

Actually I think there is a bug here, my first 3 risks I've added all had a score of 10 using this OWASP scoring system, if i go into and edit a risk, and change one value from this OWASP score, it changes the score dramatically, for eg

I changed Motive from 7 to 6, and the score went from 10, to 2.8, which does seem accurate now.

If i change it back to 7 it goes up to 2.9.


Reply to this email directly or view it on GitHub.

@nojeffrey
Copy link
Author

Yep the latest: 20140224-001

@jsokol
Copy link
Member

jsokol commented Feb 26, 2014

Awesome. Thanks for the quick response. I will investigate and get
back with you on this.

On 2/25/14, 8:05 PM, jeffrey32 wrote:

Yep the latest: 20140224-001


Reply to this email directly or view it on GitHub
#1 (comment).

@jsokol
Copy link
Member

jsokol commented Feb 27, 2014

OK, I've confirmed that OWASP updated this metric in October 2013:

https://www.owasp.org/index.php?title=OWASP_Risk_Rating_Methodology&diff=167414&oldid=161133

This change will be incorporated into the next release of SimpleRisk, but in the meantime, if you'd like to update the verbiage yourself, just search for "No Technical Skills" in the /includes/display.php page. It only affects the help section and not the rating itself. I'm taking a look at the second issue you reported around the actual scoring now.

@jsokol
Copy link
Member

jsokol commented Feb 27, 2014

There was definitely a bug in the way I was calculating the OWASP score. Part of this is brought on by the fact that the OWASP Risk Scoring Methodology is non-specific about how a final score should be calculated. The way I do this now is by averaging the two impact values together and then the two likelihood values together. Then I multiply both of those values and divide by 10. This is what the Javascript was doing in the scorer when you first score a risk and I should have just done that in the other locations as well. This will be rolled into the next release, but as I just released a new version and don't want to roll a whole new version so quickly after the last for a relatively minor bug, I'm providing the following bundle which you can extract on top of your existing instance to correct both of the issues you reported. No need to run any upgrade.php script for this.

https://simplerisk.it/simplerisk-20140227-001.tgz

Thanks for reporting the bug and I hope this helps!

@jsokol jsokol closed this as completed Feb 27, 2014
@nojeffrey
Copy link
Author

Awesome, thanks for the quick fix!

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

No branches or pull requests

2 participants