justdan96 / tsMuxer

tsMuxer is a transport stream muxer for remuxing/muxing elementary streams, EVO/VOB/MPG, MKV/MKA, MP4/MOV, TS, M2TS to TS to M2TS. Supported video codecs H.264/AVC, H.265/HEVC, VC-1, MPEG2. Supported audio codecs AAC, AC3 / E-AC3(DD+), DTS/ DTS-HD.
Apache License 2.0
829 stars 140 forks source link

WARNING - MKV with DTS result in audio out of sync #874

Closed Hellboy00000 closed 1 month ago

Hellboy00000 commented 1 month ago

When using an MKV file with DTS, the result is a file with out-of-sync audio.

The bug was introduced in w64 2024-04-04-01-54-45.

For now i going back to use w64 2024-02-17-01-49-50 Hope people see this warning and don't need to go back to fix a few files like i have.

jcdr428 commented 1 month ago

@Hellboy00000 can you provide a sample of DTS MKV resulting in the bug ? I've tried most of the DTS MKV samples from https://kodi.wiki/view/Samples and could not reproduce.

Hellboy00000 commented 1 month ago

@jcdr428 I created a sample with the first 11 minutes of Super Mario Bros. (1993). (6.2GB) I tested in "M2TS muxing" and "Blu-ray folder" and the bug occurs on both.

It is easier to see the error starting at minute 9:30. Watch when the man collides with the glass and then when they are on the phone.

Password = tsmuxer

https://drive.google.com/file/d/1ulMBIXxqLkWaj_mLki4w3zgZQNs9SrNW/view?usp=sharing https://drive.google.com/file/d/1UKhaqle-Pcx33fgWqHeZQhVt-vLjMuQ9/view?usp=sharing https://drive.google.com/file/d/1FuEDcTksvf4aDZn_yHvmD86wVnQtRtsj/view?usp=sharing https://drive.google.com/file/d/1AKjFbHMQr_hxlMjNkWaEFR78-IQfCJQm/view?usp=sharing https://drive.google.com/file/d/1r4KOyRJG5BYtDNJSUEzFoSuU0Rs-f9Bn/view?usp=sharing https://drive.google.com/file/d/1Mk5XTUmNPOh4dDQg4nM3z6VFNuUj7O4_/view?usp=sharing https://drive.google.com/file/d/1LC25BnK2N9ahNZS9LK-vr2RvHrPtzm2_/view?usp=sharing

Please let me know when finish downloading the files.

jcdr428 commented 1 month ago

@Hellboy00000 I have downloaded the file, thank you.

Edit: good advice about the man colliding in the glass... I've pushed a fix, please try tomorrow's release. @JP3BGY the issue was created by commit 5c0dd70. Could you please confirm the change does not bring any buffer overflow with your poc ?

Hellboy00000 commented 1 month ago

Tested and work great. Thanks.