Index: media/filters/jpeg_parser.cc |
diff --git a/media/filters/jpeg_parser.cc b/media/filters/jpeg_parser.cc |
index bb07bd710aa54f8d6444e4c9af31fd7dad320f59..d46a7e306a477cbc297dce3fae1fb293a0f9d2cb 100644 |
--- a/media/filters/jpeg_parser.cc |
+++ b/media/filters/jpeg_parser.cc |
@@ -269,6 +269,66 @@ static bool ParseSOS(const char* buffer, |
return true; |
} |
+// |eoi_ptr| will point to the end of image (after EOI marker) after search |
+// succeeds. Returns true on EOI marker found, or false. |
+static bool SearchEOI(const char* buffer, size_t length, const char** eoi_ptr) { |
+ DCHECK(buffer); |
+ DCHECK(eoi_ptr); |
+ BigEndianReader reader(buffer, length); |
+ uint8_t marker2; |
+ |
+ while (reader.remaining() > 0) { |
+ const char* marker1_ptr = static_cast<const char*>( |
+ memchr(reader.ptr(), JPEG_MARKER_PREFIX, reader.remaining())); |
+ if (!marker1_ptr) |
+ return false; |
+ reader.Skip(marker1_ptr - reader.ptr() + 1); |
+ |
+ do { |
+ READ_U8_OR_RETURN_FALSE(&marker2); |
+ } while (marker2 == JPEG_MARKER_PREFIX); // skip fill bytes |
+ |
+ switch (marker2) { |
+ // Compressed data escape. |
+ case 0x00: |
+ break; |
+ // Restart |
+ case JPEG_RST0: |
+ case JPEG_RST1: |
+ case JPEG_RST2: |
+ case JPEG_RST3: |
+ case JPEG_RST4: |
+ case JPEG_RST5: |
+ case JPEG_RST6: |
+ case JPEG_RST7: |
+ break; |
+ case JPEG_EOI: |
+ *eoi_ptr = reader.ptr(); |
+ return true; |
+ default: |
+ // Skip for other markers. |
+ uint16_t size; |
+ READ_U16_OR_RETURN_FALSE(&size); |
+ if (size < sizeof(size)) { |
+ DLOG(ERROR) << "Ill-formed JPEG. Segment size (" << size |
+ << ") is smaller than size field (" << sizeof(size) |
+ << ")"; |
+ return false; |
+ } |
+ size -= sizeof(size); |
+ |
+ if (!reader.Skip(size)) { |
+ DLOG(ERROR) << "Ill-formed JPEG. Remaining size (" |
+ << reader.remaining() |
+ << ") is smaller than header specified (" << size << ")"; |
+ return false; |
+ } |
+ break; |
+ } |
+ } |
+ return false; |
+} |
+ |
// |result| is already initialized to 0 in ParseJpegPicture. |
static bool ParseSOI(const char* buffer, |
size_t length, |
@@ -372,6 +432,8 @@ static bool ParseSOI(const char* buffer, |
// Scan data follows scan header immediately. |
result->data = reader.ptr(); |
result->data_size = reader.remaining(); |
+ const size_t kSoiSize = 2; |
+ result->image_size = length + kSoiSize; |
return true; |
} |
@@ -395,4 +457,25 @@ bool ParseJpegPicture(const uint8_t* buffer, |
return ParseSOI(reader.ptr(), reader.remaining(), result); |
} |
+bool ParseJpegStream(const uint8_t* buffer, |
+ size_t length, |
+ JpegParseResult* result) { |
+ DCHECK(buffer); |
+ DCHECK(result); |
+ if (!ParseJpegPicture(buffer, length, result)) |
+ return false; |
+ |
+ BigEndianReader reader( |
+ reinterpret_cast<const char*>(result->data), result->data_size); |
+ const char* eoi_ptr = nullptr; |
+ if (!SearchEOI(reader.ptr(), reader.remaining(), &eoi_ptr)) { |
+ DLOG(ERROR) << "SearchEOI failed"; |
+ return false; |
+ } |
+ DCHECK(eoi_ptr); |
+ result->data_size = eoi_ptr - result->data; |
+ result->image_size = eoi_ptr - reinterpret_cast<const char*>(buffer); |
+ return true; |
+} |
+ |
} // namespace media |