[luci/pass] tizen svace issue on luci/pass

[chunseoklee]

ONE/compiler/luci/pass/src/PropagateQParamBackwardPass.cpp

Line 102 in d2680f8

if (target_qparam == nullptr)

Tizen static analyzer comments(severity: Major) about the line above :

Warning Message
The pointer returned by operator new can't be null, but it's compared with null at PropagateQParamBackwardPass.cpp 102.

[chunseoklee]

@seanshpark We have 2 options :

1. remove this by patch
2. keep it as tag "intentional" ( just in case)

Which one do you prefer ?

[seanshpark]

@jinevening PTAL

[jinevening]

The code seems unnecessary. I made a PR to remove it. #14504

[hseok-oh]

FYI. Several similar issues are in luci/service, but they were marked as ignored as intentional.

ex. compiler/luci/service/src/Nodes/CircleUnpackOut.cpp

  auto *cloned = _graph->nodes()->create<luci::CircleUnpackOut>();
  if (cloned != nullptr)
    cloned->index(node->index());

[seanshpark]

fixed