Correct motion vector scaling in B-frames for RV3/4

[ffmpeg] / libavcodec / nuv.c

diff --git a/libavcodec/nuv.c b/libavcodec/nuv.c
index b87e4bd8c1bd6052cce75d1d318ee56a6fdde39a..7f6e5d336eb7744ce9521c04076512d0462ad594 100644 (file)
--- a/libavcodec/nuv.c
+++ b/libavcodec/nuv.c
@@ -21,11 +21,10 @@
 #include <stdio.h>
 #include <stdlib.h>
 
+#include "libavutil/bswap.h"
+#include "libavutil/lzo.h"
 #include "avcodec.h"
-
-#include "bswap.h"
 #include "dsputil.h"
-#include "lzo.h"
 #include "rtjpeg.h"
 
 typedef struct {
@@ -133,6 +132,8 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *data_size,
     NuvContext *c = avctx->priv_data;
     AVFrame *picture = data;
     int orig_size = buf_size;
+    int keyframe;
+    int result;
     enum {NUV_UNCOMPRESSED = '0', NUV_RTJPEG = '1',
           NUV_RTJPEG_IN_LZO = '2', NUV_LZO = '3',
           NUV_BLACK = 'N', NUV_COPY_LAST = 'L'} comptype;
@@ -160,6 +161,15 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *data_size,
         return -1;
     }
     comptype = buf[1];
+    switch (comptype) {
+        case NUV_RTJPEG_IN_LZO:
+        case NUV_RTJPEG:
+            keyframe = !buf[2]; break;
+        case NUV_COPY_LAST:
+            keyframe = 0; break;
+        default:
+            keyframe = 1; break;
+    }
     // skip rest of the frameheader.
     buf = &buf[12];
     buf_size -= 12;
@@ -185,18 +195,19 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *data_size,
         buf_size -= 12;
     }
 
-    if (c->pic.data[0])
+    if (keyframe && c->pic.data[0])
         avctx->release_buffer(avctx, &c->pic);
     c->pic.reference = 1;
     c->pic.buffer_hints = FF_BUFFER_HINTS_VALID | FF_BUFFER_HINTS_READABLE |
                           FF_BUFFER_HINTS_PRESERVE | FF_BUFFER_HINTS_REUSABLE;
-    if (avctx->get_buffer(avctx, &c->pic) < 0) {
+    result = keyframe ? avctx->get_buffer(avctx, &c->pic) : avctx->reget_buffer(avctx, &c->pic);
+    if (result < 0) {
         av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
         return -1;
     }
 
-    c->pic.pict_type = FF_I_TYPE;
-    c->pic.key_frame = 1;
+    c->pic.pict_type = keyframe ? FF_I_TYPE : FF_P_TYPE;
+    c->pic.key_frame = keyframe;
     // decompress/copy/whatever data
     switch (comptype) {
         case NUV_LZO:
@@ -221,8 +232,6 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *data_size,
             break;
         }
         case NUV_COPY_LAST: {
-            c->pic.pict_type = FF_P_TYPE;
-            c->pic.key_frame = 0;
             /* nothing more to do here */
             break;
         }
@@ -271,6 +280,6 @@ AVCodec nuv_decoder = {
     decode_end,
     decode_frame,
     CODEC_CAP_DR1,
-    .long_name = "NuppelVideo",
+    .long_name = NULL_IF_CONFIG_SMALL("NuppelVideo"),
 };