Bug 1917536 Comment 16 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

### Security Approval Request
* **How easily could an exploit be constructed based on the patch?**: 
* **Do comments in the patch, the check-in comment, or tests included in the patch paint a bulls-eye on the security problem?**: Yes
* **Which branches (beta, release, and/or ESR) are affected by this flaw, and do the release status flags reflect this affected/unaffected state correctly?**: All Branches
* **If not all supported branches, which bug introduced the flaw?**: None
* **Do you have backports for the affected branches?**: No
* **If not, how different, hard to create, and risky will they be?**: Patches are anticipated to apply with minimal effort to older branches.
* **How likely is this patch to cause regressions; how much testing does it need?**: Low chance of regression, but high impact if a regression occurs.

Given this relates to the updater and we could be in a better place with respect to testing, this should receive QA attention before shipping.
* **Is the patch ready to land after security approval is given?**: Yes
* **Is Android affected?**: No
### Security Approval Request
* **How easily could an exploit be constructed based on the patch?**: Given the small size of the patch, someone familiar with the hazard could likely reason about it quickly.
* **Do comments in the patch, the check-in comment, or tests included in the patch paint a bulls-eye on the security problem?**: Yes
* **Which branches (beta, release, and/or ESR) are affected by this flaw, and do the release status flags reflect this affected/unaffected state correctly?**: All Branches
* **If not all supported branches, which bug introduced the flaw?**: None
* **Do you have backports for the affected branches?**: No
* **If not, how different, hard to create, and risky will they be?**: Patches are anticipated to apply with minimal effort to older branches.
* **How likely is this patch to cause regressions; how much testing does it need?**: Low chance of regression, but high impact if a regression occurs.

Given this relates to the updater and we could be in a better place with respect to testing, this should receive QA attention before shipping.
* **Is the patch ready to land after security approval is given?**: Yes
* **Is Android affected?**: No
### Security Approval Request
* **How easily could an exploit be constructed based on the patch?**: Given the small size of the patch, someone familiar with the hazard could likely reason about it quickly.
* **Do comments in the patch, the check-in comment, or tests included in the patch paint a bulls-eye on the security problem?**: The issue is not directly called out, but could use an audit for being too obvious.
* **Which branches (beta, release, and/or ESR) are affected by this flaw, and do the release status flags reflect this affected/unaffected state correctly?**: All Branches
* **If not all supported branches, which bug introduced the flaw?**: None
* **Do you have backports for the affected branches?**: No
* **If not, how different, hard to create, and risky will they be?**: Patches are anticipated to apply with minimal effort to older branches.
* **How likely is this patch to cause regressions; how much testing does it need?**: Low chance of regression, but high impact if a regression occurs.

Given this relates to the updater and we could be in a better place with respect to testing, this should receive QA attention before shipping.
* **Is the patch ready to land after security approval is given?**: Yes
* **Is Android affected?**: No
### Security Approval Request
* **How easily could an exploit be constructed based on the patch?**: Given the small size of the patch, someone familiar with the hazard could likely reason about it quickly.
* **Do comments in the patch, the check-in comment, or tests included in the patch paint a bulls-eye on the security problem?**: The issue is not directly called out, but could use an audit for being too obvious. Messaging could probably be tweaked to make this look more like a cleanup refactor.
* **Which branches (beta, release, and/or ESR) are affected by this flaw, and do the release status flags reflect this affected/unaffected state correctly?**: All Branches
* **If not all supported branches, which bug introduced the flaw?**: None
* **Do you have backports for the affected branches?**: No
* **If not, how different, hard to create, and risky will they be?**: Patches are anticipated to apply with minimal effort to older branches.
* **How likely is this patch to cause regressions; how much testing does it need?**: Low chance of regression, but high impact if a regression occurs.

Given this relates to the updater and we could be in a better place with respect to testing, this should receive QA attention before shipping.
* **Is the patch ready to land after security approval is given?**: Yes
* **Is Android affected?**: No
### Security Approval Request
* **How easily could an exploit be constructed based on the patch?**: Given the small size of the patch, someone familiar with the hazard could likely reason about it quickly.
* **Do comments in the patch, the check-in comment, or tests included in the patch paint a bulls-eye on the security problem?**: The issue is not directly called out, but could use an audit for being too obvious. Messaging could probably be tweaked to make this look more like a cleanup refactor.
* **Which branches (beta, release, and/or ESR) are affected by this flaw, and do the release status flags reflect this affected/unaffected state correctly?**: All Branches
* **If not all supported branches, which bug introduced the flaw?**: None
* **Do you have backports for the affected branches?**: No
* **If not, how different, hard to create, and risky will they be?**: Patches are anticipated to apply with minimal effort to older branches. Comments on risk of regression apply to risk of back porting.
* **How likely is this patch to cause regressions; how much testing does it need?**: Low chance of regression, but high impact if a regression occurs.

Given this relates to the updater and we could be in a better place with respect to testing, this should receive QA attention before shipping.
* **Is the patch ready to land after security approval is given?**: Yes
* **Is Android affected?**: No

Back to Bug 1917536 Comment 16