diff options
| author | Seunghun Lee <shiin.lee@samsung.com> | 2015-06-02 14:42:33 +0900 |
|---|---|---|
| committer | Seunghun Lee <shiin.lee@samsung.com> | 2015-06-03 18:05:15 -0700 |
| commit | a0912ef5b73c74f3277951ef91549d287ab45fe8 (patch) | |
| tree | 6cf97eb5706d4ec8ad5420fca58033d10e4fa59d | |
| parent | 742334b2e03482e9819bc247ec3ec1e0534feb3b (diff) | |
| download | efl-a0912ef5b73c74f3277951ef91549d287ab45fe8.tar.gz | |
ecore-buffer: fix wrong check for failure case
Change-Id: I3e59d41166b4c8180f643c7b825836186ed469a6
| -rw-r--r-- | src/lib/ecore_buffer/ecore_buffer.c | 5 | ||||
| -rw-r--r-- | src/modules/ecore_buffer/x11_dri2/ecore_buffer_x11_dri2.c | 2 |
2 files changed, 3 insertions, 4 deletions
diff --git a/src/lib/ecore_buffer/ecore_buffer.c b/src/lib/ecore_buffer/ecore_buffer.c index 818dfa2d6f..84564017c7 100644 --- a/src/lib/ecore_buffer/ecore_buffer.c +++ b/src/lib/ecore_buffer/ecore_buffer.c @@ -86,11 +86,10 @@ _ecore_buffer_get_backend(const char *name) else bm = eina_hash_find(_backends, backend_name); - if ((!bm) || (!bm->be)) + if ((!bm) || (!bm->be) || (!bm->be->init)) return NULL; - if ((!bm->data) && (bm->be->init)) - bm->data = bm->be->init(NULL, NULL); + bm->data = bm->be->init(NULL, NULL); return bm; } diff --git a/src/modules/ecore_buffer/x11_dri2/ecore_buffer_x11_dri2.c b/src/modules/ecore_buffer/x11_dri2/ecore_buffer_x11_dri2.c index 5ce3c4918a..e51d9edddb 100644 --- a/src/modules/ecore_buffer/x11_dri2/ecore_buffer_x11_dri2.c +++ b/src/modules/ecore_buffer/x11_dri2/ecore_buffer_x11_dri2.c @@ -253,7 +253,7 @@ _ecore_buffer_x11_dri2_init(const char *context EINA_UNUSED, const char *options if (fd < 0) goto on_error; - if (!(drmGetMagic(fd, &magic))) + if (drmGetMagic(fd, &magic) < 0) goto on_error; if (!(DRI2Authenticate(xdpy, root, magic))) |