Merge topic 'fixNullptrAccess'

b38a9f0 BUG: fixed null access found by UBSan

Author: dzenanz

Modules/Core/Transform/include/itkTransform.hxx

@@ -516,12 +516,16 @@ Transform<TParametersValueType, NInputDimensions, NOutputDimensions>
 ::CopyInParameters(const ParametersValueType * const begin,
                    const ParametersValueType * const end)
 {
-   //Ensure that we are not copying onto self
-   if( begin != &(this->m_Parameters[0]) )
-   {
-   //Copy raw values array
-   std::copy(begin,end,this->m_Parameters.data_block() );
-   }
+  if ( begin == end )
+    {
+    return;
+    }
+  //Ensure that we are not copying onto self
+  if( begin != &(this->m_Parameters[0]) )
+    {
+    //Copy raw values array
+    std::copy(begin,end,this->m_Parameters.data_block() );
+    }
   //Now call child class set parameter to interpret raw values
   this->SetParameters(this->m_Parameters);
 }
@@ -534,12 +538,16 @@ Transform<TParametersValueType, NInputDimensions, NOutputDimensions>
 ::CopyInFixedParameters(const FixedParametersValueType * const begin,
                         const FixedParametersValueType * const end)
 {
-   //Ensure that we are not copying onto self
-   if( begin != &(this->m_FixedParameters[0]) )
-   {
-   //Copy raw values array
-   std::copy(begin,end,this->m_FixedParameters.data_block() );
-   }
+  if ( begin == end )
+    {
+    return;
+    }
+  //Ensure that we are not copying onto self
+  if( begin != &(this->m_FixedParameters[0]) )
+    {
+    //Copy raw values array
+    std::copy(begin,end,this->m_FixedParameters.data_block() );
+    }
   //Now call child class set parameter to interpret raw values
   this->SetFixedParameters(this->m_FixedParameters);
 }