Changeset 231920 in webkit
- Timestamp:
- May 17, 2018 2:02:23 PM (6 years ago)
- Location:
- trunk
- Files:
-
- 3 added
- 4 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r231918 r231920 1 2018-05-17 Jer Noble <jer.noble@apple.com> 2 3 CRASH in ImageDecoderAVFObjC::sampleAtIndex() 4 https://bugs.webkit.org/show_bug.cgi?id=185734 5 <rdar://problem/40295094> 6 7 Reviewed by Eric Carlson. 8 9 * fast/images/animated-image-mp4-crash-expected.txt: Added. 10 * fast/images/animated-image-mp4-crash.html: Added. 11 * fast/images/resources/two-samples-with-same-pts.mp4: Added. 12 1 13 2018-05-17 Youenn Fablet <youenn@apple.com> 2 14 -
trunk/Source/WebCore/ChangeLog
r231915 r231920 1 2018-05-17 Jer Noble <jer.noble@apple.com> 2 3 CRASH in ImageDecoderAVFObjC::sampleAtIndex() 4 https://bugs.webkit.org/show_bug.cgi?id=185734 5 <rdar://problem/40295094> 6 7 Reviewed by Eric Carlson. 8 9 Test: fast/images/animated-image-mp4-crash.html 10 11 Test the correct size value before iterating over the SampleMap in presentationOrder() 12 13 * Modules/mediasource/SampleMap.h: 14 (WebCore::PresentationOrderSampleMap::size const): 15 * platform/graphics/avfoundation/objc/ImageDecoderAVFObjC.mm: 16 (WebCore::ImageDecoderAVFObjC::sampleAtIndex const): 17 1 18 2018-05-17 Wenson Hsieh <wenson_hsieh@apple.com> 2 19 -
trunk/Source/WebCore/Modules/mediasource/SampleMap.h
r229893 r231920 54 54 reverse_iterator rend() { return m_samples.rend(); } 55 55 const_reverse_iterator rend() const { return m_samples.rend(); } 56 57 size_t size() const { return m_samples.size(); } 56 58 57 59 WEBCORE_EXPORT iterator findSampleWithPresentationTime(const MediaTime&); -
trunk/Source/WebCore/platform/graphics/avfoundation/objc/ImageDecoderAVFObjC.mm
r231179 r231920 671 671 const ImageDecoderAVFObjCSample* ImageDecoderAVFObjC::sampleAtIndex(size_t index) const 672 672 { 673 if (index >= m_sampleData. size())673 if (index >= m_sampleData.presentationOrder().size()) 674 674 return nullptr; 675 675 … … 680 680 for (size_t i = 0; i != index; ++i) 681 681 ++iter; 682 682 683 683 return toSample(iter); 684 684 }
Note: See TracChangeset
for help on using the changeset viewer.