diff mbox

[FFmpeg-devel] libavcodec/mpegaudiodecheader.h: fix version check pattern

Message ID 20180712073025.22494-1-ottoka@posteo.de
State Accepted
Commit ce372bc278ce7ff96e661f3bb109fc74d9a22279
Headers show

Commit Message

Karsten Otto July 12, 2018, 7:30 a.m. UTC 
This fixes the check for the reserved MPEG audio version ID,
used to detect an invalid frame header.
---
 libavcodec/mpegaudiodecheader.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Michael Niedermayer July 12, 2018, 11:18 p.m. UTC | #1 
On Thu, Jul 12, 2018 at 09:30:25AM +0200, Karsten Otto wrote:
> This fixes the check for the reserved MPEG audio version ID,
> used to detect an invalid frame header.
> ---
>  libavcodec/mpegaudiodecheader.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

will apply

thx

[...]
diff mbox

Patch

diff --git a/libavcodec/mpegaudiodecheader.h b/libavcodec/mpegaudiodecheader.h
index ed9961250a..1da2a4cb6f 100644
--- a/libavcodec/mpegaudiodecheader.h
+++ b/libavcodec/mpegaudiodecheader.h
@@ -63,7 +63,7 @@  static inline int ff_mpa_check_header(uint32_t header){
     if ((header & 0xffe00000) != 0xffe00000)
         return -1;
     /* version check */
-    if ((header & (3<<19)) == 1)
+    if ((header & (3<<19)) == 1<<19)
         return -1;
     /* layer check */
     if ((header & (3<<17)) == 0)