From: Ivan M. <iv...@ma...> - 2018-05-10 15:25:41
|
Hi Christophe I sent you an email privately but I'm not sure if you got it. The first patch was based on my erroneous belief that the library was only allocating memory for one line of chroma for each frame. Looking at the code again now I can see that the library is allocating exactly the right amount of memory. Both of my patches and my original message should therefore be disregarded. I have some useful modifications to libmpeg2 which report decoding errors to the user via a flag in the mpeg2_info_t struct, however I want to test them more and write some documentation for the README before submitting them. I would also be interested in attempting to write error concealment code for libmpeg2 if nobody else is doing it. All the best, Ivan On 05/05/18 22:54, Christophe Massiot wrote: > Hi, > > Can you provide a sample to reproduce the crash supposedly fixed by your first patch? Privately if needed. > > Thanks in advance > >> On 19 Apr 2018, at 00:50, Ivan Maguidhir <iv...@ma...> wrote: >> >> Dear libmpeg2 developers, >> >> Find attached two patches which fix problems that I encountered while >> processing MPEG2 streams from a large number of DVD-Video discs using the >> library. The P0_buffer_overflow.patch fixes bugs to do with the allocation >> of memory for each frame's chroma information. The >> P1_unexpected_gop_header.patch causes the parser to accept a GOP header when >> a picture has already started. This can happen where streams have been >> spliced together, for example. I hope these are of some use. >> >> Kind regards, >> Ivan Maguidhir >> >> <P0_buffer_overflow.patch><P1_unexpected_gop_header.patch>------------------------------------------------------------------------------ >> Check out the vibrant tech community on one of the world's most >> engaging tech sites, Slashdot.org! http://sdm.link/slashdot_______________________________________________ >> Libmpeg2-devel mailing list >> Lib...@li... >> https://lists.sourceforge.net/lists/listinfo/libmpeg2-devel |