[FFmpeg-devel] avformat/nutdec: fix obvious typo

Submitted by Paul B Mahol on Dec. 13, 2018, 10:18 p.m.

Details

Message ID 20181213221822.28094-1-onemda@gmail.com
State Accepted
Headers show

Commit Message

Paul B Mahol Dec. 13, 2018, 10:18 p.m.
Signed-off-by: Paul B Mahol <onemda@gmail.com>
---
 libavformat/nutdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

mypopy@gmail.com Dec. 14, 2018, 12:54 a.m.
On Fri, Dec 14, 2018 at 6:18 AM Paul B Mahol <onemda@gmail.com> wrote:
>
> Signed-off-by: Paul B Mahol <onemda@gmail.com>
> ---
>  libavformat/nutdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/nutdec.c b/libavformat/nutdec.c
> index 27440c88d4..9910ffe936 100644
> --- a/libavformat/nutdec.c
> +++ b/libavformat/nutdec.c
> @@ -582,7 +582,7 @@ static int decode_info_header(NUTContext *nut)
>              if (stream_id_plus1 && !strcmp(name, "r_frame_rate")) {
>                  sscanf(str_value, "%d/%d", &st->r_frame_rate.num, &st->r_frame_rate.den);
>                  if (st->r_frame_rate.num >= 1000LL*st->r_frame_rate.den ||
> -                    st->r_frame_rate.num < 0 || st->r_frame_rate.num < 0)
> +                    st->r_frame_rate.num < 0 || st->r_frame_rate.den < 0)
>                      st->r_frame_rate.num = st->r_frame_rate.den = 0;
>                  continue;
>              }
> --
> 2.17.1
>
LGTM
Michael Niedermayer Dec. 14, 2018, 3:23 a.m.
On Thu, Dec 13, 2018 at 11:18:22PM +0100, Paul B Mahol wrote:
> Signed-off-by: Paul B Mahol <onemda@gmail.com>
> ---
>  libavformat/nutdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

LGTM

thx

[...]

Patch hide | download patch | download mbox

diff --git a/libavformat/nutdec.c b/libavformat/nutdec.c
index 27440c88d4..9910ffe936 100644
--- a/libavformat/nutdec.c
+++ b/libavformat/nutdec.c
@@ -582,7 +582,7 @@  static int decode_info_header(NUTContext *nut)
             if (stream_id_plus1 && !strcmp(name, "r_frame_rate")) {
                 sscanf(str_value, "%d/%d", &st->r_frame_rate.num, &st->r_frame_rate.den);
                 if (st->r_frame_rate.num >= 1000LL*st->r_frame_rate.den ||
-                    st->r_frame_rate.num < 0 || st->r_frame_rate.num < 0)
+                    st->r_frame_rate.num < 0 || st->r_frame_rate.den < 0)
                     st->r_frame_rate.num = st->r_frame_rate.den = 0;
                 continue;
             }