[FFmpeg-cvslog] ffprobe: report unavailable SAR correctly in stream info

Timo Teräs git at videolan.org
Tue Apr 17 22:14:36 EEST 2018


ffmpeg | branch: master | Timo Teräs <timo.teras at iki.fi> | Thu Apr 12 11:07:36 2018 +0300| [c663dce031b3973e37c83ae1818f1484e1cf482c] | committer: Rostislav Pehlivanov

ffprobe: report unavailable SAR correctly in stream info

av_guess_sample_aspect_ratio() will return undefined or missing
value as {0,1}. This fixes show_stream() to check numerator to
display 'N/A' when appropriate. show_frame() does this already
correctly.

Signed-off-by: Timo Teräs <timo.teras at iki.fi>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=c663dce031b3973e37c83ae1818f1484e1cf482c
---

 fftools/ffprobe.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c
index 82dfe4f58a..8b2a18b6b1 100644
--- a/fftools/ffprobe.c
+++ b/fftools/ffprobe.c
@@ -2521,7 +2521,7 @@ static int show_stream(WriterContext *w, AVFormatContext *fmt_ctx, int stream_id
 #endif
         print_int("has_b_frames", par->video_delay);
         sar = av_guess_sample_aspect_ratio(fmt_ctx, stream, NULL);
-        if (sar.den) {
+        if (sar.num) {
             print_q("sample_aspect_ratio", sar, ':');
             av_reduce(&dar.num, &dar.den,
                       par->width  * sar.num,



More information about the ffmpeg-cvslog mailing list