devilhorns pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=39db828aade3021cddb89b7d7202d7e8ea288d9f

commit 39db828aade3021cddb89b7d7202d7e8ea288d9f
Author: Chris Michael <[email protected]>
Date:   Tue Sep 23 15:09:47 2014 -0400

    ecore-evas-drm: Fix calling ecore_drm_device_find twice in some cases
    
    Summary: If no device name was passed in to ecore_drm_device_find
    function, then if we fail to find the default drm device on the first
    call, there is No point in calling the same function again to get the
    same result.
    
    @fix
    
    Signed-off-by: Chris Michael <[email protected]>
---
 src/modules/ecore_evas/engines/drm/ecore_evas_drm.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/modules/ecore_evas/engines/drm/ecore_evas_drm.c 
b/src/modules/ecore_evas/engines/drm/ecore_evas_drm.c
index 3376485..12acf35 100644
--- a/src/modules/ecore_evas/engines/drm/ecore_evas_drm.c
+++ b/src/modules/ecore_evas/engines/drm/ecore_evas_drm.c
@@ -457,6 +457,10 @@ _ecore_evas_drm_init(const char *device)
      {
         ERR("Could not find drm device with name: %s. Falling back to default 
device.", device);
 
+        /* if we already passed in NULL as device name, then no point in 
+         * calling the find function below with no name either */
+        if (!device) goto dev_err;
+
         /* try getting the default drm device */
         if (!(dev = ecore_drm_device_find(NULL, NULL)))
           goto dev_err;

-- 


Reply via email to