如何解决C6386警告：向m_NID.szTip写入时缓冲区溢出？

问题根源 #

警告C6386 Buffer overrun while writing to m_NID.szTip的核心原因是：StringCchCopy的第二个参数需要传入目标缓冲区的字符数量（包含终止符），但你代码里用sizeof(m_NID.szTip)获取的是缓冲区的字节数。由于NOTIFYICONDATA::szTip是TCHAR类型数组，在Unicode编译环境下TCHAR占2字节，这就导致传入的大小是实际可用字符数的2倍，代码分析工具判定存在溢出风险。

解决方法 #

方法1：使用_countof宏获取数组元素个数 #

_countof是VC编译器提供的宏，专门用于计算数组的元素数量（即字符数），自动适配ANSI/Unicode编译环境。修改代码如下：

bool CCenterCursorOnScreenDlg::SetTrayIconTipText()
{
    if (StringCchCopy(m_NID.szTip, _countof(m_NID.szTip), m_strTrayIconTip) != S_OK)
        return FALSE;

    m_NID.uFlags |= NIF_TIP;

    return Shell_NotifyIcon(NIM_MODIFY, &m_NID);
}

方法2：手动计算字符数 #

如果不想依赖编译器特定宏，也可以手动将字节数转换为字符数：

bool CCenterCursorOnScreenDlg::SetTrayIconTipText()
{
    const size_t maxTipChars = sizeof(m_NID.szTip) / sizeof(TCHAR);
    if (StringCchCopy(m_NID.szTip, maxTipChars, m_strTrayIconTip) != S_OK)
        return FALSE;

    m_NID.uFlags |= NIF_TIP;

    return Shell_NotifyIcon(NIM_MODIFY, &m_NID);
}

额外优化（可选） #

为了更严谨，可以提前检查源字符串长度，避免不必要的截断：

bool CCenterCursorOnScreenDlg::SetTrayIconTipText()
{
    const size_t maxTipChars = _countof(m_NID.szTip);
    if (m_strTrayIconTip.GetLength() >= maxTipChars)
    {
        // 可在此添加截断提示或日志逻辑
        m_strTrayIconTip = m_strTrayIconTip.Left(maxTipChars - 1);
    }
    
    if (StringCchCopy(m_NID.szTip, maxTipChars, m_strTrayIconTip) != S_OK)
        return FALSE;

    m_NID.uFlags |= NIF_TIP;

    return Shell_NotifyIcon(NIM_MODIFY, &m_NID);
}

内容的提问来源于stack exchange，提问作者Andrew Truckle