svace fix

Change-Id: I61776af7c381137efaa752ca0d3ba5937929ba26
Signed-off-by: pr.jung <pr.jung@samsung.com>

diff --git a/apps/extended-sd/src/es-internal-storage-page.c b/apps/extended-sd/src/es-internal-storage-page.c
index 17d53c35d87da0d8320b8e6d6a68a1ffde1e2a56..e0dc3e7747724884e10b6cb95ea09fa9d5f97747 100644 (file)
--- a/apps/extended-sd/src/es-internal-storage-page.c
+++ b/apps/extended-sd/src/es-internal-storage-page.c
@@ -463,10 +463,12 @@ static Evas_Object* create_page_1(appdata_s* ad)
        Evas_Object* page_content = NULL;
        char *str_back;
        char *str_format;
-
-       page_content = create_layout(ad->internal_storage_page_data->internal_storage_page_base_layout, "page_with_2_bottom_button");
        Elm_Genlist_Item_Class *extended_multiline_genlist_itc;
        genlist_item_s *item;
+       Evas_Object* genlist;
+       Evas_Object* back_button;
+       Evas_Object* format_button;
+       page_content = create_layout(ad->internal_storage_page_data->internal_storage_page_base_layout, "page_with_2_bottom_button");
 
        extended_multiline_genlist_itc = elm_genlist_item_class_new();
        extended_multiline_genlist_itc->item_style = "extended-multiline";
@@ -475,7 +477,7 @@ static Evas_Object* create_page_1(appdata_s* ad)
        extended_multiline_genlist_itc->func.state_get = NULL;
        extended_multiline_genlist_itc->func.del = _internal_storage_page_genlist_del;
 
-       Evas_Object* genlist = elm_genlist_add(page_content);
+       genlist = elm_genlist_add(page_content);
        elm_genlist_mode_set(genlist, ELM_LIST_COMPRESS);
        elm_object_part_content_set(page_content, "elm.swallow.content", genlist);
 
@@ -519,21 +521,20 @@ static Evas_Object* create_page_1(appdata_s* ad)
        item->it = elm_genlist_item_append(genlist, extended_multiline_genlist_itc, item, NULL, ELM_GENLIST_ITEM_NONE, NULL, item);
        elm_genlist_item_select_mode_set(item->it, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
-       elm_genlist_item_class_free(extended_multiline_genlist_itc);
-
        str_back = dgettext(LANG_DOMAIN, "IDS_ST_BUTTON_BACK");
-       Evas_Object* back_button = create_bottom_button(page_content);
+       back_button = create_bottom_button(page_content);
        elm_object_text_set(back_button, str_back);
        elm_object_part_content_set(page_content, "bottom_button_1", back_button);
        evas_object_smart_callback_add(back_button, "clicked", navi_back_cb, ad);
 
        str_format = dgettext(LANG_DOMAIN, "IDS_ST_BUTTON_FORMAT_ABB3");
-       Evas_Object* format_button = create_bottom_button(page_content);
+       format_button = create_bottom_button(page_content);
        elm_object_text_set(format_button, str_format);
        elm_object_part_content_set(page_content, "bottom_button_2", format_button);
        evas_object_smart_callback_add(format_button, "clicked", _format_click_cb, ad);
 
 out:
+       elm_genlist_item_class_free(extended_multiline_genlist_itc);
        FUNC_END();
        return page_content;
 }
@@ -544,6 +545,7 @@ static Evas_Object* create_page_2(appdata_s* ad)
 
        Elm_Genlist_Item_Class *extended_multiline_genlist_itc;
        genlist_item_s *item;
+       Evas_Object* genlist;
 
        extended_multiline_genlist_itc = elm_genlist_item_class_new();
        extended_multiline_genlist_itc->item_style = "extended-multiline";
@@ -552,7 +554,7 @@ static Evas_Object* create_page_2(appdata_s* ad)
        extended_multiline_genlist_itc->func.state_get = NULL;
        extended_multiline_genlist_itc->func.del = _internal_storage_page_genlist_del;
 
-       Evas_Object* genlist = elm_genlist_add(ad->internal_storage_page_data->internal_storage_page_base_layout);
+       genlist = elm_genlist_add(ad->internal_storage_page_data->internal_storage_page_base_layout);
        elm_genlist_mode_set(genlist, ELM_LIST_COMPRESS);
 
        item = (genlist_item_s *)malloc(sizeof(genlist_item_s));
@@ -595,10 +597,8 @@ static Evas_Object* create_page_2(appdata_s* ad)
        item->it = elm_genlist_item_append(genlist, extended_multiline_genlist_itc, item, NULL, ELM_GENLIST_ITEM_NONE, NULL, item);
        elm_genlist_item_select_mode_set(item->it, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
-       elm_genlist_item_class_free(extended_multiline_genlist_itc);
-
-
 out:
+       elm_genlist_item_class_free(extended_multiline_genlist_itc);
        FUNC_END();
        return genlist;
 }
@@ -610,11 +610,14 @@ static Evas_Object* create_page_3(appdata_s* ad)
        Evas_Object* page_content = NULL;
        char *str_back;
        char *str_continue;
-
-       page_content = create_layout(ad->internal_storage_page_data->internal_storage_page_base_layout, "page_with_2_bottom_button");
+       Evas_Object* genlist;
+       Evas_Object* back_button;
+       Evas_Object* continue_button;
        Elm_Genlist_Item_Class *extended_multiline_genlist_itc;
        genlist_item_s *item;
 
+       page_content = create_layout(ad->internal_storage_page_data->internal_storage_page_base_layout, "page_with_2_bottom_button");
+
        extended_multiline_genlist_itc = elm_genlist_item_class_new();
        extended_multiline_genlist_itc->item_style = "extended-multiline";
        extended_multiline_genlist_itc->func.text_get = _internal_storage_page_genlist_text_get;
@@ -622,7 +625,7 @@ static Evas_Object* create_page_3(appdata_s* ad)
        extended_multiline_genlist_itc->func.state_get = NULL;
        extended_multiline_genlist_itc->func.del = _internal_storage_page_genlist_del;
 
-       Evas_Object* genlist = elm_genlist_add(page_content);
+       genlist = elm_genlist_add(page_content);
        elm_genlist_mode_set(genlist, ELM_LIST_COMPRESS);
        elm_object_part_content_set(page_content, "elm.swallow.content", genlist);
 
@@ -656,21 +659,20 @@ static Evas_Object* create_page_3(appdata_s* ad)
        item->it = elm_genlist_item_append(genlist, extended_multiline_genlist_itc, item, NULL, ELM_GENLIST_ITEM_NONE, NULL, item);
        elm_genlist_item_select_mode_set(item->it, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
-       elm_genlist_item_class_free(extended_multiline_genlist_itc);
-
        str_back = dgettext(LANG_DOMAIN, "IDS_ST_BUTTON_BACK");
-       Evas_Object* back_button = create_bottom_button(page_content);
+       back_button = create_bottom_button(page_content);
        elm_object_text_set(back_button, str_back);
        elm_object_part_content_set(page_content, "bottom_button_1", back_button);
        evas_object_smart_callback_add(back_button, "clicked", navi_back_cb, ad);
 
        str_continue = dgettext(LANG_DOMAIN, "IDS_ST_BUTTON_CONTINUE");
-       Evas_Object* continue_button = create_bottom_button(page_content);
+       continue_button = create_bottom_button(page_content);
        elm_object_text_set(continue_button, str_continue);
        elm_object_part_content_set(page_content, "bottom_button_2", continue_button);
        evas_object_smart_callback_add(continue_button, "clicked", _continue_click_cb, ad);
 
 out:
+       elm_genlist_item_class_free(extended_multiline_genlist_itc);
        FUNC_END();
        return page_content;
 }
@@ -681,6 +683,7 @@ static Evas_Object* create_page_4(appdata_s* ad)
 
        Elm_Genlist_Item_Class *extended_multiline_genlist_itc;
        genlist_item_s *item;
+       Evas_Object* genlist;
 
        extended_multiline_genlist_itc = elm_genlist_item_class_new();
        extended_multiline_genlist_itc->item_style = "extended-multiline";
@@ -689,7 +692,7 @@ static Evas_Object* create_page_4(appdata_s* ad)
        extended_multiline_genlist_itc->func.state_get = NULL;
        extended_multiline_genlist_itc->func.del = _internal_storage_page_genlist_del;
 
-       Evas_Object* genlist = elm_genlist_add(ad->internal_storage_page_data->internal_storage_page_base_layout);
+       genlist = elm_genlist_add(ad->internal_storage_page_data->internal_storage_page_base_layout);
        elm_genlist_mode_set(genlist, ELM_LIST_COMPRESS);
 
        item = (genlist_item_s *)malloc(sizeof(genlist_item_s));
@@ -722,9 +725,8 @@ static Evas_Object* create_page_4(appdata_s* ad)
        item->it = elm_genlist_item_append(genlist, extended_multiline_genlist_itc, item, NULL, ELM_GENLIST_ITEM_NONE, NULL, item);
        elm_genlist_item_select_mode_set(item->it, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
-       elm_genlist_item_class_free(extended_multiline_genlist_itc);
-
 out:
+       elm_genlist_item_class_free(extended_multiline_genlist_itc);
        FUNC_END();
        return genlist;
 }
@@ -735,10 +737,11 @@ static Evas_Object* create_page_5(appdata_s* ad)
 
        Evas_Object* page_content = NULL;
        char *str_done;
-
-       page_content = create_layout(ad->internal_storage_page_data->internal_storage_page_base_layout, "page_with_1_bottom_button");
+       Evas_Object* genlist;
        Elm_Genlist_Item_Class *extended_multiline_genlist_itc;
        genlist_item_s *item;
+       Evas_Object* done_button;
+       page_content = create_layout(ad->internal_storage_page_data->internal_storage_page_base_layout, "page_with_1_bottom_button");
 
        extended_multiline_genlist_itc = elm_genlist_item_class_new();
        extended_multiline_genlist_itc->item_style = "extended-multiline";
@@ -747,7 +750,7 @@ static Evas_Object* create_page_5(appdata_s* ad)
        extended_multiline_genlist_itc->func.state_get = NULL;
        extended_multiline_genlist_itc->func.del = _internal_storage_page_genlist_del;
 
-       Evas_Object* genlist = elm_genlist_add(page_content);
+       genlist = elm_genlist_add(page_content);
        elm_genlist_mode_set(genlist, ELM_LIST_COMPRESS);
        elm_object_part_content_set(page_content, "elm.swallow.content", genlist);
 
@@ -811,15 +814,14 @@ static Evas_Object* create_page_5(appdata_s* ad)
        item->it = elm_genlist_item_append(genlist, extended_multiline_genlist_itc, item, NULL, ELM_GENLIST_ITEM_NONE, NULL, item);
        elm_genlist_item_select_mode_set(item->it, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
-       elm_genlist_item_class_free(extended_multiline_genlist_itc);
-
        str_done = dgettext(LANG_DOMAIN, "IDS_ST_BUTTON_DONE");
-       Evas_Object* done_button = create_bottom_button(page_content);
+       done_button = create_bottom_button(page_content);
        elm_object_text_set(done_button, str_done);
        elm_object_part_content_set(page_content, "bottom_button_1", done_button);
        evas_object_smart_callback_add(done_button, "clicked", navi_done_cb, ad);
 
 out:
+       elm_genlist_item_class_free(extended_multiline_genlist_itc);
        FUNC_END();
        return page_content;
 }
@@ -842,7 +844,7 @@ void create_internal_storage_page_base_layout(appdata_s *ad)
                ad->internal_storage_page_data = NULL;
        }
 
-       internal_storage_page_data_s *internal_storage_page_data = (internal_storage_page_data_s *)malloc(sizeof(internal_storage_page_data));
+       internal_storage_page_data_s *internal_storage_page_data = (internal_storage_page_data_s *)malloc(sizeof(internal_storage_page_data_s));
        if (!internal_storage_page_data) {
                DMSG_ERR("malloc failed");
                FUNC_END();

diff --git a/apps/extended-sd/src/es-portable-storage-page.c b/apps/extended-sd/src/es-portable-storage-page.c
index 509bfa6474fbff27d03fa2deeb7ef528cb4676de..bbb3065083223c35d5163edae76d0500fd0cb509 100644 (file)
--- a/apps/extended-sd/src/es-portable-storage-page.c
+++ b/apps/extended-sd/src/es-portable-storage-page.c
@@ -412,13 +412,15 @@ static Evas_Object* create_page_1(appdata_s* ad)
        FUNC_BEGIN();
 
        Evas_Object* page_content = NULL;
-
-       page_content = create_layout(ad->portable_storage_page_data->portable_storage_page_base_layout, "page_with_2_bottom_button");
        Elm_Genlist_Item_Class *extended_multiline_genlist_itc;
        genlist_item_s *item;
-
        char *str_back;
        char *str_format;
+       Evas_Object* genlist;
+       Evas_Object* back_button;
+       Evas_Object* format_button;
+
+       page_content = create_layout(ad->portable_storage_page_data->portable_storage_page_base_layout, "page_with_2_bottom_button");
 
        extended_multiline_genlist_itc = elm_genlist_item_class_new();
        extended_multiline_genlist_itc->item_style = "extended-multiline";
@@ -427,7 +429,7 @@ static Evas_Object* create_page_1(appdata_s* ad)
        extended_multiline_genlist_itc->func.state_get = NULL;
        extended_multiline_genlist_itc->func.del = _portable_storage_page_genlist_del;
 
-       Evas_Object* genlist = elm_genlist_add(page_content);
+       genlist = elm_genlist_add(page_content);
        elm_genlist_mode_set(genlist, ELM_LIST_COMPRESS);
        elm_object_part_content_set(page_content, "elm.swallow.content", genlist);
 
@@ -471,21 +473,20 @@ static Evas_Object* create_page_1(appdata_s* ad)
        item->it = elm_genlist_item_append(genlist, extended_multiline_genlist_itc, item, NULL, ELM_GENLIST_ITEM_NONE, NULL, item);
        elm_genlist_item_select_mode_set(item->it, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
-       elm_genlist_item_class_free(extended_multiline_genlist_itc);
-
-       Evas_Object* back_button = create_bottom_button(page_content);
+       back_button = create_bottom_button(page_content);
        str_back = dgettext(LANG_DOMAIN, "IDS_ST_BUTTON_BACK");
        elm_object_text_set(back_button, str_back);
        elm_object_part_content_set(page_content, "bottom_button_1", back_button);
        evas_object_smart_callback_add(back_button, "clicked", navi_back_cb, ad);
 
-       Evas_Object* format_button = create_bottom_button(page_content);
+       format_button = create_bottom_button(page_content);
        str_format = dgettext(LANG_DOMAIN, "IDS_ST_BUTTON_FORMAT_ABB3");
        elm_object_text_set(format_button, str_format);
        elm_object_part_content_set(page_content, "bottom_button_2", format_button);
        evas_object_smart_callback_add(format_button, "clicked", _format_click_cb, ad);
 
 out:
+       elm_genlist_item_class_free(extended_multiline_genlist_itc);
        FUNC_END();
        return page_content;
 }
@@ -495,13 +496,14 @@ static Evas_Object* create_page_3(appdata_s* ad)
        FUNC_BEGIN();
 
        Evas_Object* page_content = NULL;
-
-       page_content = create_layout(ad->portable_storage_page_data->portable_storage_page_base_layout, "page_with_1_bottom_button");
        Elm_Genlist_Item_Class *extended_multiline_genlist_itc;
        genlist_item_s *item;
-
+       Evas_Object* done_button;
+       Evas_Object* genlist;
        char *str_done;
 
+       page_content = create_layout(ad->portable_storage_page_data->portable_storage_page_base_layout, "page_with_1_bottom_button");
+
        extended_multiline_genlist_itc = elm_genlist_item_class_new();
        extended_multiline_genlist_itc->item_style = "extended-multiline";
        extended_multiline_genlist_itc->func.text_get = _portable_storage_page_genlist_text_get;
@@ -509,7 +511,7 @@ static Evas_Object* create_page_3(appdata_s* ad)
        extended_multiline_genlist_itc->func.state_get = NULL;
        extended_multiline_genlist_itc->func.del = _portable_storage_page_genlist_del;
 
-       Evas_Object* genlist = elm_genlist_add(page_content);
+       genlist = elm_genlist_add(page_content);
        elm_genlist_mode_set(genlist, ELM_LIST_COMPRESS);
        elm_object_part_content_set(page_content, "elm.swallow.content", genlist);
 
@@ -533,7 +535,6 @@ static Evas_Object* create_page_3(appdata_s* ad)
        item->it = elm_genlist_item_append(genlist, extended_multiline_genlist_itc, item, NULL, ELM_GENLIST_ITEM_NONE, NULL, item);
        elm_genlist_item_select_mode_set(item->it, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
-
        item = (genlist_item_s *)malloc(sizeof(genlist_item_s));
        if (!item) {
                DMSG_ERR("malloc failed");
@@ -554,15 +555,14 @@ static Evas_Object* create_page_3(appdata_s* ad)
        item->it = elm_genlist_item_append(genlist, extended_multiline_genlist_itc, item, NULL, ELM_GENLIST_ITEM_NONE, NULL, item);
        elm_genlist_item_select_mode_set(item->it, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
-       elm_genlist_item_class_free(extended_multiline_genlist_itc);
-
-       Evas_Object* done_button = create_bottom_button(page_content);
+       done_button = create_bottom_button(page_content);
        str_done = dgettext(LANG_DOMAIN, "IDS_ST_BUTTON_DONE");
        elm_object_text_set(done_button, str_done);
        elm_object_part_content_set(page_content, "bottom_button_1", done_button);
        evas_object_smart_callback_add(done_button, "clicked", navi_done_cb, ad);
 
 out:
+       elm_genlist_item_class_free(extended_multiline_genlist_itc);
        FUNC_END();
        return page_content;
 }
@@ -573,6 +573,7 @@ static Evas_Object* create_page_2(appdata_s* ad)
 
        Elm_Genlist_Item_Class *extended_multiline_genlist_itc;
        genlist_item_s *item;
+       Evas_Object* genlist;
 
        extended_multiline_genlist_itc = elm_genlist_item_class_new();
        extended_multiline_genlist_itc->item_style = "extended-multiline";
@@ -581,7 +582,7 @@ static Evas_Object* create_page_2(appdata_s* ad)
        extended_multiline_genlist_itc->func.state_get = NULL;
        extended_multiline_genlist_itc->func.del = _portable_storage_page_genlist_del;
 
-       Evas_Object* genlist = elm_genlist_add(ad->portable_storage_page_data->portable_storage_page_base_layout);
+       genlist = elm_genlist_add(ad->portable_storage_page_data->portable_storage_page_base_layout);
        elm_genlist_mode_set(genlist, ELM_LIST_COMPRESS);
 
        item = (genlist_item_s *)malloc(sizeof(genlist_item_s));
@@ -614,9 +615,8 @@ static Evas_Object* create_page_2(appdata_s* ad)
        item->it = elm_genlist_item_append(genlist, extended_multiline_genlist_itc, item, NULL, ELM_GENLIST_ITEM_NONE, NULL, item);
        elm_genlist_item_select_mode_set(item->it, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
-       elm_genlist_item_class_free(extended_multiline_genlist_itc);
-
 out:
+       elm_genlist_item_class_free(extended_multiline_genlist_itc);
        FUNC_END();
        return genlist;
 }
@@ -641,7 +641,7 @@ void create_portable_storage_page_base_layout(appdata_s *ad)
                ad->portable_storage_page_data = NULL;
        }
 
-       portable_storage_page_data_s* portable_storage_page_data = (portable_storage_page_data_s*)malloc(sizeof(portable_storage_page_data));
+       portable_storage_page_data_s* portable_storage_page_data = (portable_storage_page_data_s*)malloc(sizeof(portable_storage_page_data_s));
        if (!portable_storage_page_data) {
                DMSG_ERR("malloc failed");
                return;

diff --git a/src/block/block.c b/src/block/block.c
index 9f843d3ecb7ce994b90a67a93ab7647b095231a6..a348b9a170f03e2ec745cdb5ebbc96c16b4582b2 100644 (file)
--- a/src/block/block.c
+++ b/src/block/block.c
@@ -3504,6 +3504,11 @@ static DBusMessage *request_check_speed(dbus_method_reply_handle_h reply_handle,
 
        _D("speed check: %s", data->devnode);
        fd = open(data->devnode, O_RDWR | O_SYNC);
+       if (fd < 0) {
+               _E("failed to open fd(%s) : %d", data->devnode, errno);
+               result = -1;
+               goto out;
+       }
        buf = calloc(1, SPEEDCHECK << 20);
        if (!buf) {
                _E("calloc() failed");