Flac files : Slave get_time_pos wrong

[Bertrand Florat]

Summary of the bug: When playing Flac files using slave mode, get_time_pos returns erroneous values.

Using your 1.4.0 debian package

How to reproduce:
Download this Flac sample (I reproduced against all of my own flac files I tested) :
​https://helpguide.sony.net/high-res/sample1/v1/data/Sample_BeeMoved_96kHz24bit.flac.zip

mplayer -v 
MPlayer SVN-r38304-8 (C) 2000-2021 MPlayer Team
CPU vendor name: AuthenticAMD  max cpuid level: 13
CPU: AMD Ryzen 5 3400G with Radeon Vega Graphics     (Family: 23, Model: 8, Stepping: 1)
extended cpuid-level: 31
extended cache-info: 33579328
Detected cache-line size is 64 bytes
CPUflags:  MMX: 1 MMX2: 1 3DNow: 0 3DNowExt: 0 SSE: 1 SSE2: 1 SSE3: 1 SSSE3: 1 SSE4: 1 SSE4.2: 1 AVX: 1
Compiled for x86 CPU with extensions: MMX MMX2 SSE SSE2 SSE3 SSSE3 SSE4 SSE4.2 CMOV
get_path('codecs.conf') -> '/home/bflorat/.mplayer/codecs.conf'
Reading optional codecs config file /home/bflorat/.mplayer/codecs.conf: No such file or directory
Reading optional codecs config file /etc/mplayer/codecs.conf: No such file or directory
Using built-in default codecs.conf.
init_freetype
Using SSE2 Optimized OnScreenDisplay

mplayer -quiet -slave  "Sample_BeeMoved_96kHz24bit.flac"
get_time_pos
ANS_TIME_POSITION=-9223372036854775808.0
... some time
get_time_pos
ANS_TIME_POSITION=-9223372036854775808.0
get_percent_pos
ANS_PERCENT_POSITION=7

No issue with mp3 files I tested :

get_time_pos
ANS_TIME_POSITION=13.6

Expected : a positive value in secs
Observed : a negative value (doesn't change during the play)

Only flac format seems affected.

Strange that get_percent_pos on the other hand returns a correct value.

Thanks

[reimar]

I don't think this happens with all flac files (not the one in our samples server for example).
Using -novideo avoids the issue.
The problem is likely the embedded image, which does not have a time stamp.
Don't know if there is a simple, always correct way to know that get_time_pos should NOT return the video time stamp but the audio time stamp.

[reimar]

This works, but it's a bit of a hack and I don't know what the risk is, both breaking something or not fixing some case:

Index: command.c
===================================================================
--- command.c	(Revision 38313)
+++ command.c	(Arbeitskopie)
@@ -3360,7 +3360,7 @@
 
         case MP_CMD_GET_TIME_POS:{
                 float pos = 0;
-                if (sh_video)
+                if (sh_video && sh_video->pts != MP_NOPTS_VALUE)
                     pos = sh_video->pts;
                 else if (sh_audio && mpctx->audio_out)
                     pos =

[Bertrand Florat]

Thanks, I confirm -novideo fixes my issue.