Fix SVACE major issues

To fix SVACE issue WID: 1318462,1318469,1318470,13185311,1319790

Change-Id: I0bd31f57a9b4d0d05b94bd2a2492719885bfccac
Signed-off-by: shayh <yuhong.sha@samsung.com>

diff --git a/tizen_src/chromium_impl/media/filters/media_player_esplusplayer_tv.cc b/tizen_src/chromium_impl/media/filters/media_player_esplusplayer_tv.cc
index eebba8e403577d6b594f2dad012f122c5b110f2b..08548326eda05185aea44452cccbfa7588dd7681 100644 (file)
--- a/tizen_src/chromium_impl/media/filters/media_player_esplusplayer_tv.cc
+++ b/tizen_src/chromium_impl/media/filters/media_player_esplusplayer_tv.cc
@@ -670,6 +670,7 @@ void MediaPlayerESPlusPlayerTV::SetCallBackFrameSize(const gfx::Size& size) {
 
 bool MediaPlayerESPlusPlayerTV::RequestVideoDecodedBuffer() {
   NOTIMPLEMENTED();
+  return false;
 }
 
 void MediaPlayerESPlusPlayerTV::SetStreamInfo(DemuxerStream::Type type,

diff --git a/tizen_src/ewk/efl_integration/common/trusted_pepper_plugin_info_cache.cc b/tizen_src/ewk/efl_integration/common/trusted_pepper_plugin_info_cache.cc
index 6499aaf5196102ad9bd3e324e73da42111de35e2..2892b9c2bec45f1c7a8901ba953aab843bbc084e 100644 (file)
--- a/tizen_src/ewk/efl_integration/common/trusted_pepper_plugin_info_cache.cc
+++ b/tizen_src/ewk/efl_integration/common/trusted_pepper_plugin_info_cache.cc
@@ -530,7 +530,7 @@ bool ParsePepperPluginManifest(const FilePath& pmf,
   const base::Value::Dict* dict = root.value().GetIfDict();
 
 
-  if (!ValidatePluginType(pmf, dict))
+  if (!dict || !ValidatePluginType(pmf, dict))
     return false;
 
   // Let's set the default values for the info in case they are not present

diff --git a/tizen_src/ewk/efl_integration/eweb_view.cc b/tizen_src/ewk/efl_integration/eweb_view.cc
index ed2208f7df19db71b3fd816837cce74a8b475514..d7f357eb453041b0635a3657aac99e6c38b176cb 100644 (file)
--- a/tizen_src/ewk/efl_integration/eweb_view.cc
+++ b/tizen_src/ewk/efl_integration/eweb_view.cc
@@ -1596,6 +1596,9 @@ void EWebView::HandleLongPressGesture(  // LCOV_EXCL_LINE
     return;                                               // LCOV_EXCL_LINE
   }
 
+  if (!rwhva() || !rwhva()->offscreen_helper())
+    return;
+
   content::ContextMenuParams convertedParams = params;  // LCOV_EXCL_LINE
   gfx::Point convertedPoint =
       /* LCOV_EXCL_START */

diff --git a/tizen_src/ewk/efl_integration/public/ewk_value.cc b/tizen_src/ewk/efl_integration/public/ewk_value.cc
index 01c545978b17b2c879da0bf740b4a2b02afaffef..8d94b1fe834ed528882dffead1bc8335da648cd9 100644 (file)
--- a/tizen_src/ewk/efl_integration/public/ewk_value.cc
+++ b/tizen_src/ewk/efl_integration/public/ewk_value.cc
@@ -178,6 +178,9 @@ Eina_Bool ewk_value_array_get(Ewk_Value array,
   if (result) {
     // TODO(m.majczak) consider a workaround for the deep copy
     list = EwkValueCast(array)->GetValue()->GetIfList();
+    if (!list)
+        return EINA_FALSE;
+
     base::Value* val = nullptr;
     val = &(list->operator[](position));
     if (!val)

diff --git a/tizen_src/ewk/efl_integration/renderer/pepper/pepper_shared_memory_message_filter.cc b/tizen_src/ewk/efl_integration/renderer/pepper/pepper_shared_memory_message_filter.cc
index 1fd507c45a358b452936294a84667f954dc579b1..f05a3120939135c39026240003ef0bdec5a354e5 100644 (file)
--- a/tizen_src/ewk/efl_integration/renderer/pepper/pepper_shared_memory_message_filter.cc
+++ b/tizen_src/ewk/efl_integration/renderer/pepper/pepper_shared_memory_message_filter.cc
@@ -60,10 +60,15 @@ void PepperSharedMemoryMessageFilter::OnHostMsgCreateSharedMemory(
       base::UnsafeSharedMemoryRegion::TakeHandleForSerialization(
           host_->ShareUnsafeSharedMemoryRegionWithRemote(shm)));
 
-  *host_handle_id =
-      content::PepperPluginInstance::Get(instance)
-          ->GetVarTracker()
-          ->TrackSharedMemoryRegion(instance, std::move(shm), size);
+  content::PepperPluginInstance* plugin_instance = content::PepperPluginInstance::Get(instance);
+  if (!plugin_instance)
+    return;
+
+  ppapi::VarTracker* tracker = plugin_instance->GetVarTracker();
+  if (!tracker)
+    return;
+
+  *host_handle_id = tracker->TrackSharedMemoryRegion(instance, std::move(shm), size);
 }
 
 }  // namespace pepper