Re: CFrameWnd:: SetDockState

From:

"Sheng Jiang[MVP]" <sheng_jiang@hotmail.com.discuss>

Newsgroups:

microsoft.public.vc.mfc

Date:

Mon, 25 Feb 2008 12:00:55 -0600

Message-ID:

<Og$TUA9dIHA.5996@TK2MSFTNGP04.phx.gbl>

It seems you are reporting this issue to Microsoft at
http://connect.microsoft.com/VisualStudio/feedback/ViewFeedback.aspx?FeedbackID=329354.
You can provide a reset command line switch or option to reset saved toolbar
settings.
--
Sheng Jiang
Microsoft MVP in VC++
"rjparkin" <rjparkin@discussions.microsoft.com> wrote in message
news:8BE49DCF-5DD5-4D3E-9BFE-AC3851139E01@microsoft.com...

Hi

I'm in the process of updating a project from VS2003 to VS2008. After
finally getting the project to build/link its crashes on start up. After

some

investigation I narrowed the problem down to the following function
CFrameWnd:: SetDockState.

VS2008 Version of CFrameWnd:: SetDockState

else // regular dock bar or toolbar in the loading state
{
     pInfo->m_pBar = GetControlBar(pInfo->m_nBarID);

     ASSERT(pInfo->m_pBar != NULL); //toolbar id's probably changed

     if ((pInfo->m_pBar->m_pDockBar != NULL) &&
         (pInfo->m_pBar->m_pDockBar->m_bFloating)) // control is floating

the current state
     {

     // need to dock the control bar

          DockControlBar (pInfo->m_pBar);
     }
}

Should the code not check to make sure pInfo_m_pBar != NULL before
attempting to access pInfo->m_pBar->m_pDockBar? It certainly seems to for

the

rest of the code in the function

e.g.

// the second pass will actually dock all of the control bars and
// set everything correctly
for (i = 0; i < state.m_arrBarInfo.GetSize(); i++)
{

CControlBarInfo* pInfo = (CControlBarInfo*)state.m_arrBarInfo;

ASSERT(pInfo != NULL);

if (pInfo->m_pBar != NULL)

pInfo->m_pBar->SetBarInfo(pInfo, this);

}

This is not a problem for the VS2003 version as it never accesses

m_pDockBar.

VS2003 Version of CFrameWnd:: SetDockState

else // regular dock bar or toolbar
{

pInfo->m_pBar = GetControlBar(pInfo->m_nBarID);

ASSERT(pInfo->m_pBar != NULL); //toolbar id's probably changed
}

Any thoughts?

Regards

Richard