Hi again,

will it help if we explicitely cast the (-1) to char in the two
occasions where it is used for sbr_present_flag?

 - Fabian
diff --git a/libfaad/mp4.c b/libfaad/mp4.c
index 72b2af6..14e607a 100644
--- a/libfaad/mp4.c
+++ b/libfaad/mp4.c
@@ -174,7 +174,7 @@ int8_t AudioSpecificConfigFromBitfile(bitfile *ld,
 #endif
 
 #ifdef SBR_DEC
-    mp4ASC->sbr_present_flag = -1;
+    mp4ASC->sbr_present_flag = (char)-1;
     if (mp4ASC->objectTypeIndex == 5)
     {
         uint8_t tmp;
@@ -276,7 +276,7 @@ int8_t AudioSpecificConfigFromBitfile(bitfile *ld,
 
     /* no SBR signalled, this could mean either implicit signalling or no SBR in this file */
     /* MPEG specification states: assume SBR on files with samplerate <= 24000 Hz */
-    if (mp4ASC->sbr_present_flag == -1)
+    if (mp4ASC->sbr_present_flag == (char)-1)
     {
         if (mp4ASC->samplingFrequency <= 24000)
         {

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to