OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-04 18:06:22 +03:00
Code Packages Releases Activity

0031333: Data Exchange - RWGltf_TriangulationReader::readBuffer() fails to read interleaved buffer at the end of file

Browse Source 
Standard_ReadBuffer has been extended with a flag for reading an interleaved data.
RWGltf_TriangulationReader::readBuffer() uses new flag for reading vertex attributes.
This commit is contained in:
kgv 2020-01-30 17:10:29 +03:00 committed by bugmaster
parent 62beabff7c
commit e2550e48f1
2 changed files with 22 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/RWGltf/RWGltf_TriangulationReader.cxx
View File

@ -283,7 +283,7 @@ bool RWGltf_TriangulationReader::readBuffer (std::istream& theStream,
return false;
}
Standard_ReadBuffer aBuffer (theAccessor.Count * aStride, aStride);
Standard_ReadBuffer aBuffer (theAccessor.Count * aStride - (aStride - sizeof(Graphic3d_Vec3)), aStride, true);
if (!myCoordSysConverter.IsEmpty())
{
for (Standard_Integer aVertIter = 0; aVertIter < aNbNodes; ++aVertIter)
@ -336,7 +336,7 @@ bool RWGltf_TriangulationReader::readBuffer (std::istream& theStream,
{
return false;
}
Standard_ReadBuffer aBuffer (theAccessor.Count * aStride, aStride);
Standard_ReadBuffer aBuffer (theAccessor.Count * aStride - (aStride - sizeof(Graphic3d_Vec3)), aStride, true);
if (!myCoordSysConverter.IsEmpty())
{
for (Standard_Integer aVertIter = 0; aVertIter < aNbNodes; ++aVertIter)
@ -402,7 +402,7 @@ bool RWGltf_TriangulationReader::readBuffer (std::istream& theStream,
return false;
}
Standard_ReadBuffer aBuffer (theAccessor.Count * aStride, aStride);
Standard_ReadBuffer aBuffer (theAccessor.Count * aStride - (aStride - sizeof(Graphic3d_Vec2)), aStride, true);
for (int aVertIter = 0; aVertIter < aNbNodes; ++aVertIter)
{
Graphic3d_Vec2* aVec2 = aBuffer.ReadChunk<Graphic3d_Vec2> (theStream);

24
src/Standard/Standard_ReadBuffer.hxx
View File

@ -25,7 +25,8 @@ public:
//! Constructor with initialization.
Standard_ReadBuffer (int64_t theDataLen,
size_t theChunkLen)
size_t theChunkLen,
bool theIsPartialPayload = false)
: myBufferPtr(NULL),
myBufferEnd(NULL),
myDataLen (0),
@ -34,17 +35,28 @@ public:
myNbChunks (0),
myBufferLen(0)
{
Init (theDataLen, theChunkLen);
Init (theDataLen, theChunkLen, theIsPartialPayload);
}
//! Initialize the buffer.
//! @param theDataLen the full length of input data to read from stream.
//! @param theChunkLen the length of single chunk to read
//! @param theDataLen [in] the full length of input data to read from stream.
//! @param theChunkLen [in] the length of single chunk to read
//! @param theIsPartialPayload [in] when FALSE, theDataLen will be automatically aligned to the multiple of theChunkLen;
//! when TRUE, last chunk will be read from stream exactly till theDataLen
//! allowing portion of chunk to be uninitialized (useful for interleaved data)
void Init (int64_t theDataLen,
size_t theChunkLen)
size_t theChunkLen,
bool theIsPartialPayload = false)
{
myDataRead = 0;
if (theIsPartialPayload)
{
myDataLen = theDataLen;
}
else
{
myDataLen = theDataLen - theDataLen % int64_t(theChunkLen);
}
myChunkLen = theChunkLen;
myNbChunks = sizeof(myBuffer) / theChunkLen;
myBufferLen = theChunkLen * myNbChunks;
@ -95,7 +107,7 @@ private:
}
const int64_t aDataLeft = myDataLen - myDataRead;
if (aDataLeft == 0) // myDataLen should be multiple of myChunkLen
if (aDataLeft <= 0) // myDataLen is normally multiple of myChunkLen, but can be smaller in interleaved data
{
myBufferPtr = NULL;
return NULL;