ffprobe: report unavailable SAR correctly in stream info
authorTimo Teräs <timo.teras@iki.fi>
Thu, 12 Apr 2018 08:07:36 +0000 (11:07 +0300)
committerRostislav Pehlivanov <atomnuker@gmail.com>
Tue, 17 Apr 2018 19:11:31 +0000 (20:11 +0100)
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@iki.fi>
fftools/ffprobe.c

index 82dfe4f..8b2a18b 100644 (file)
@@ -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,