1485484 - Improper use of ApplicationID plugin causing unbalanced NSIS stack

Status: VERIFIED FIXED

(Firefox :: Installer, enhancement)

Description

[Matt Howell (he/him) [:mhowell]]

In the function UpdateShortcutAppModelIDs [1] we make 5 calls to ApplicationID::Set where we pass it only two parameters, but it expects a third (or an empty stack otherwise). The result is an extra item being popped off the NSIS stack.

This was originally discussed and patched in bug 425167, but this is a different root cause from the issue that bug was originally filed for.

[1] https://searchfox.org/mozilla-central/rev/3fa761ade83ed0b8ab463acb057c2cf0b104689e/toolkit/mozapps/installer/windows/nsis/common.nsh#6999

Comment 1

[Matt Howell (he/him) [:mhowell]]

Attachment: Bug 1485484 - Always fill in the optional third parameter to ApplicationID::Set. r=agashlin

In theory, the third parameter (dualMode) to the NSIS ApplicationID plugin's
Set function is optional, but the plugin assumes that either it's there or
there's nothing else on the NSIS stack, so it pops three items from the stack
unconditionally. This means that supplying only two parameters results in
one item silently being dropped from the NSIS stack. Fortunately this one
function is the only place where we were doing that, so it only became a
problem if there were an awful lot of shortcuts (around 7) to the same
installation.

Comment 2

[Adam Gashlin (he/him) [:agashlin]]

Comment on attachment 9003264 [details]
Bug 1485484 - Always fill in the optional third parameter to ApplicationID::Set. r=agashlin

Adam Gashlin [:agashlin] has approved the revision.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/202f0986f459
Always fill in the optional third parameter to ApplicationID::Set. r=agashlin

Comment 4

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/202f0986f459

Comment 5

[Mark Hammond [:markh]]

Most recent nightly update no longer has the issue for me \o/ - thanks Matt.