fix_1522_jp2image_exif_asan
This commit is contained in:
parent
a750ea2780
commit
22ea582c6b
@ -28,6 +28,7 @@
|
||||
#include "image.hpp"
|
||||
#include "image_int.hpp"
|
||||
#include "basicio.hpp"
|
||||
#include "enforce.hpp"
|
||||
#include "error.hpp"
|
||||
#include "futils.hpp"
|
||||
#include "types.hpp"
|
||||
@ -353,7 +354,7 @@ static void boxes_check(size_t b,size_t m)
|
||||
if (io_->error()) throw Error(kerFailedToReadImageData);
|
||||
if (bufRead != rawData.size_) throw Error(kerInputDataReadFailed);
|
||||
|
||||
if (rawData.size_ > 0)
|
||||
if (rawData.size_ > 8) // "II*\0long"
|
||||
{
|
||||
// Find the position of Exif header in bytes array.
|
||||
long pos = ( (rawData.pData_[0] == rawData.pData_[1])
|
||||
@ -497,6 +498,7 @@ static void boxes_check(size_t b,size_t m)
|
||||
position = io_->tell();
|
||||
box.length = getLong((byte*)&box.length, bigEndian);
|
||||
box.type = getLong((byte*)&box.type, bigEndian);
|
||||
enforce(box.length <= io_->size()-io_->tell() , Exiv2::kerCorruptedMetadata);
|
||||
|
||||
if (bPrint) {
|
||||
out << Internal::stringFormat("%8ld | %8ld | ", (size_t)(position - sizeof(box)),
|
||||
@ -581,12 +583,13 @@ static void boxes_check(size_t b,size_t m)
|
||||
throw Error(kerInputDataReadFailed);
|
||||
|
||||
if (bPrint) {
|
||||
out << Internal::binaryToString(makeSlice(rawData, 0, 40));
|
||||
out << Internal::binaryToString(
|
||||
makeSlice(rawData, 0, rawData.size_>40?40:rawData.size_));
|
||||
out.flush();
|
||||
}
|
||||
lf(out, bLF);
|
||||
|
||||
if (bIsExif && bRecursive && rawData.size_ > 0) {
|
||||
if (bIsExif && bRecursive && rawData.size_ > 8) { // "II*\0long"
|
||||
if ((rawData.pData_[0] == rawData.pData_[1]) &&
|
||||
(rawData.pData_[0] == 'I' || rawData.pData_[0] == 'M')) {
|
||||
BasicIo::AutoPtr p = BasicIo::AutoPtr(new MemIo(rawData.pData_, rawData.size_));
|
||||
|
||||
Loading…
Reference in New Issue
Block a user