1359536 - Incorrect js-ctypes ABI for GetVersionEx in toolkit/mozapps/update/nsUpdateService.js

Status: RESOLVED FIXED

(Toolkit :: Application Update, defect)

Comment 1

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Bug 1359536: Fix incorrect ABI for js-ctypes declaration of GetVersionExW;

Review commit: https://reviewboard.mozilla.org/r/133522/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/133522/

Comment 2

[Molly Howell (she/her) (mostly inactive)]

Comment on attachment 8861555 [details]
Bug 1359536: Fix incorrect ABI for js-ctypes declaration of GetVersionExW;

https://reviewboard.mozilla.org/r/133522/#review136422

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2b9a7420d42f
Fix incorrect ABI for js-ctypes declaration of GetVersionExW; r=mhowell

Comment 4

[(No longer employed by Mozilla) Aaron Klotz]

Comment on attachment 8861555 [details]
Bug 1359536: Fix incorrect ABI for js-ctypes declaration of GetVersionExW;

Approval Request Comment
[Feature/Bug causing the regression]: JS-ctypes code in update service
[User impact if declined]: Potential for crashes and/or data corruption
[Is this code covered by automated tests?]: Yes
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: Trivial patch
[String changes made/needed]: None

Comment 5

[(No longer employed by Mozilla) Aaron Klotz]

Comment on attachment 8861555 [details]
Bug 1359536: Fix incorrect ABI for js-ctypes declaration of GetVersionExW;

[Approval Request Comment]
If this is not a sec:{high,crit} bug, please state case for ESR consideration: This is a stability issue
User impact if declined: Potential for crashing / data corruption
Fix Landed on Version: 55, 54 requested
Risk to taking this patch (and alternatives if risky): None. Trivial patch
String or UUID changes made by this patch: None

See https://wiki.mozilla.org/Release_Management/ESR_Landing_Process for more info.

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2b9a7420d42f

Comment 7

[Gerry Chang [:gchang]]

Comment on attachment 8861555 [details]
Bug 1359536: Fix incorrect ABI for js-ctypes declaration of GetVersionExW;

Fix a potential crash/data corruption. Beta54+. Should be in 54 beta 3.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/0ee6d3ac7a06

Comment 9

[(No longer employed by Mozilla) Aaron Klotz]

Comment on attachment 8861555 [details]
Bug 1359536: Fix incorrect ABI for js-ctypes declaration of GetVersionExW;

Cancelling ESR52 request. I have looked at the libffi code and there is no chance of crashing given the way it is implemented.

Comment 10

[Andrei Vaida [:avaida]]

(In reply to Aaron Klotz [:aklotz] (a11y work receiving priority right now) from comment #4)
> [Is this code covered by automated tests?]: Yes
> [Has the fix been verified in Nightly?]: Yes
> [Needs manual test from QE? If yes, steps to reproduce]: No

Setting qe-verify- based on Aaron's assessment on manual testing needs and the fact that this fix has automated coverage.