Opened 6 years ago

Closed 4 years ago

Last modified 4 years ago

#6751 closed defect (fixed)

libavformat/mpegtsenc.c:1646: bad if statement ?

Reported by: dcb Owned by:
Priority: normal Component: avformat
Version: unspecified Keywords:
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description (last modified by Carl Eugen Hoyos)

ffmpeg-3.4/libavformat/mpegtsenc.c:1646]: (warning) Logical conjunction always evaluates to false: EXPR < 32 && EXPR >= 48.

Source code is

       if ((state & 0x7e) < 2*16 && (state & 0x7e) >= 2*24)

maybe better code

       if ((state & 0x7e) < 2*16 || (state & 0x7e) >= 2*24)

Change History (3)

comment:1 by Balling, 4 years ago

Resolution: duplicate
Status: newclosed

Merged in #8156

Last edited 4 years ago by Balling (previous) (diff)

comment:2 by Carl Eugen Hoyos, 4 years ago

Description: modified (diff)
Resolution: duplicate
Status: closedreopened

comment:3 by Jun Zhao, 4 years ago

Resolution: fixed
Status: reopenedclosed

Closed, fixed by commit: 8f7a043609554fd5e6917afeaed5d95bca420859

Last edited 4 years ago by Carl Eugen Hoyos (previous) (diff)
Note: See TracTickets for help on using tickets.