Bug#641593: add project category crash resurfaced

[Guo Yixuan]

tags 641593 + pending
thanks


Hello,

On 05/30/2012 03:53 PM, Rom Walton wrote:
> Here is a change that I think might resolve the issue.
>
> I'm having problems committing it right now, SSL is going through the
> process of shutting down to repair an electrical short that happened
> a few weeks ago.
>
> If this doesn't fix the problem I have another idea, but I think this
> will resolve the issue.

Yes, this works. I've pushed the patch to our packaging git repo. Thank 
you, Rom. (Note: I've edited it to remove some unrelated part to apply 
it cleanly against 7.0.28, attached the shorted patch.)

> Thanks for a complete stacktrace and the steps you used to create it.
> I didn't know about --enable-wx-debug.

Actually I found it when try to solve another boincmgr bug (shrinked 
menubar, which is fixed in wx2.8 now).

Cheers,

Guo Yixuan


Index: boinc/clientgui/ProjectInfoPage.cpp
--- boinc.orig/clientgui/ProjectInfoPage.cpp	2012-05-30 
23:05:52.000000000 +0800
+++ boinc/clientgui/ProjectInfoPage.cpp	2012-05-30 23:05:55.000000000 +0800
@@ -488,6 +488,7 @@
  void CProjectInfoPage::OnProjectSelected( wxCommandEvent& 
WXUNUSED(event) ) {
      wxLogTrace(wxT("Function Start/End"), 
wxT("CProjectInfoPage::OnProjectSelected - Function Begin"));

+    if (m_pProjectsCtrl->GetSelection() != wxNOT_FOUND) {
      CProjectInfo* pProjectInfo = 
(CProjectInfo*)m_pProjectsCtrl->GetClientData(m_pProjectsCtrl->GetSelection());

      wxString strURL = pProjectInfo->m_strURL;
@@ -539,6 +540,7 @@
      // Set tooltip to full text in case ellipsed
 
m_pProjectDetailsOrganizationCtrl->SetToolTip(pProjectInfo->m_strOrganization);

+    }
      wxLogTrace(wxT("Function Start/End"), 
wxT("CProjectInfoPage::OnProjectSelected - Function End"));
  }