[Insight-developers] Valgrind need attention

[Johnson, Hans J]

Whoo Hoo!

That sounds like a nasty little bug.

Hans


On 7/29/11 4:54 PM, "Magnotta, Vincent A" <vincent-magnotta at uiowa.edu>
wrote:

>Kent,
>
>I have tracked down and fixed the memory leak. It resulted from a
>circular reference between Nodes and Elements. I have a patch that I will
>send along Monday once after more testing.
>
>Vince
>
>--
>Associate Professor
>Department of Radiology
>0453-D JCP
>200 Hawkins Drive
>Iowa City, IA 52242
>E-mail: vincent-magnotta at uiowa.edu
>Phone: 319-356-8255 Fax: 319-353-6275
>Website: http://www.healthcare.uiowa.edu/Radiology
>
>
>________________________________________
>From: insight-developers-bounces at itk.org
>[insight-developers-bounces at itk.org] on behalf of Williams, Norman K
>[norman-k-williams at uiowa.edu]
>Sent: Thursday, July 28, 2011 4:27 PM
>To: Bill Lorensen
>Cc: ITK
>Subject: Re: [Insight-developers] Valgrind need attention
>
>That's an excellent question, Bill. And when I finish the other stuff I
>have to get done and my eyes stop bleeding from staring at the code for
>days last week, I'll attempt to answer it.
>
>
>On 7/28/11 4:16 PM, "Bill Lorensen" <bill.lorensen at gmail.com> wrote:
>
>>Why didn't the fem code leak before the refactorization?
>>
>>On Thu, Jul 28, 2011 at 5:09 PM, Williams, Norman K
>><norman-k-williams at uiowa.edu> wrote:
>>
>>As I explained in direct e-mail to Bill, I spent a couple of days staring
>>at the FEM code and couldn't find the problems causing the remaining FEM
>>valgrind leak reports.  Then I threw up my hands and Vince Magnotta has
>>been putting some hours in and came to similar conclusion.
>>
>>
>>The problem is that memory that is allocated by the ::New() method, and
>>only ever assigned to smart pointers, is being reported as leaked. Which
>>means that there are 3 possibilities:
>>
>>1. Somehow, somewhere, the reference count is getting double-incremented.
>>2. There really is a leak somewhere that's eluded both Vince & I.
>>3. Valgrind is wrong.
>>
>>My last exchange with Bill he indicated that he believed the leak was
>>real.  If that's the case, it's eluded 2 reasonably intelligent &
>>experienced programmers for a minimum of 40 man hours.  If someone else
>>has any insights they would be extremely welcome.
>>
>>On 7/28/11 3:48 PM, "Nicholas Tustison" <ntustison at gmail.com> wrote:
>>
>>>I'll get to it right away.
>>>Nick
>>>
>>>On Jul 28, 2011, at 4:09 PM, Bill Lorensen wrote:
>>>
>>>
>>>FEM and BSpline checkins have numerous valgrind defects. There are also
>>>a
>>>bunch of doxygen related defects. We need to zero these out.
>>>
>>>http://www.cdash.org/CDash/viewDynamicAnalysis.php?buildid=1373394
>>>
>>>Bill
>>>
>>>
>>>
>>>
>>>
>>
>>
>>
>>
>>
>>________________________________
>>Notice: This UI Health Care e-mail (including attachments) is covered by
>>the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is
>>confidential and may be legally privileged.  If you are not the intended
>>recipient, you are hereby notified that any retention, dissemination,
>>distribution, or copying of this communication is strictly prohibited.
>>Please reply to the sender that you have received the message in error,
>>then delete it.  Thank you.
>>
>>________________________________
>>
>>
>>
>>
>
>
>
>________________________________
>Notice: This UI Health Care e-mail (including attachments) is covered by
>the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is
>confidential and may be legally privileged.  If you are not the intended
>recipient, you are hereby notified that any retention, dissemination,
>distribution, or copying of this communication is strictly prohibited.
>Please reply to the sender that you have received the message in error,
>then delete it.  Thank you.
>________________________________
>_______________________________________________
>Powered by www.kitware.com
>
>Visit other Kitware open-source projects at
>http://www.kitware.com/opensource/opensource.html
>
>Kitware offers ITK Training Courses, for more information visit:
>http://kitware.com/products/protraining.html
>
>Please keep messages on-topic and check the ITK FAQ at:
>http://www.itk.org/Wiki/ITK_FAQ
>
>Follow this link to subscribe/unsubscribe:
>http://www.itk.org/mailman/listinfo/insight-developers
>
>
>________________________________
>Notice: This UI Health Care e-mail (including attachments) is covered by
>the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is
>confidential and may be legally privileged.  If you are not the intended
>recipient, you are hereby notified that any retention, dissemination,
>distribution, or copying of this communication is strictly prohibited.
>Please reply to the sender that you have received the message in error,
>then delete it.  Thank you.
>________________________________
>_______________________________________________
>Powered by www.kitware.com
>
>Visit other Kitware open-source projects at
>http://www.kitware.com/opensource/opensource.html
>
>Kitware offers ITK Training Courses, for more information visit:
>http://kitware.com/products/protraining.html
>
>Please keep messages on-topic and check the ITK FAQ at:
>http://www.itk.org/Wiki/ITK_FAQ
>
>Follow this link to subscribe/unsubscribe:
>http://www.itk.org/mailman/listinfo/insight-developers



________________________________
Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged.  If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited.  Please reply to the sender that you have received the message in error, then delete it.  Thank you.
________________________________