libmodplug: simplify and fix read_packet() callback.
authorClément Bœsch <ubitux@gmail.com>
Wed, 5 Oct 2011 19:20:28 +0000 (21:20 +0200)
committerClément Bœsch <ubitux@gmail.com>
Wed, 5 Oct 2011 19:25:03 +0000 (21:25 +0200)
In case of av_new_packet() error, a correct return error code is raised,
the data memcpy is avoided, and pkt dts/pts are not assigned anymore
(since the defaults are good).

libavformat/libmodplug.c

index 212e225..b4978b9 100644 (file)
@@ -66,20 +66,16 @@ static int modplug_read_header(AVFormatContext *s, AVFormatParameters *ap)
 
 static int modplug_read_packet(AVFormatContext *s, AVPacket *pkt)
 {
-    int ret, n;
     ModPlugContext *modplug = s->priv_data;
-    uint8_t buf[512];
 
-    n = ModPlug_Read(modplug->f, buf, sizeof(buf));
-    if (n <= 0)
-        return AVERROR(EIO);
+    if (av_new_packet(pkt, 512) < 0)
+        return AVERROR(ENOMEM);
 
-    ret = av_new_packet(pkt, n);
-    if (ret)
-        return ret;
-    pkt->pts  = pkt->dts = AV_NOPTS_VALUE;
-    pkt->size = n;
-    memcpy(pkt->data, buf, n);
+    pkt->size = ModPlug_Read(modplug->f, pkt->data, 512);
+    if (pkt->size <= 0) {
+        av_free_packet(pkt);
+        return AVERROR(EIO);
+    }
     return 0;
 }