代码分析 C26408 — 替换 InitInstance 中的 m_pszHelpFilePath 变量

Question

在我的应用程序的InitInstance 函数中，我有以下代码来重写 CHM 帮助文档的位置：

CString strHelp = GetProgramPath();
strHelp += _T("MeetSchedAssist.CHM");
free((void*)m_pszHelpFilePath);
m_pszHelpFilePath = _tcsdup(strHelp);

一切正常，但它给了我一个代码分析警告：

C26408 避免使用malloc() 和free()，更喜欢nothrow 版本的new 和delete (r.10)。

---

当您查看m_pszHelpFilePath 的官方文档时，它确实声明：

如果你给m_pszHelpFilePath赋值，它必须在堆上动态分配。 CWinApp 析构函数使用此指针调用 free( )。许多人想使用_tcsdup( ) 运行时库函数来进行分配。此外，在分配新值之前释放与当前指针关联的内存。

是否可以重写此代码以避免代码分析警告，或者我必须添加__pragma？

Answer 1

您可以（应该？）使用智能指针来包装您重新分配的 m_pszHelpFilePath 缓冲区。不过，这虽然不是小事，但也可以不费吹灰之力地完成。

首先，在您的派生应用程序类中声明一个适当的std::unique_ptr 成员：

class MyApp : public CWinApp // Presumably
{
// Add this member...
public:
    std::unique_ptr<TCHAR[]> spHelpPath;
// ...
};

然后，您需要修改构建和分配帮助路径的代码，如下所示（我已将您的 C 风格转换更改为可以说更好的 C++ 转换）：

// First three (almost) lines as before ...
CString strHelp = GetProgramPath();
strHelp += _T("MeetSchedAssist.CHM");
free(const_cast<TCHAR *>(m_pszHelpFilePath));

// Next, allocate the shared pointer data and copy the string...
size_t strSize = static_cast<size_t>(strHelp.GetLength() + 1); 
spHelpPath std::make_unique<TCHAR[]>(strSize);
_tcscpy_s(spHelpPath.get(), strHelp.GetString()); // Use the "_s" 'safe' version!

// Now, we can use the embedded raw pointer for m_pszHelpFilePath ...
m_pszHelpFilePath = spHelpPath.get();

到目前为止，一切都很好。当您的应用程序对象被销毁时，智能指针中分配的数据将被自动释放，代码分析警告应该会消失。但是，我们需要进行最后一项修改，以防止 MFC 框架试图释放我们分配的 m_pszHelpFilePath 指针。这可以通过在ExitInstance 的MyApp 类覆盖中将其设置为nullptr 来完成：

int MyApp::ExitInstance()
{
    // <your other exit-time code>
    m_pszHelpFilePath = nullptr;
    return CWinApp::ExitInstance(); // Call base class
}

但是，这可能看起来无所事事，正如其他人所说，您可能有理由简单地取消警告。

Answer 2

从技术上讲，您可以利用 new / delete 在 Visual C++ 中默认映射到通常的 malloc/free 这一事实，然后继续替换。便携性不会受到太大影响，因为 MFC 无论如何都不是便携的。当然你可以使用unique_ptr<TCHAR[]>而不是直接new/delete，像这样：

CString strHelp = GetProgramPath();
strHelp += _T("MeetSchedAssist.CHM");
std::unique_ptr<TCHAR[]> str_old(m_pszHelpFilePath);
auto str_new = std::make_unique<TCHAR[]>(strHelp.GetLength() + 1);
_tcscpy_s(str_new.get(), strHelp.GetLength() + 1, strHelp.GetString());
m_pszHelpFilePath = str_new.release();
str_old.reset();

为了保证替换 new 操作符的稳健性，并且为了至少意外原则，您应该保留 free / strdup。

如果您替换了多个 CWinApp 字符串，建议为它们编写一个函数，以便在一个位置出现 free / strdup 并带有隐藏的警告。