From b4ebbe74486a4137334064d640068f4b2618d4ea Mon Sep 17 00:00:00 2001 From: Shweta Khatri Date: Wed, 25 Sep 2024 22:53:28 -0400 Subject: [PATCH] hsakmt: spm.c: Fix Coverity reported warnings Fix unused ret value and initialize gpu_id Change-Id: Ib3acc7db4bbab519318d0970786a5dc641dcc9eb [ROCm/ROCR-Runtime commit: 9f43c9fd512a5e2fcd0cbc8feaaaeb9d37290ad1] --- projects/rocr-runtime/libhsakmt/src/spm.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/projects/rocr-runtime/libhsakmt/src/spm.c b/projects/rocr-runtime/libhsakmt/src/spm.c index 1a1e6e72db..3ad72ccf2f 100644 --- a/projects/rocr-runtime/libhsakmt/src/spm.c +++ b/projects/rocr-runtime/libhsakmt/src/spm.c @@ -59,11 +59,12 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtSPMSetDestBuffer(HSAuint32 PreferredNode, { int ret; struct kfd_ioctl_spm_args args = {0}; - uint32_t gpu_id; - - ret = HSAKMT_STATUS_SUCCESS; + uint32_t gpu_id = 0; ret = hsakmt_validate_nodeid(PreferredNode, &gpu_id); + if (ret != HSAKMT_STATUS_SUCCESS) { + return ret; + } args.timeout = *timeout; args.dest_buf = (uint64_t)DestMemoryAddress;