[svace] fix SIGN_EXTENSION

This patch fixes svace issues.
WGID 516502, 517025, 517446, 517866

Signed-off-by: Yelin Jeong <yelini.jeong@samsung.com>

diff --git a/src/core/comm/CommPlugin.h b/src/core/comm/CommPlugin.h
index ca7a99b3ea5f55561d620d666e4e1c59b70c3870..792e04b0e59085460dc6685df0ea6314613cb120 100644 (file)
--- a/src/core/comm/CommPlugin.h
+++ b/src/core/comm/CommPlugin.h
@@ -27,7 +27,7 @@ class CommPlugin {
 
   /** host -> npu-engine */
   virtual int extractGenericBuffer (const generic_buffer *buf, void *data,
-                                    dataManipFunc func = nullptr, size_t offset = 0,
+                                    dataManipFunc func = nullptr, off_t offset = 0,
                                     size_t size = 0) = 0;
   /** npu-engine -> host */
   virtual int insertGenericBuffer (void *data, generic_buffer *buf,

diff --git a/src/core/comm/plugin-comm-ip.cc b/src/core/comm/plugin-comm-ip.cc
index 779b9c605499e126f19a27895779b94819f4a0dc..71747383901d0cb620167cd988573b71f5adbef6 100644 (file)
--- a/src/core/comm/plugin-comm-ip.cc
+++ b/src/core/comm/plugin-comm-ip.cc
@@ -47,7 +47,7 @@ class CommPluginIP : public CommPlugin {
    * @return 0 if no error. otherwise a negative errno
    */
   int extractGenericBuffer (const generic_buffer *buf, void *data, dataManipFunc func = nullptr,
-                            size_t offset = 0, size_t size = 0) {
+                            off_t offset = 0, size_t size = 0) {
     size_t count = 0;
 
     if (buf == nullptr || data == nullptr)
@@ -67,12 +67,12 @@ class CommPluginIP : public CommPlugin {
           if (func != nullptr) {
             char *tmp = new char[size];
 
-            ifs.read (tmp, size);
+            ifs.read (tmp, static_cast<std::streamsize> (size));
             func (data, tmp, size);
 
             delete[] tmp;
           } else {
-            ifs.read (static_cast<char *> (data), size);
+            ifs.read (static_cast<char *> (data), static_cast<std::streamsize> (size));
           }
           count = ifs.gcount ();
           ifs.close ();

diff --git a/utils/model_inspect/model_inspect.cc b/utils/model_inspect/model_inspect.cc
index 0c2fb1bc288d7a1f3f7bee700d3be85004ec8565..f29288b83fff5b241c727e41fcc194359ed976c2 100644 (file)
--- a/utils/model_inspect/model_inspect.cc
+++ b/utils/model_inspect/model_inspect.cc
@@ -314,7 +314,9 @@ Inspector::show_v3 () {
 
 void
 Inspector::dump () {
-  size_t size_metadata = getNPUmodel_metasize (meta_);
+  std::streamsize size_metadata = static_cast<std::streamsize> (getNPUmodel_metasize (meta_));
+  std::streamsize meta_program_size = static_cast<std::streamsize> (meta_->program_size);
+  std::streamsize meta_weight_size = static_cast<std::streamsize> (meta_->weight_size);
 
   if (dump_meta_) {
     std::ofstream ofs (dump_output_ + "/metadata.bin", std::ios::binary);
@@ -335,8 +337,8 @@ Inspector::dump () {
       char *buf = new char[meta_->program_size];
 
       ifs_.seekg (size_metadata);
-      ifs_.read (buf, meta_->program_size);
-      ofs.write (buf, meta_->program_size);
+      ifs_.read (buf, meta_program_size);
+      ofs.write (buf, meta_program_size);
 
       delete[] buf;
     }
@@ -347,9 +349,9 @@ Inspector::dump () {
     if (ofs.good ()) {
       char *buf = new char[meta_->weight_size];
 
-      ifs_.seekg (size_metadata + meta_->program_size);
-      ifs_.read (buf, meta_->weight_size);
-      ofs.write (buf, meta_->weight_size);
+      ifs_.seekg (size_metadata + meta_program_size);
+      ifs_.read (buf, meta_weight_size);
+      ofs.write (buf, meta_weight_size);
 
       delete[] buf;
     }