e_comp_canvas: changed default value to E_LAYER_COUNT-1 in e_comp_canvas_layer_map. 17/107417/4 accepted/tizen/3.0.m2/mobile/20170104.143053 accepted/tizen/3.0.m2/tv/20170104.143517 accepted/tizen/3.0.m2/wearable/20170104.143915 accepted/tizen/3.0/common/20161229.112640 accepted/tizen/3.0/ivi/20161229.053957 accepted/tizen/3.0/mobile/20161229.053902 accepted/tizen/3.0/tv/20161229.053916 accepted/tizen/3.0/wearable/20161229.053937 accepted/tizen/common/20161228.162513 accepted/tizen/ivi/20161229.112524 accepted/tizen/mobile/20161229.112354 accepted/tizen/tv/20161229.112442 accepted/tizen/wearable/20161229.112503 submit/tizen/20161228.050541 submit/tizen_3.0.m2/20170104.093752 submit/tizen_3.0/20161228.050712
authorDoyoun Kang <doyoun.kang@samsung.com>
Wed, 28 Dec 2016 04:17:05 +0000 (13:17 +0900)
committerGwanglim Lee <gl77.lee@samsung.com>
Wed, 28 Dec 2016 04:45:39 +0000 (20:45 -0800)
This change prevents to access the e_comp->layers list out of bounds.

Change-Id: I7e7ca5938ea23cd8b515f0ccf113052cb8a0bd00

src/bin/e_client.c
src/bin/e_comp_canvas.c

index 756f3cc3202e556522e8c1fa287cda3ff1c97b80..e9adf3c4e41fa4533a7b1eb64f4bf1e5b9b20e1e 100644 (file)
@@ -4369,8 +4369,10 @@ e_client_below_get(const E_Client *ec)
           }
      }
 
+   if (ec->layer == E_LAYER_CLIENT_DESKTOP) return NULL;
+   if (e_comp_canvas_client_layer_map(ec->layer) == 9999) return NULL;
+
    /* go down the layers until we find one */
-   if (e_comp_canvas_layer_map(ec->layer) > e_comp_canvas_layer_map(E_LAYER_MAX)) return NULL;
    x = e_comp_canvas_layer_map(ec->layer);
    if (x > 0) x--;
 
index 01249aafd509d16ccc46c90a0097e3be0fe2766d..4068fa99b6308113f65d27337098f4cb3db30567 100644 (file)
@@ -604,7 +604,7 @@ e_comp_canvas_layer_map(E_Layer layer)
 
    /* prevents to access the e_comp->layers list out of bounds. */
    ELOGF("WARNING", "Used not defined layer:%d", NULL, NULL, layer);
-   return E_LAYER_MAX;
+   return E_LAYER_COUNT - 1;
 }
 
 E_API unsigned int