Parcourir la source

Merge pull request #1008 from wise86-android/coverity_scan_fix

153083/2 Buffer not null terminated
Kim Kulling il y a 9 ans
Parent
commit
a3be51ecf3
2 fichiers modifiés avec 3 ajouts et 3 suppressions
  1. 1 1
      code/MDLLoader.cpp
  2. 2 2
      code/SMDLoader.cpp

+ 1 - 1
code/MDLLoader.cpp

@@ -1557,7 +1557,7 @@ void MDLImporter::InternReadFile_3DGS_MDL7( )
 			} else {
 				pcNode->mName.length = ::strlen(szBuffer);
 			}
-            ::strncpy(pcNode->mName.data,szBuffer,MAXLEN);
+            ::strncpy(pcNode->mName.data,szBuffer,MAXLEN-1);
             ++p;
         }
     }

+ 2 - 2
code/SMDLoader.cpp

@@ -641,7 +641,7 @@ void SMDImporter::ComputeAbsoluteBoneTransformations()
         bone.mOffsetMatrix.Inverse();
     }
 }
-
+\
 // ------------------------------------------------------------------------------------------------
 // create output materials
 void SMDImporter::CreateOutputMaterials()
@@ -660,7 +660,7 @@ void SMDImporter::CreateOutputMaterials()
 
         if (aszTextures[iMat].length())
         {
-            ::strncpy(szName.data, aszTextures[iMat].c_str(),MAXLEN );
+            ::strncpy(szName.data, aszTextures[iMat].c_str(),MAXLEN-1);
             szName.length = aszTextures[iMat].length();
             pcMat->AddProperty(&szName,AI_MATKEY_TEXTURE_DIFFUSE(0));
         }