### Beta/Release Uplift Approval Request * **User impact if declined**: Firefox would crash and the exploit might be used for attacking * **Is this code covered by automated tests?**: No * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: No * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: This change is to add a new conditional check to avoid triggering the UAF, it doesn't introduce any new behavior. * **String changes made/needed**: no
Bug 1745874 Comment 13 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Per this [process](https://firefox-source-docs.mozilla.org/bug-mgmt/processes/security-approval.html#process-for-security-bugs-sec-approver-perspective) I would like to request a uplift and land them as close as possible. However, this patch won't work on Release, I would need to update another verison of patch in order to avoid the merging conflict. ### Beta/Release Uplift Approval Request * **User impact if declined**: Firefox would crash and the exploit might be used for attacking * **Is this code covered by automated tests?**: No * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: No * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: This change is to add a new conditional check to avoid triggering the UAF, it doesn't introduce any new behavior. * **String changes made/needed**: no