evas_device: Fix dereferencing null pointer

Coverity reports that 'pos' could be null here and we are potentially
dereferencing a NULL pointer, so lets add a check for 'pos' here
before trying to use it.

Fixes Coverity CID1399091

@fix

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8313

diff --git a/src/lib/evas/canvas/evas_device.c b/src/lib/evas/canvas/evas_device.c
index 2e5e141b7639e1d4ebf5702c8993a97018f5d4e0..c07748ea442408974b2a7448d7df1fa14cb0c00d 100644 (file)
--- a/src/lib/evas/canvas/evas_device.c
+++ b/src/lib/evas/canvas/evas_device.c
@@ -538,7 +538,9 @@ _evas_canvas_efl_canvas_scene_pointer_position_get(const Eo *eo_e, Evas_Public_D
    Eina_Iterator *it;
    Eo *child;
 
-   if (pos) *pos = EINA_POSITION2D(0, 0);
+   if (!pos) return EINA_FALSE;
+
+   *pos = EINA_POSITION2D(0, 0);
    if (!e->default_seat) return EINA_FALSE;
    if (!seat)
      {
@@ -553,6 +555,7 @@ _evas_canvas_efl_canvas_scene_pointer_position_get(const Eo *eo_e, Evas_Public_D
        break;
    if (child)
      *pos = efl_input_pointer_position_get(child);
+
    eina_iterator_free(it);
    return !!child;
 }