misc fixes
authorJean-Marc Valin <jmvalin@jmvalin.ca>
Fri, 19 May 2017 04:19:20 +0000 (00:19 -0400)
committerJean-Marc Valin <jmvalin@jmvalin.ca>
Fri, 19 May 2017 04:19:20 +0000 (00:19 -0400)
include/opusenc.h
src/opusenc.c
src/picture.c

index adda355..a1b7e3e 100644 (file)
@@ -135,8 +135,8 @@ extern "C" {
 /* FIXME: Add type-checking macros to these. */
 #define OPE_SET_PACKET_CALLBACK(x,u) OPE_SET_PACKET_CALLBACK_REQUEST, (x), (u)
 #define OPE_GET_PACKET_CALLBACK(x,u) OPE_GET_PACKET_CALLBACK_REQUEST, (x), (u)
-#define OPE_SET_HEADER_GAIN(x,u) OPE_SET_HEADER_GAIN_REQUEST, (x), (u)
-#define OPE_GET_HEADER_GAIN(x,u) OPE_GET_HEADER_GAIN_REQUEST, (x), (u)
+#define OPE_SET_HEADER_GAIN(x,u) OPE_SET_HEADER_GAIN_REQUEST, __opus_check_int(x)
+#define OPE_GET_HEADER_GAIN(x,u) OPE_GET_HEADER_GAIN_REQUEST, __opus_check_int_ptr(x)
 /*@}*/
 /*@}*/
 
index fd10c93..bb56b83 100644 (file)
@@ -813,7 +813,9 @@ const char *ope_strerror(int error) {
     "invalid picture file",
     "invalid icon file (pictures of type 1 MUST be 32x32 PNGs)"
   };
-  if (error > -30) return opus_strerror(error+10);
+  if (error == 0) return "success";
+  else if (error >= -10) return "unknown error";
+  else if (error > -30) return opus_strerror(error+10);
   else if (error >= OPE_INVALID_ICON) return ope_error_strings[-error-30];
   else return "unknown error";
 }
index c89eaef..fdebd4f 100644 (file)
@@ -376,6 +376,8 @@ char *parse_picture_specification(const char *filename, int picture_type, const
   if(out!=NULL){
     base64_encode(out,(char *)buf+data_offset,data_length);
     if(picture_type>=1&&picture_type<=2)*seen_file_icons|=picture_type;
+  } else {
+    *error = OPE_ALLOC_FAIL;
   }
   free(buf);
   return out;