fixup! Saving tab orgins between browser starts 62/73462/1
authorMaciej Skrzypkowski <m.skrzypkows@samsung.com>
Wed, 8 Jun 2016 07:45:21 +0000 (09:45 +0200)
committerMaciej Skrzypkowski <m.skrzypkows@samsung.com>
Wed, 8 Jun 2016 07:45:21 +0000 (09:45 +0200)
Change-Id: I7771b7fb7f876243f2ad40840ec9e54288af9a1b

services/TabService/TabService.cpp

index a983f6c305f87b9f5dabcf1b6e36ae62cc0b2f36..e50d198e6a2bf4fd20f58681441e0dd94928b836 100644 (file)
@@ -73,7 +73,7 @@ void TabService::errorPrint(std::string method) const
 std::shared_ptr<std::vector<basic_webengine::TabContent> > TabService::getAllTabs()
 {
     BROWSER_LOGD("[%s:%d]", __PRETTY_FUNCTION__, __LINE__);
-    int* items;
+    int* items = nullptr;
     int count;
     auto vec = std::make_shared<std::vector<basic_webengine::TabContent> >(std::vector<basic_webengine::TabContent>());
     if (bp_tab_adaptor_get_full_ids_p(&items, &count) < 0) {
@@ -104,7 +104,8 @@ std::shared_ptr<std::vector<basic_webengine::TabContent> > TabService::getAllTab
                 std::string(info.title),
                 basic_webengine::TabOrigin(info.index)));
     }
-    free(items);
+    if (count > 0)
+        free(items);
 
     return vec;
 }