From: Gwenole Beauchesne <gbeauchesne@splitted-desktop.com>
Date: Wed, 8 Jul 2009 07:50:40 +0000 (+0000)
Subject: Fix va_DisplayContextGetDriverName() to check for LIBVA_DRIVER_NAME first.
X-Git-Tag: 20091130_Alpha4~46^2~15
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c00a20b3c19b90f7f9fbd4007e02ba4ce20e975b;p=profile%2Fivi%2Flibva.git

Fix va_DisplayContextGetDriverName() to check for LIBVA_DRIVER_NAME first.

Signed-off-by: Austin Yuan <shengquan.yuan@intel.com>
---

diff --git a/src/x11/va_x11.c b/src/x11/va_x11.c
index 1e53401..cfb88be 100644
--- a/src/x11/va_x11.c
+++ b/src/x11/va_x11.c
@@ -206,30 +206,23 @@ static VAStatus va_DisplayContextGetDriverName (
     int driver_minor;
     int driver_patch;
     Bool result = True;
-    char *x_driver_name = NULL;
+    char *driver_name_env;
 
     if (driver_name)
 	*driver_name = NULL;
-    
-    vaStatus = va_DRI2GetDriverName(pDisplayContext, driver_name);
-    if (vaStatus != VA_STATUS_SUCCESS)
-        vaStatus = va_DRIGetDriverName(pDisplayContext, driver_name);
 
-    if ((vaStatus == VA_STATUS_SUCCESS)
+    if ((driver_name_env = getenv("LIBVA_DRIVER_NAME")) != NULL
         && geteuid() == getuid())
     {
         /* don't allow setuid apps to use LIBVA_DRIVER_NAME */
-        if (getenv("LIBVA_DRIVER_NAME"))
-        {
-            /* For easier debugging */
-            if (*driver_name)
-                XFree(*driver_name);
-            
-            *driver_name = strdup(getenv("LIBVA_DRIVER_NAME"));
-            return VA_STATUS_SUCCESS;
-        }
+        *driver_name = strdup(driver_name_env);
+        return VA_STATUS_SUCCESS;
     }
-    
+
+    vaStatus = va_DRI2GetDriverName(pDisplayContext, driver_name);
+    if (vaStatus != VA_STATUS_SUCCESS)
+        vaStatus = va_DRIGetDriverName(pDisplayContext, driver_name);
+   
     return vaStatus;
 }