diff options
| author | Seunghun Lee <shiin.lee@samsung.com> | 2014-11-21 09:18:29 -0500 |
|---|---|---|
| committer | Chris Michael <cp.michael@samsung.com> | 2014-11-21 09:18:36 -0500 |
| commit | b2b81f2cd49b65d9567e4bfe27fb173f8216d7bc (patch) | |
| tree | a529f9c1cb5561f5c537a6328c4dbd7a70001367 | |
| parent | 9c26b572d0ff21e371e72c43594529f49d4d7910 (diff) | |
| download | efl-b2b81f2cd49b65d9567e4bfe27fb173f8216d7bc.tar.gz | |
ecore-drm: fix error check of _device_flags_set function
Summary: the function return -1 if it's failed so checking for ! is
incorrect. We should check if the return is less than zero.
@fix
Reviewers: gwanglim, devilhorns
Reviewed By: devilhorns
Subscribers: cedric
Differential Revision: https://phab.enlightenment.org/D1682
| -rw-r--r-- | src/lib/ecore_drm/ecore_drm_inputs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/ecore_drm/ecore_drm_inputs.c b/src/lib/ecore_drm/ecore_drm_inputs.c index 98ca4262f3..9a1dcdf07b 100644 --- a/src/lib/ecore_drm/ecore_drm_inputs.c +++ b/src/lib/ecore_drm/ecore_drm_inputs.c @@ -97,7 +97,7 @@ _cb_device_opened(void *data, const Eldbus_Message *msg, Eldbus_Pending *pending goto cleanup; } - if (!(fd = _device_flags_set(fd))) + if ((fd = _device_flags_set(fd)) < 0) { ERR("\tCould not set fd flags"); goto release; |