Codejock Developer Community : Bug in CXTPTabClientWnd::UpdateContents(). http://forum.codejock.com/ Copyright (c) 2006-2013 Web Wiz Forums - All Rights Reserved. Wed, 10 Jun 2026 07:57:46 +0000 Mon, 19 Jun 2006 08:57:49 +0000 http://blogs.law.harvard.edu/tech/rss Web Wiz Forums 12.04 360 forum.codejock.com/RSS_post_feed.asp?TID=4434 <![CDATA[Codejock Developer Community]]> http://forum.codejock.com/forum_images/codejock-logo.gif http://forum.codejock.com/ <![CDATA[Bug in CXTPTabClientWnd::UpdateContents(). : I think you have a bug in method: void...]]> http://forum.codejock.com/forum_posts.asp?TID=4434&PID=13707&title=bug-in-cxtptabclientwndupdatecontents#13707 Author: Sergey
Subject: 4434
Posted: 19 June 2006 at 8:57am

I think you have a bug in method:

void CXTPTabClientWnd::UpdateContents().

The method contains the following code:

if ((pChildWnd->GetStyle() & WS_VISIBLE) == WS_VISIBLE)

{

// add item

pNewItem = pFoundItem = AddItem(pChildWnd);

pFoundItem->m_hWnd = pChildWnd->GetSafeHwnd();

pFrameWnd->SendMessage(WM_XTP_NEWTABITEM, (WPARAM)pNewItem);

pNewItem->GetTabManager()->SetSelectedItem(pNewItem);

}

Method AddItem(pChildWnd) may return NULL (and I really got it!). In this case my application raises Access Violation on the next line because pFoundItem=NULL. So verification for NULL is needed:

if ((pChildWnd->GetStyle() & WS_VISIBLE) == WS_VISIBLE)

{

// add item

   pNewItem = pFoundItem = AddItem(pChildWnd);

   if (pFoundItem)
   {
      pFoundItem->m_hWnd = pChildWnd->GetSafeHwnd();

      pFrameWnd->SendMessage(WM_XTP_NEWTABITEM, (WPARAM)pNewItem);

      pNewItem->GetTabManager()->SetSelectedItem(pNewItem);

   }
}
 
This code works fine!
]]> Mon, 19 Jun 2006 08:57:49 +0000 http://forum.codejock.com/forum_posts.asp?TID=4434&PID=13707&title=bug-in-cxtptabclientwndupdatecontents#13707