devilhorns pushed a commit to branch master.
http://git.enlightenment.org/core/efl.git/commit/?id=b36833f49b87b985d34ed1f52ed33a0192a53237
commit b36833f49b87b985d34ed1f52ed33a0192a53237
Author: Christopher Michael <cp.mich...@samsung.com>
Date: Wed Apr 24 08:52:02 2019 -0400
ecore-evas-x: Fix double free
Coverity reports, and reading the code confirms, that the variable
'out' has already been freed (or is NULL) when reaching this line of
execution.
Fixes CID1382851
@fix
---
src/modules/ecore_evas/engines/x/ecore_evas_x.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/modules/ecore_evas/engines/x/ecore_evas_x.c
b/src/modules/ecore_evas/engines/x/ecore_evas_x.c
index d8f8540b19..8c12aa44a3 100644
--- a/src/modules/ecore_evas/engines/x/ecore_evas_x.c
+++ b/src/modules/ecore_evas/engines/x/ecore_evas_x.c
@@ -3528,7 +3528,6 @@ _ecore_evas_x_screen_geometry_get(const Ecore_Evas *ee,
int *x, int *y, int *w,
if (!out)
{
norandr:
- if (out) free(out);
if (x) *x = 0;
if (y) *y = 0;
ecore_x_window_size_get(root, w, h);
--
