From 5d11208aa09fdb98a3a74f4d0fb0c359a4d382ef Mon Sep 17 00:00:00 2001 From: Kevin Backhouse Date: Sat, 18 Mar 2023 17:26:00 +0000 Subject: [PATCH] Minor refactorings to stop UBSAN from complaining. --- src/tiffcomposite_int.cpp | 2 +- src/tiffvisitor_int.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/tiffcomposite_int.cpp b/src/tiffcomposite_int.cpp index 564338da..a62092eb 100644 --- a/src/tiffcomposite_int.cpp +++ b/src/tiffcomposite_int.cpp @@ -295,7 +295,6 @@ void TiffImageEntry::setStrips(const Value* pSize, const byte* pData, size_t siz } for (size_t i = 0; i < pValue()->count(); ++i) { const size_t offset = pValue()->toUint32(i); - const byte* pStrip = pData + baseOffset + offset; const size_t size = pSize->toUint32(i); if (size > sizeData || offset > sizeData - size || baseOffset > sizeData - size - offset) { @@ -304,6 +303,7 @@ void TiffImageEntry::setStrips(const Value* pSize, const byte* pData, size_t siz << tag() << ": Strip " << std::dec << i << " is outside of the data area; ignored.\n"; #endif } else if (size != 0) { + const byte* pStrip = pData + baseOffset + offset; strips_.emplace_back(pStrip, size); } } diff --git a/src/tiffvisitor_int.cpp b/src/tiffvisitor_int.cpp index 2f5aa80f..822ebb58 100644 --- a/src/tiffvisitor_int.cpp +++ b/src/tiffvisitor_int.cpp @@ -1316,7 +1316,7 @@ void TiffReader::readTiffEntry(TiffEntryBase* object) { << size << ", exceeds buffer size by " // cast to make MSVC happy - << static_cast(pData + size - pLast_) << " Bytes; truncating the entry\n"; + << size - static_cast(pLast_ - pData) << " Bytes; truncating the entry\n"; #endif size = 0; }