[nnc] fix static analysis warning and add missed break (#2287)

- Add assert, which protects us from passing a null pointer to a function.
- Add break statement into switch in tflite_importer.

Signed-off-by: Pavel <p.fattakhov@partner.samsung.com>

diff --git a/contrib/nnc/passes/tflite_frontend/tflite_importer.cpp b/contrib/nnc/passes/tflite_frontend/tflite_importer.cpp
index 318658a..d96fe5a 100644 (file)
--- a/contrib/nnc/passes/tflite_frontend/tflite_importer.cpp
+++ b/contrib/nnc/passes/tflite_frontend/tflite_importer.cpp
@@ -188,6 +188,7 @@ void TfliteImporter::walkOperator(const Operator* op) {
       break;
     case BuiltinOperator_SQUEEZE:
       outputs = _opCreator->createSqueeze(inputs, params, op->builtin_options_as<SqueezeOptions>());
+      break;
     case BuiltinOperator_ADD:
       outputs = _opCreator->createAdd(inputs, params, op->builtin_options_as<AddOptions>());
       break;
@@ -264,6 +265,7 @@ std::vector<std::shared_ptr<IrTensor>> TfliteImporter::createOpParams(const Oper
 std::shared_ptr<IrTensor> TfliteImporter::createTensor(const Tensor* t, const Buffer* b) {
   // Create TensorVariant by copying the tensor buffer contents.
   // Another option is to copy the data in a TensorVariant constructor.
+  assert(b->data() != nullptr);
   std::shared_ptr<char> tensor_buffer_copy(new char[b->data()->size()],
                                            [](char* d) { delete[] d; });
   std::copy(b->data()->begin(), b->data()->end(), tensor_buffer_copy.get());