From: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
Date: Tue, 20 Nov 2012 13:36:29 +0000 (+0100)
Subject: image: fix GstVaapiImage map and unmap.
X-Git-Tag: 1.19.3~503^2~2874
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c38c0853ebf2979f1321892d42217afb1b9b88fa;p=platform%2Fupstream%2Fgstreamer.git

image: fix GstVaapiImage map and unmap.

Fix gst_vaapi_image_map() to return TRUE and the GstVaapiImageRaw
structure correctly filled in if the image was already mapped.
Likewise, make gst_vaapi_image_unmap() return TRUE if the image
was already unmapped.
---

diff --git a/gst-libs/gst/vaapi/gstvaapiimage.c b/gst-libs/gst/vaapi/gstvaapiimage.c
index 0961234..65d79a5 100644
--- a/gst-libs/gst/vaapi/gstvaapiimage.c
+++ b/gst-libs/gst/vaapi/gstvaapiimage.c
@@ -753,12 +753,11 @@ _gst_vaapi_image_map(GstVaapiImage *image, GstVaapiImageRaw *raw_image)
 {
     GstVaapiImagePrivate * const priv = image->priv;
     GstVaapiDisplay *display;
-    void *image_data;
     VAStatus status;
     guint i;
 
     if (_gst_vaapi_image_is_mapped(image))
-        return TRUE;
+        goto map_success;
 
     display = GST_VAAPI_OBJECT_DISPLAY(image);
     if (!display)
@@ -767,15 +766,14 @@ _gst_vaapi_image_map(GstVaapiImage *image, GstVaapiImageRaw *raw_image)
     GST_VAAPI_DISPLAY_LOCK(display);
     status = vaMapBuffer(
         GST_VAAPI_DISPLAY_VADISPLAY(display),
-        image->priv->image.buf,
-        &image_data
+        priv->image.buf,
+        (void **)&priv->image_data
     );
     GST_VAAPI_DISPLAY_UNLOCK(display);
     if (!vaapi_check_status(status, "vaMapBuffer()"))
         return FALSE;
 
-    image->priv->image_data = image_data;
-
+map_success:
     if (raw_image) {
         const VAImage * const va_image = &priv->image;
         raw_image->format     = priv->format;
@@ -783,7 +781,8 @@ _gst_vaapi_image_map(GstVaapiImage *image, GstVaapiImageRaw *raw_image)
         raw_image->height     = va_image->height;
         raw_image->num_planes = va_image->num_planes;
         for (i = 0; i < raw_image->num_planes; i++) {
-            raw_image->pixels[i] = (guchar *)image_data + va_image->offsets[i];
+            raw_image->pixels[i] = (guchar *)priv->image_data +
+                va_image->offsets[i];
             raw_image->stride[i] = va_image->pitches[i];
         }
     }
@@ -815,7 +814,7 @@ _gst_vaapi_image_unmap(GstVaapiImage *image)
     VAStatus status;
 
     if (!_gst_vaapi_image_is_mapped(image))
-        return FALSE;
+        return TRUE;
 
     display = GST_VAAPI_OBJECT_DISPLAY(image);
     if (!display)