From 607afbe1c02cabc719cdb449ea6a363d9a30fafb Mon Sep 17 00:00:00 2001 From: Yufeng Li Date: Tue, 14 Jun 2022 14:02:15 -0700 Subject: [PATCH] =?UTF-8?q?fix=20valgrind=20warnings:Conditional=20jump=20?= =?UTF-8?q?or=20move=20depends=20on=20uninitialis=E2=80=A6=20(#11822)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * fix valgrind warnings:Conditional jump or move depends on uninitialised value(s) --- onnxruntime/core/providers/cpu/quantization/qlinearconv.cc | 1 + onnxruntime/core/session/inference_session.h | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/onnxruntime/core/providers/cpu/quantization/qlinearconv.cc b/onnxruntime/core/providers/cpu/quantization/qlinearconv.cc index af85dddfb6..36f2f76cfa 100644 --- a/onnxruntime/core/providers/cpu/quantization/qlinearconv.cc +++ b/onnxruntime/core/providers/cpu/quantization/qlinearconv.cc @@ -644,6 +644,7 @@ Status QLinearConv::Compute(OpKernelContext* context) const { group_col_buffer_size += MLAS_SYMM_QGEMM_BUF_OVERRUN; auto* col_data = alloc->Alloc(SafeInt(sizeof(ActType)) * group_col_buffer_size); col_buffer = BufferUniquePtr(col_data, BufferDeleter(alloc)); + memset(col_data, 0, SafeInt(sizeof(ActType)) * group_col_buffer_size); } } if (use_indirection_buffer) { diff --git a/onnxruntime/core/session/inference_session.h b/onnxruntime/core/session/inference_session.h index 3318e89c11..996011da43 100644 --- a/onnxruntime/core/session/inference_session.h +++ b/onnxruntime/core/session/inference_session.h @@ -713,7 +713,7 @@ class InferenceSession { DataTransferManager data_transfer_mgr_; // Number of concurrently running executors - std::atomic current_num_runs_; + std::atomic current_num_runs_ = 0; mutable onnxruntime::OrtMutex session_mutex_; // to ensure only one thread can invoke Load/Initialize bool is_model_loaded_ = false; // GUARDED_BY(session_mutex_)