msi: InstallPackage check for UI level must not disregard flags
truiken at gmail.com
Tue Feb 6 01:54:24 CST 2007
On 2/5/07, Misha Koshelev <mk144210 at bcm.tmc.edu> wrote:
> This is an update, a little less hacky thanks to advice from Dmitry
> Timoshkov. This fixes bug #6992. The installer for Vector NTI 10 was
> getting to the
> point where it was trying to launch the .msi files that do the actual
> install, but was quitting with the message "This module is not designed
> for direct execution." Install worked fine with native MSI. After a
> while of playing with it and disassembling the MSI file with dark.exe
> tool, I noticed that the .msi files ran in execute mode but not UI mode.
> Finally, I checked the UI level that wine msi was reading, and it was
> 102 which is INSTALLUILEVEL_NONE | INSTALLUILEVEL_SOURCESONLY. However,
> MSI_InstallPackage was only checking if the uilevel >=
> INSTALLUILEVEL_REDUCED, which would have been true in this case as
> INSTALLUILEVEL_SOURCESONLY > INSTALLUILEVEL_REDUCED but the actual ui
> level is INSTALLUILEVEL_NONE which is < INSTALLUILEVEL_REDUCED. This
> patch fixes this check, and now the installer finished successfully
> (although it seems upon a quick check that there are more install bugs
> for this program in wine, as not all files that should be there seem to
> be there after this install).
> * msi: InstallPackage check for UI level must not disregard
> INSTALLUILEVEL flags.
> + if ( (msi_get_property_int(package, szUILevel, 0) & 0xf) >= INSTALLUILEVEL_REDUCED )
Magic numbers are bad. What is 0xF?
More information about the wine-devel