fileselector: add NULL check
authorJihoon Kim <jihoon48.kim@samsung.com>
Tue, 31 Dec 2013 02:22:20 +0000 (11:22 +0900)
committerDaniel Juyung Seo <juyung.seo@samsung.com>
Tue, 31 Dec 2013 04:23:30 +0000 (13:23 +0900)
src/lib/elc_fileselector.c

index e6a9b8e..6e4635a 100644 (file)
@@ -1366,7 +1366,7 @@ _is_save_get(Eo *obj EINA_UNUSED, void *_pd, va_list *list)
 {
    Eina_Bool *ret = va_arg(*list, Eina_Bool *);
    Elm_Fileselector_Smart_Data *sd = _pd;
-   *ret = !elm_object_disabled_get(sd->name_entry);
+   if (ret) *ret = !elm_object_disabled_get(sd->name_entry);
 }
 
 EAPI void
@@ -1403,7 +1403,7 @@ _folder_only_get(Eo *obj EINA_UNUSED, void *_pd, va_list *list)
 {
    Eina_Bool *ret = va_arg(*list, Eina_Bool *);
    Elm_Fileselector_Smart_Data *sd = _pd;
-   *ret = sd->only_folder;
+   if (ret) *ret = sd->only_folder;
 }
 
 EAPI void
@@ -1468,7 +1468,7 @@ _buttons_ok_cancel_get(Eo *obj EINA_UNUSED, void *_pd, va_list *list)
 {
    Eina_Bool *ret = va_arg(*list, Eina_Bool *);
    Elm_Fileselector_Smart_Data *sd = _pd;
-   *ret = sd->ok_button ? EINA_TRUE : EINA_FALSE;
+   if (ret) *ret = sd->ok_button ? EINA_TRUE : EINA_FALSE;
 }
 
 EAPI void
@@ -1504,7 +1504,7 @@ _expandable_get(Eo *obj EINA_UNUSED, void *_pd, va_list *list)
 {
    Eina_Bool *ret = va_arg(*list, Eina_Bool *);
    Elm_Fileselector_Smart_Data *sd = _pd;
-   *ret = sd->expand;
+   if (ret) *ret = sd->expand;
 }
 
 EAPI void
@@ -1540,7 +1540,7 @@ _path_get(Eo *obj EINA_UNUSED, void *_pd, va_list *list)
 {
    const char **ret = va_arg(*list, const char **);
    Elm_Fileselector_Smart_Data *sd = _pd;
-   *ret = sd->path;
+   if (ret) *ret = sd->path;
 }
 
 EAPI void
@@ -1599,7 +1599,7 @@ _mode_get(Eo *obj EINA_UNUSED, void *_pd, va_list *list)
 {
    Elm_Fileselector_Mode *ret = va_arg(*list, Elm_Fileselector_Mode *);
    Elm_Fileselector_Smart_Data *sd = _pd;
-   *ret = sd->mode;
+   if (ret) *ret = sd->mode;
 }
 
 EAPI void
@@ -1665,7 +1665,7 @@ _multi_select_get(Eo *obj EINA_UNUSED, void *_pd, va_list *list)
    Eina_Bool *ret = va_arg(*list, Eina_Bool *);
    Elm_Fileselector_Smart_Data *sd = _pd;
 
-   *ret = sd->multi;
+   if (ret) *ret = sd->multi;
 }
 
 EAPI const char *
@@ -1696,19 +1696,19 @@ _selected_get(Eo *obj EINA_UNUSED, void *_pd, va_list *list)
         return;
      }
 
-   *ret = sd->path;
+   if (ret) *ret = sd->path;
 
    if (sd->mode == ELM_FILESELECTOR_LIST)
      {
         Elm_Object_Item *gl_it = elm_genlist_selected_item_get(sd->files_view);
 
-        if (gl_it) *ret = elm_object_item_data_get(gl_it);
+        if (gl_it && ret) *ret = elm_object_item_data_get(gl_it);
      }
    else
      {
         Elm_Object_Item *gg_it = elm_gengrid_selected_item_get(sd->files_view);
 
-        if (gg_it) *ret = elm_object_item_data_get(gg_it);
+        if (gg_it && ret) *ret = elm_object_item_data_get(gg_it);
      }
 
 }
@@ -1902,21 +1902,21 @@ _hidden_visible_get(Eo *obj EINA_UNUSED, void *_pd, va_list *list EINA_UNUSED)
    Eina_Bool *ret = va_arg(*list, Eina_Bool *);
    Elm_Fileselector_Smart_Data *sd = _pd;
 
-   *ret = sd->hidden_visible;
+   if (ret) *ret = sd->hidden_visible;
 }
 
 static void
 _elm_fileselector_smart_focus_next_manager_is(Eo *obj EINA_UNUSED, void *_pd EINA_UNUSED, va_list *list)
 {
    Eina_Bool *ret = va_arg(*list, Eina_Bool *);
-   *ret = EINA_FALSE;
+   if (ret) *ret = EINA_FALSE;
 }
 
 static void
 _elm_fileselector_smart_focus_direction_manager_is(Eo *obj EINA_UNUSED, void *_pd EINA_UNUSED, va_list *list)
 {
    Eina_Bool *ret = va_arg(*list, Eina_Bool *);
-   *ret = EINA_FALSE;
+   if (ret) *ret = EINA_FALSE;
 }
 
 static void