Menu

#942 Division by zero at low ParseSpeed for certain .ts

Crash
pending
None
9
2015-08-15
2015-08-15
No

cli mediainfo crashes with division-by-zero when ParseSpeed is set to 0.2 or lower for a specific .ts file.
Crash occurs on 0.7.76, but doesn't occur on 0.7.74.
I compiled the code to get debug symbols and traced the crash to File_MpegTs.cpp line 1945. Duration is zero.
The file has a couple of text streams, so it probably doesn't find more than one frame at that ParseSpeed resulting in a zero duration.

Originally it crashed in an app that uses the Open_Buffer_Continue api, it happens at around 64k so I can supply the first 128kb or so to repro, lemme know if you need it.
Atm I worked around it by increasing the ParseSpeed to 0.3 for .ts files, which probably a good idea anyway for such files.

Let me know if you need the file fragment or any additional information. Thank you.

Discussion

  • Jerome Martinez

    Jerome Martinez - 2015-08-15
    • status: open --> pending
    • assigned_to: Jerome Martinez
    • Priority: 5 --> 9
     
  • Jerome Martinez

    Jerome Martinez - 2015-08-15

    File_MpegTs.cpp line 1945.

    Thanks for having looked at it.

    Let me know if you need the file fragment

    I do.
    (I prefer to have it for checking the exact correct behavior and for doing more than handling the duration of 0 and for adding it to non regression tests)

     
  • Taloth Saldono

    Taloth Saldono - 2015-08-15

    Okay, sent it over the mail.

     

Log in to post a comment.