From: Gwanglim Lee <gl77.lee@samsung.com>
Date: Thu, 18 May 2017 04:55:23 +0000 (+0900)
Subject: e_comp_wl_rsm: changed to use EINA_SAFETY for checking validation of tizen remote... 
X-Git-Tag: accepted/tizen/unified/20170531.082639~13
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=68671190e4f26f1376e7d77b7d07f44b02812f5c;p=platform%2Fupstream%2Fenlightenment.git

e_comp_wl_rsm: changed to use EINA_SAFETY for checking validation of tizen remote surface requests

Change-Id: I22eeb3b37460712162535ecd758a38326770cd3b
---

diff --git a/src/bin/e_comp_wl_rsm.c b/src/bin/e_comp_wl_rsm.c
index 0642260..bee8ae3 100644
--- a/src/bin/e_comp_wl_rsm.c
+++ b/src/bin/e_comp_wl_rsm.c
@@ -1158,8 +1158,8 @@ _remote_surface_cb_redirect(struct wl_client *client, struct wl_resource *resour
    EINA_SAFETY_ON_NULL_RETURN(_rsm);
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
 
    if (remote_surface->provider)
      {
@@ -1249,8 +1249,8 @@ _remote_surface_cb_unredirect(struct wl_client *client, struct wl_resource *reso
    E_Comp_Wl_Remote_Surface *remote_surface;
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
 
    remote_surface->redirect = EINA_FALSE;
 //   _remote_surface_visible_set(remote_surface, EINA_FALSE);
@@ -1273,10 +1273,10 @@ _remote_surface_cb_mouse_event_transfer(struct wl_client *client, struct wl_reso
    double eradx, erady, epressure, eangle;
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->provider) return;
-   if (!remote_surface->provider->ec) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface->provider);
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface->provider->ec);
 
    provider = remote_surface->provider;
    ec = provider->ec;
@@ -1401,10 +1401,10 @@ _remote_surface_cb_mouse_wheel_transfer(struct wl_client *client, struct wl_reso
    Ecore_Device *edev = NULL;
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->provider) return;
-   if (!remote_surface->provider->ec) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface->provider);
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface->provider->ec);
 
    provider = remote_surface->provider;
    ec = provider->ec;
@@ -1429,10 +1429,10 @@ _remote_surface_cb_touch_event_transfer(struct wl_client *client, struct wl_reso
    double eradx, erady, epressure, eangle;
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->provider) return;
-   if (!remote_surface->provider->ec) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface->provider);
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface->provider->ec);
 
    provider = remote_surface->provider;
    ec = provider->ec;
@@ -1513,10 +1513,10 @@ _remote_surface_cb_touch_cancel_transfer(struct wl_client *client, struct wl_res
    E_Client *ec;
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->provider) return;
-   if (!remote_surface->provider->ec) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface->provider);
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface->provider->ec);
 
    provider = remote_surface->provider;
    ec = provider->ec;
@@ -1536,10 +1536,10 @@ _remote_surface_cb_key_event_transfer(struct wl_client *client, struct wl_resour
    Ecore_Device_Class eclas;
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->provider) return;
-   if (!remote_surface->provider->ec) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface->provider);
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface->provider->ec);
 
    provider = remote_surface->provider;
    ec = provider->ec;
@@ -1594,8 +1594,8 @@ _remote_surface_cb_visibility_transfer(struct wl_client *client, struct wl_resou
    E_Comp_Wl_Remote_Surface *remote_surface;
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
 
    if (visibility_type == TIZEN_REMOTE_SURFACE_VISIBILITY_TYPE_INVISIBLE)
      {
@@ -1614,8 +1614,8 @@ _remote_surface_cb_owner_set(struct wl_client *client, struct wl_resource *resou
    E_Client *owner = NULL;
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
 
    if (surface_resource)
      owner = wl_resource_get_user_data(surface_resource);
@@ -1645,8 +1645,8 @@ _remote_surface_cb_region_create(struct wl_client *client, struct wl_resource *r
    E_Comp_Wl_Remote_Provider *provider;
 
    remote_surface = wl_resource_get_user_data(remote_surface_resource);
-   if (!remote_surface) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
 
    resource = wl_resource_create(client,
                                  &tizen_remote_surface_region_interface,
@@ -1688,8 +1688,8 @@ _remote_surface_cb_release(struct wl_client *client, struct wl_resource *resourc
    E_Comp_Wl_Remote_Buffer *remote_buffer;
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
 
    remote_buffer = _e_comp_wl_remote_buffer_get(remote_buffer_resource);
    EINA_SAFETY_ON_NULL_RETURN(remote_buffer);
@@ -1705,8 +1705,8 @@ _remote_surface_cb_remote_render_set(struct wl_client *client, struct wl_resourc
    E_Comp_Wl_Remote_Source *source = NULL;
 
    remote_surface = wl_resource_get_user_data(resource);
-   if (!remote_surface) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
 
    source = remote_surface->source;
    if (!source) return;
@@ -1897,8 +1897,8 @@ _remote_manager_cb_surface_bind(struct wl_client *client, struct wl_resource *re
    E_Client *ec = NULL;
 
    remote_surface = wl_resource_get_user_data(remote_surface_resource);
-   if (!remote_surface) return;
-   if (!remote_surface->valid) return;
+   EINA_SAFETY_ON_NULL_RETURN(remote_surface);
+   EINA_SAFETY_ON_FALSE_RETURN(remote_surface->valid);
 
    provider = remote_surface->provider;
    if (!provider) return;