Fix issues detected by Coverity (2024.09.30)

Signed-off-by: Michal Zientkiewicz <michalz@nvidia.com>
NVIDIA · Sep 30, 2024 · ffb87e5 · ffb87e5
1 parent 57909cc
commit ffb87e5
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 2 deletions.
diff --git a/dali/operators/image/remap/cvcuda/matrix_adjust.cu b/dali/operators/image/remap/cvcuda/matrix_adjust.cu
@@ -55,6 +55,7 @@ void adjustMatrices(nvcv::Tensor &matrices, cudaStream_t stream) {
  int num_blocks = div_ceil(bs, 256);
  int threads_per_block = std::min(bs, 256);
  adjustMatricesKernel2<<<num_blocks, threads_per_block, 0, stream>>>(wrap, bs);
+ CUDA_CALL(cudaGetLastError());
 }
 
 } // namespace warp_perspective

diff --git a/dali/operators/image/remap/cvcuda/warp_perspective.cc b/dali/operators/image/remap/cvcuda/warp_perspective.cc
@@ -231,7 +231,7 @@ class WarpPerspective : public nvcvop::NVCVSequenceOperator<StatelessOperator> {
  NVCVBorderType border_mode_ = NVCV_BORDER_CONSTANT;
  NVCVInterpolationType interp_type_ = NVCV_INTERP_NEAREST;
  std::vector<float> fill_value_arg_{0, 0, 0, 0};
- float4 fill_value_;
+ float4 fill_value_{};
  bool inverse_map_ = false;
  bool ocv_pixel_ = true;
  std::optional<cvcuda::WarpPerspective> warp_perspective_;

diff --git a/dali/pipeline/executor/executor2/exec2.cc b/dali/pipeline/executor/executor2/exec2.cc
@@ -347,7 +347,7 @@ class Executor2::Impl {
  auto stream_idx = assignment[&node];
 
  node.env.order = stream_idx.has_value()
- ? AccessOrder(streams_[*stream_idx])
+ ? AccessOrder(streams_[*stream_idx].get())
  : AccessOrder::host();
  }
  }

diff --git a/dali/pipeline/executor/executor2/exec2_ops_for_test.cu b/dali/pipeline/executor/executor2/exec2_ops_for_test.cu
@@ -99,6 +99,7 @@ void DummyOpGPU::RunImpl(Workspace &ws) {
  scratch.ToGPU(ws.stream(), pointers),
  ws.NumInput() + 1,
  N);
+ CUDA_CALL(cudaGetLastError());
 }
 
 

diff --git a/dali/pipeline/executor/executor2/exec_node_task.cc b/dali/pipeline/executor/executor2/exec_node_task.cc
@@ -259,6 +259,7 @@ void OpTask::RunOp() {
  ResetInputLayouts();
  PropagateSourceInfo(*ws_);
  }
+ assert(ws_->GetIterationData());
  if (auto cpt = ws_->GetIterationData()->checkpoint) {
  node_->op->SaveState(cpt->GetOpCheckpoint(node_->instance_name), ws_->output_order());
  }