From 201b409da0d81dce19caa1393fe1c9f8e17aeb92 Mon Sep 17 00:00:00 2001
From: Stefano Sabatini <stefasab@gmail.com>
Date: Fri, 22 Jun 2012 17:43:40 +0200
Subject: [PATCH] lavc/pcm: clarify error message from pcm_decode_frame()

Extend/clarify an error message in case of invalid packet size, and
return a proper error message instead of -1.
Improve debuggability.
---
 libavcodec/pcm.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/libavcodec/pcm.c b/libavcodec/pcm.c
index c72b2003c4..826267fdb2 100644
--- a/libavcodec/pcm.c
+++ b/libavcodec/pcm.c
@@ -293,8 +293,10 @@ static int pcm_decode_frame(AVCodecContext *avctx, void *data,
 
     if (n && buf_size % n) {
         if (buf_size < n) {
-            av_log(avctx, AV_LOG_ERROR, "invalid PCM packet\n");
-            return -1;
+            av_log(avctx, AV_LOG_ERROR,
+                   "Invalid PCM packet, data has size %d but at least a size of %d was expected\n",
+                   buf_size, n);
+            return AVERROR_INVALIDDATA;
         } else
             buf_size -= buf_size % n;
     }