Browse Source

fixed coverity warning

pull/1994/head
zhao-lupeng 3 years ago
parent
commit
722b86e6bd
13 changed files with 18 additions and 0 deletions
  1. +2
    -0
      ge/common/formats/format_transfers/format_transfer_fractal_nz.cc
  2. +1
    -0
      ge/common/op/ge_op_utils.cc
  3. +1
    -0
      ge/ge_local_engine/engine/host_cpu_engine.cc
  4. +1
    -0
      ge/graph/load/model_manager/model_utils.cc
  5. +3
    -0
      ge/graph/manager/graph_var_manager.cc
  6. +1
    -0
      ge/host_kernels/add_kernel.cc
  7. +1
    -0
      ge/host_kernels/empty_kernel.cc
  8. +1
    -0
      ge/host_kernels/fill_kernel.cc
  9. +1
    -0
      ge/host_kernels/floordiv_kernel.cc
  10. +1
    -0
      ge/host_kernels/floormod_kernel.cc
  11. +2
    -0
      ge/host_kernels/gather_v2_kernel.cc
  12. +2
    -0
      ge/host_kernels/rsqrt_kernel.cc
  13. +1
    -0
      ge/hybrid/node_executor/rts/rts_node_task.cc

+ 2
- 0
ge/common/formats/format_transfers/format_transfer_fractal_nz.cc View File

@@ -94,6 +94,7 @@ Status TransShapeToFracNz(const ShapeVector &src_shape, DataType data_type, Shap
return ACL_ERROR_GE_SHAPE_INVALID;
}
return SUCCESS;
break;
default:
auto size = src_shape.size();
int64_t times = 1;
@@ -116,6 +117,7 @@ Status TransShapeToFracNz(const ShapeVector &src_shape, DataType data_type, Shap
return ACL_ERROR_GE_SHAPE_INVALID;
}
return SUCCESS;
break;
}
}



+ 1
- 0
ge/common/op/ge_op_utils.cc View File

@@ -346,6 +346,7 @@ Status OpUtils::SetOutputSliceData(void *data, int64_t data_size, int32_t data_t
default:
GELOGW("Unsupported data type: %s", TypeUtils::DataTypeToSerialString(static_cast<DataType>(data_type)).c_str());
return PARAM_INVALID;
break;
}
return ret;
}


+ 1
- 0
ge/ge_local_engine/engine/host_cpu_engine.cc View File

@@ -198,6 +198,7 @@ Status HostCpuEngine::PrepareOutputs(const ge::ConstOpDescPtr &op_desc,
GELOGW("data type %s not support.",
TypeUtils::DataTypeToSerialString(out_desc.GetDataType()).c_str());
return NOT_CHANGED;
break;
}
}



+ 1
- 0
ge/graph/load/model_manager/model_utils.cc View File

@@ -423,6 +423,7 @@ Status ModelUtils::GetVarAddr(const RuntimeParam &model_param, const ConstOpDesc
GELOGE(PARAM_INVALID, "[Check][Param] Get mem_type:%d for offset:%ld is unsupported, check invalid",
mem_type, offset);
return PARAM_INVALID;
break;
}
GE_CHECK_NOTNULL(var_addr);
return SUCCESS;


+ 3
- 0
ge/graph/manager/graph_var_manager.cc View File

@@ -246,10 +246,13 @@ MemResource *MemResource::BuildMemResourceFromType(rtMemType_t mem_type) {
switch (mem_type) {
case RT_MEMORY_HBM:
return new (std::nothrow) HbmMemResource();
break;
case RT_MEMORY_RDMA_HBM:
return new (std::nothrow) RdmaMemResource();
break;
default:
return nullptr;
break;
}
}



+ 1
- 0
ge/host_kernels/add_kernel.cc View File

@@ -189,6 +189,7 @@ Status AddKernel::Compute(const OpDescPtr op_desc_ptr, const std::vector<ConstGe
default:
GELOGI("Add kernel data type %s not support.", TypeUtils::DataTypeToSerialString(data_type).c_str());
return NOT_CHANGED;
break;
}

if (ret != SUCCESS) {


+ 1
- 0
ge/host_kernels/empty_kernel.cc View File

@@ -124,6 +124,7 @@ Status EmptyKernel::Compute(const OpDescPtr op_desc_ptr, const std::vector<Const
default:
GELOGW("invalid data type: %s", TypeUtils::DataTypeToSerialString(data_type).c_str());
return NOT_CHANGED;
break;
}

if (ret != SUCCESS) {


+ 1
- 0
ge/host_kernels/fill_kernel.cc View File

@@ -115,6 +115,7 @@ Status FillKernel::Compute(const ge::OpDescPtr op_desc_ptr, const std::vector<ge
default:
GELOGW("invalid data type: %s", TypeUtils::DataTypeToSerialString(data_type).c_str());
return NOT_CHANGED;
break;
}
if (ret != SUCCESS) {
GELOGE(ret, "GenData failed, data_type: %s", TypeUtils::DataTypeToSerialString(data_type).c_str());


+ 1
- 0
ge/host_kernels/floordiv_kernel.cc View File

@@ -244,6 +244,7 @@ Status FloorDivKernel::ComputeByDataType(DataType data_type, const std::vector<C
default:
GELOGW("FloorDivKernel does not support Data type:%s", TypeUtils::DataTypeToSerialString(data_type).c_str());
return NOT_CHANGED;
break;
}
return ret;
}


+ 1
- 0
ge/host_kernels/floormod_kernel.cc View File

@@ -58,6 +58,7 @@ Status CheckYIsZero(T const &y, DataType &type) {
break;
default:
return INTERNAL_ERROR;
break;
}
return SUCCESS;
}


+ 2
- 0
ge/host_kernels/gather_v2_kernel.cc View File

@@ -210,6 +210,7 @@ Status GatherV2Kernel::GenData(const int64_t data_num, ConstGeTensorPtr tensor_x
default:
GELOGI("Only support 4 dims and below but input axis is %ld", axis);
return NOT_CHANGED;
break;
}
return ret;
}
@@ -269,6 +270,7 @@ Status GatherV2Kernel::Process(int64_t axis, DataType data_type, ConstGeTensorPt
default:
GELOGI("GatherV2Kernel does not support this Data type:%s", TypeUtils::DataTypeToSerialString(data_type).c_str());
return NOT_CHANGED;
break;
}
return ret;
}


+ 2
- 0
ge/host_kernels/rsqrt_kernel.cc View File

@@ -96,6 +96,7 @@ Status RsqrtKernel::RsqrtCompute(ConstGeTensorPtr &input_tensor_ptr, GeTensorPtr
default:
GELOGW("Input data type must be FP16, FP32 and DOUBLE.");
return NOT_CHANGED;
break;
}
}
GE_IF_BOOL_EXEC(output_tensor_ptr->SetData(reinterpret_cast<uint8_t *>(buf.get()), data_size) != GRAPH_SUCCESS,
@@ -136,6 +137,7 @@ Status RsqrtKernel::Compute(const OpDescPtr op_desc_ptr, const std::vector<Const
default:
GELOGW("Input data type must be FP16, FP32 and DOUBLE.");
return NOT_CHANGED;
break;
}
if (ret != SUCCESS) {
GELOGW("Rsqrt folding failed.");


+ 1
- 0
ge/hybrid/node_executor/rts/rts_node_task.cc View File

@@ -80,6 +80,7 @@ Status RtsNodeTask::GetScalarIndexValue(TaskContext &task_context, uint32_t inde
default: {
GELOGE(UNSUPPORTED, "Data type %s not index type.", TypeUtils::DataTypeToSerialString(data_type).c_str());
return UNSUPPORTED;
break;
}
}



Loading…
Cancel
Save