From: Chunseok Lee Date: Thu, 10 Jun 2021 01:55:58 +0000 (+0900) Subject: Fix coverity 1229862 : uinit variable in PoolParams X-Git-Tag: accepted/tizen/unified/20210615.120307^0 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5cda897e9e77c9c8fae94a9d5046d6042ae0df70;p=platform%2Fcore%2Fml%2Fnnfw.git Fix coverity 1229862 : uinit variable in PoolParams Remove activation, padding_type Add initilization for others ONE-DCO-1.0-Signed-off-by: Chunseok Lee --- diff --git a/compute/cker/include/cker/Types.h b/compute/cker/include/cker/Types.h index c5dd63b..495c894 100644 --- a/compute/cker/include/cker/Types.h +++ b/compute/cker/include/cker/Types.h @@ -80,8 +80,6 @@ enum class BroadcastableOpCategory : uint8_t struct PoolParams { - FusedActivationFunctionType activation; - PaddingType padding_type; PaddingValues padding_values; int stride_height; int stride_width; diff --git a/runtime/onert/backend/cpu/ops/PoolLayer.cc b/runtime/onert/backend/cpu/ops/PoolLayer.cc index 101b6f2..088ca5f 100644 --- a/runtime/onert/backend/cpu/ops/PoolLayer.cc +++ b/runtime/onert/backend/cpu/ops/PoolLayer.cc @@ -79,7 +79,11 @@ PoolLayer::PoolLayer() : _input(nullptr), _output(nullptr), _kernel() op_params.filter_height = kernelHeight; \ op_params.filter_width = kernelWidth; \ op_params.padding_values.height = (int8_t)paddingTop; \ - op_params.padding_values.width = (int8_t)paddingLeft; + op_params.padding_values.width = (int8_t)paddingLeft; \ + op_params.float_activation_min = 0; \ + op_params.float_activation_max = 0; \ + op_params.quantized_activation_min = 0; \ + op_params.quantized_activation_max = 0; void PoolLayer::configure(const IPortableTensor *input, const uint32_t paddingLeft, const uint32_t, const uint32_t paddingTop, const uint32_t, const uint32_t strideWidth,