Bureaucrats, canmove, Confirmed users
642
edits
No edit summary |
mNo edit summary |
||
| Line 1: | Line 1: | ||
For security bugs with no sec- severity rating assume the worst and follow the rules for sec-critical. If you have experience fixing security bugs you could also take a crack at rating it yourself following the [[Security_Severity_Ratings]] | |||
===sec-low, sec-moderate, sec-other or sec-want=== | ===sec-low, sec-moderate, sec-other or sec-want=== | ||
Core-security bug fixes should just be landed by a developer without any | Core-security bug fixes should just be landed by a developer without any | ||
| Line 10: | Line 12: | ||
If it meets the above criteria, check that patch in. | If it meets the above criteria, check that patch in. | ||
===sec-high or sec-critical=== | ===sec-high or sec-critical (or no rating)=== | ||
Otherwise, if the bug has a patch *and* is sec-high or sec-critical, the developer should set the sec-approval flag to '?' on the patch when it is ready to be checked into mozilla-central (or elsewhere if it is branch only). | Otherwise, if the bug has a patch *and* is sec-high or sec-critical, the developer should set the sec-approval flag to '?' on the patch when it is ready to be checked into mozilla-central (or elsewhere if it is branch only). | ||
If you have a patch and the bug is a hidden core-security bug with no rating then either: | If you have a patch and the bug is a hidden core-security bug with no rating then either: | ||
#request sec-approval (to be safe) and wait for a rating, <br>or | #request sec-approval (to be safe) and wait for a rating, <br>or | ||
# rate it and then proceed according to whether the bug is low/moderate or high/critical as above. | # rate it following the and then proceed according to whether the bug is low/moderate or high/critical as above. | ||
If developers are unsure about a bug and it has a patch ready, just mark | If developers are unsure about a bug and it has a patch ready, just mark | ||