无法在 UPGRADINGPRODUCTCODE、WIX_UPGRADE_DETECTED 中获取 MSI 属性

Question

我想在升级过程中跳过一些自定义操作uninstallDriver，为此我尝试检索属性 WIX_UPGRADE_DETECTED 和 UPGRADINGPRODUCTCODE，但两者都没有设置。

示例代码：

UninstallDriver(MSIHANDLE hInstall)  
{
   char szBuff[1024]; DWORD dwValue = 0;

   OutputDebugStringA("UninstallDriver");
   MsiGetPropertyA(hInstall, "UPGRADINGPRODUCTCODE", szBuff, &dwValue);
   OutputDebugStringA("UPGRADINGPRODUCTCODE");OutputDebugStringA(szBuff);

   MsiGetPropertyA(hInstall, "WIX_UPGRADE_DETECTED", szBuff, &dwValue);
   OutputDebugStringA("WIX_UPGRADE_DETECTED");OutputDebugStringA(szBuff);

   result = UninstallDriver();

   return result;
}

而我的 customAction 是，

<Custom Action='UninstallMYDriverAction'
 After='InstallInitialize'>
        REMOVE~="ALL" OR REINSTALL</Custom>

Answer 1

自定义操作条件：您应该能够直接设置自定义操作而不通过自定义操作？微星 条件因难以做到恰到好处而臭名昭著，因为 有这么多安装模式要处理：install, repair、self-repair、modify 和 minor upgrade patching， uninstall、major upgrade initiated uninstalls等……有 还有几个。

在您的情况下，添加到现有条件可能就足够了：

AND (NOT UPGRADINGPRODUCTCODE)

如果检测到升级，这将使条件无效。

我实际上会尝试：(REMOVE~="ALL") AND (NOT UPGRADINGPRODUCTCODE) (as seen in this answer)。

---

重要提示：您可以通过此处描述的方法非常有效地调试 MSI 条件： How to execute conditional custom action on install and modify only? （见底部）。您使用 VBScript 和一些 MSI API 调用 确定哪些条件为真，然后您可以更改 排序以检查不同时间的条件。他们可能 改变。

注意：这只是一个自怨自艾的可怜人的 MSI 调试器，但它可以工作。

---

属性字符串大小：我不是 C++ 专家，但这里来自 the MSI SDK：您需要先用空字符串调用 MsiGetProperty，然后再调用为 null 终止 添加 1。像往常一样，C++ 的“线路噪音”很高。很多管道和手续几乎没有效果 - 但让我们处理它:-)：

以下 C++ 示例来自 the MSI SDK：

UINT __stdcall MyCustomAction(MSIHANDLE hInstall)
{
    TCHAR* szValueBuf = NULL;
    DWORD cchValueBuf = 0;
    UINT uiStat =  MsiGetProperty(hInstall, TEXT("MyProperty"), TEXT(""), &cchValueBuf);
    //cchValueBuf now contains the size of the property's string, without null termination
    if (ERROR_MORE_DATA == uiStat)
    {
        ++cchValueBuf; // add 1 for null termination
        szValueBuf = new TCHAR[cchValueBuf];
        if (szValueBuf)
        {
            uiStat = MsiGetProperty(hInstall, TEXT("MyProperty"), szValueBuf, &cchValueBuf);
        }
    }
    if (ERROR_SUCCESS != uiStat)
    {
        if (szValueBuf != NULL) 
           delete[] szValueBuf;
        return ERROR_INSTALL_FAILURE;
    }   

// custom action uses MyProperty
// ...

delete[] szValueBuf;

return ERROR_SUCCESS;

}

VBScript：出于测试目的，可以轻松使用 VBScript（因此您可以确定是否有编码问题）：

MsgBox Session.Property("PROPERTYNAME")

---

一些进一步的链接（仅供参考和方便检索，我认为您不需要）：

• MSI Conditions
• How to execute custom action only in install (not uninstall)
• How to add a WiX custom action that happens only on uninstall (via MSI)?
• Wix Tools update uses old custom actions
• Wix custom uninstallation action - how to run before msi removing files

Answer 2

忽略调试字符串，更容易看出缓冲区处理不正确。我建议还输出来自MsiGetPropertyA() 的返回值和dwValue 中的值来确认，但这是我认为正在发生的事情（cmets 指的是dwValue）：

char szBuff[1024]; DWORD dwValue = 0;
MsiGetPropertyA(hInstall, "UPGRADINGPRODUCTCODE", szBuff, &dwValue); // passes 0, updated to ?x?
MsiGetPropertyA(hInstall, "WIX_UPGRADE_DETECTED", szBuff, &dwValue); // passes ?x?, updated to ?y?

当请求缓冲区长度为零的UPGRADINGPRODUCTCODE 属性时，提取将永远不会成功，因为它必须始终至少接受一个空字符。因此，这将返回 ERROR_MORE_DATA 并将 dwValue 设置为不包括空字符 (?x?) 的长度。

然后它将请求WIX_UPGRADE_DETECTED 的值，声称缓冲区长度为（?x?）。如果新长度 (?y?) 小于旧长度 (?x?)，您将在缓冲区中获取其内容；否则它也只会查询这个新属性的长度。

由于WIX_UPGRADE_DETECTED 包含一个或多个GUID 的列表，而UPGRADINGPRODUCTCODE 仅包含一个，并且此代码从不增加dwValue 以解释空值，因此只有在后者为?y? 时才可能成功为 0（空），?x? 为非空。但请注意，第二次调用传递了一个未经验证的值作为缓冲区的长度，这种模式是等待发生的缓冲区溢出。

所以修复你的缓冲区处理。我喜欢使用的模式（如下）类似于 Stein 描述的模式，但如果我知道缓冲区的良好默认大小，则避免第二次调用。就您而言，听起来您对 1024 元素缓冲区很满意，但请考虑您是否需要处理超过 1024 / len(GUID) 相关的升级代码。

（我猜你没问题。但至少要考虑清楚。即使 GUID 是 ASCII 所以内容无关紧要，请这些天请构建 UNICODE...）

WCHAR szBuf[1024];
DWORD cchBuf = 1024; // or _countof(szBuf);
DWORD dwErr = MsiGetPropertyW(hInstall, L"UPGRADINGPRODUCTCODE", szBuf, &cchBuf);
if (dwErr != ERROR_MORE_DATA) {
    // exercise: increment cchBuf for null, adjust buffer, call MsiGetPropertyW again
}
if (dwErr != ERROR_SUCCESS) {
    // per https://docs.microsoft.com/en-us/windows/desktop/msi/custom-action-return-values
    return ERROR_INSTALL_FAILURE;
}

// reset buffer length for next call, in case second property is longer than first
cchBuf = 1024;
dwErr = MsiGetPropertyW(hInstall, L"WIX_UPGRADE_DETECTED", szBuf, &cchBuf);
// : : :