[PATCH] winegstreamer: Prevent hanging when loading a short MP3 file.

Akihiro Sagawa sagawa.aki at gmail.com
Wed Jun 10 07:26:54 CDT 2020


If Info tag[1] isn't present in the MP3 file, GStreamer trys to read
some more frames from the file to estimate a duration. Then, it sends
a duration-changed message to winegstreamer.
However, it seems not to send the message if the file is too short to
estimate the duration, e.g. 0.5 seconds. Winegstreamer gets stuck in
this case.

This patch removes Info tag from dlls/quartz/tests/test.mp3 and fixes
the above issue by watching pad's EOS events.

[1] http://gabriel.mp3-tech.org/mp3infotag.html

Signed-off-by: Akihiro Sagawa <sagawa.aki at gmail.com>
---
 dlls/quartz/tests/rsrc.rc     |   2 +-
 dlls/quartz/tests/test.mp3    | Bin 2349 -> 2157 bytes
 dlls/winegstreamer/gstdemux.c |  13 ++++++++++---
 3 files changed, 11 insertions(+), 4 deletions(-)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-winegstreamer-Prevent-hanging-when-loading-a-short-M.patch
Type: text/x-patch
Size: 7919 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20200610/2c830f52/attachment-0001.bin>


More information about the wine-devel mailing list