Decode: return more meaningful error for animation

VP8_STATUS_NOT_ENOUGH_DATA -> VP8_STATUS_UNSUPPORTED_FEATURE

Change-Id: If5ee9fd2c99fc5502996d3c786848fd9cc118fe7

1 files changed, 12 insertions, 3 deletions

diff --git a/src/dec/webp.c b/src/dec/webp.c
index 10b2e264..39d90188 100644
--- a/src/dec/webp.c
+++ b/src/dec/webp.c
@@ -372,10 +372,19 @@ static VP8StatusCode ParseHeadersInternal(const uint8_t* data,
 }
 
 VP8StatusCode WebPParseHeaders(WebPHeaderStructure* const headers) {
+  VP8StatusCode status;
+  int has_animation = 0;
   assert(headers != NULL);
-  // fill out headers, ignore width/height/has_alpha/has_animation.
-  return ParseHeadersInternal(headers->data, headers->data_size,
-                              NULL, NULL, NULL, NULL, headers);
+  // fill out headers, ignore width/height/has_alpha.
+  status = ParseHeadersInternal(headers->data, headers->data_size,
+                                NULL, NULL, NULL, &has_animation, headers);
+  if (status == VP8_STATUS_OK || status == VP8_STATUS_NOT_ENOUGH_DATA) {
+    // TODO(jzern): full support of animation frames will require API additions.
+    if (has_animation) {
+      status = VP8_STATUS_UNSUPPORTED_FEATURE;
+    }
+  }
+  return status;
 }
 
 //------------------------------------------------------------------------------