MFC Ribbon快速访问工具栏执行状态切换命令时崩溃问题排查

问题原因 #

你遇到的崩溃是MFC Ribbon框架的经典野指针问题，核心触发逻辑如下：

• 快速访问工具栏的按钮点击后，CMFCRibbonBar内部的m_pHighlighted成员变量会保留指向当前点击按钮的指针，用来处理高亮状态更新
• 你调用LoadState方法时，MFC内部会销毁所有原有Ribbon元素（包括快速访问工具栏的全部按钮）再根据配置重建，原有按钮对象被释放后，m_pHighlighted就变成了野指针
• 后续OnLButtonUp、OnMouseMove事件触发时，MFC底层直接访问这个已经失效的m_pHighlighted指针，就会触发访问违规崩溃
• 常规Ribbon区域的按钮不会触发崩溃，是因为常规区域的点击处理逻辑会在执行命令前主动清除m_pHighlighted指针，而快速访问工具栏的事件处理没有做这个清理步骤。

修复方案 #

优先选择改动最小的方案，不需要修改继承关系也不需要调整业务逻辑，在执行状态保存加载前主动清除Ribbon的高亮状态即可：

方案1：发送WM_CANCELMODE消息（推荐） #

MFC原生支持WM_CANCELMODE消息，收到该消息后RibbonBar会自动清理所有高亮、按下状态，清空m_pHighlighted等状态指针：

// 假设m_wndRibbonBar是你主框架类中的RibbonBar成员变量
m_wndRibbonBar.SendMessage(WM_CANCELMODE);

// 再执行原来的状态保存加载逻辑
theApp.SaveState(this, ToolbarKey(i_oldAppState));
theApp.LoadState(this, ToolbarKey(i_newAppState));

方案2：延迟执行状态加载 #

如果方案1不生效，可以把状态保存加载逻辑延迟到当前消息队列处理完成后执行，等待按钮点击的全流程事件处理完毕，MFC会自动完成状态清理，此时再加载就不会有野指针问题：

// 首先在主框架类头文件定义自定义消息
#define WM_SWITCH_APP_STATE (WM_USER + 100)

// 原来的状态切换命令处理函数中，替换直接调用Save/Load的代码为PostMessage
PostMessage(WM_SWITCH_APP_STATE, i_oldAppState, i_newAppState);

// 新增自定义消息处理函数
LRESULT CMainFrame::OnSwitchAppState(WPARAM wParam, LPARAM lParam)
{
    int i_oldAppState = static_cast<int>(wParam);
    int i_newAppState = static_cast<int>(lParam);
    theApp.SaveState(this, ToolbarKey(i_oldAppState));
    theApp.LoadState(this, ToolbarKey(i_newAppState));
    return 0;
}

方案3：继承CMFCRibbonBar新增清理方法 #

如果前两种方案都不满足你的场景，可以自定义RibbonBar类新增公开清理方法，主动清空状态指针：

class CMyRibbonBar : public CMFCRibbonBar
{
public:
    void ClearAllState()
    {
        // 清理高亮状态
        if (m_pHighlighted != nullptr)
        {
            m_pHighlighted->m_bIsHighlighted = FALSE;
            m_pHighlighted->OnHighlight(FALSE);
            InvalidateRect(m_pHighlighted->GetRect());
            m_pHighlighted = nullptr;
        }
        // 同步清理按下状态避免其他潜在崩溃
        if (m_pPressed != nullptr)
        {
            m_pPressed->m_bIsPressed = FALSE;
            InvalidateRect(m_pPressed->GetRect());
            m_pPressed = nullptr;
        }
    }
};

调用Save/LoadState之前先执行m_wndRibbonBar.ClearAllState()即可。

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