search

BRAINSia / BRAINSTools

A suite of tools for medical image processing focused on brain analysis
http://brainsia.github.io/BRAINSTools/
Apache License 2.0
113 stars 96 forks source link

BRAINSABC segfaults without the '--atlasToSubjectInitialTransform' flag #153

Closed hjmjohnson closed 9 years ago

hjmjohnson commented 10 years ago

From: , Hans Johnson hans-johnson@uiowa.edu Date: Wednesday, September 24, 2014 at 9:00 AM To: Vincent Magnotta vincent-magnotta@uiowa.edu Cc: Kent Williams norman-k-williams@uiowa.edu, David Welch david-welch@uiowa.edu Subject: Re: BRAINSABC Bug

Kent,

Please initialize this variable to an identity transform so that if it is not explicitly specified that it will have the correct value.

Hans

From: , Vincent Magnotta vincent-magnotta@uiowa.edu Date: Wednesday, September 24, 2014 at 8:50 AM To: Hans Johnson hans-johnson@uiowa.edu Cc: Kent Williams norman-k-williams@uiowa.edu, David Welch david-welch@uiowa.edu Subject: BRAINSABC Bug

If you run BRAINSABC from the command line without the —atlasToSubjectInitialTransform option it will seg fault on line 433 of AtlasRegistrationMethod.hxx because the variable m_AtlasToSubjectInitialTransform was not being initialized in this case. I just tried a test where I set this variable to NULL in the class initializer and the segmentation fault here was eliminated. However it is still seg faulting in the ITKv4 Registration.

Vince

Associate Professor Department of Radiology 0453-D JCP 200 Hawkins Drive Iowa City, IA 52242 E-mail: vincent-magnotta@uiowa.edu Phone: 319-356-8255 Fax: 319-353-6275 Website: http://www.healthcare.uiowa.edu/Radiology

hjmjohnson commented 9 years ago

Ali,

I think you fixed this already. Please close issue if it is fixed.

Hans

aghayoor commented 9 years ago

Yes, it should be fixed now.