AI/rocm-systems
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

amdsmitst crash when read ECC counter

Browse Source 
If the amdsmi is fail to read ECC counter from sysfs file, it
still assumes the data is read into vector, which causes crash.

Add the check to make sure the data is read into the vector.

Change-Id: Ia24331ee6e75408120248b0ed7bc58581a2f23ad


[ROCm/amdsmi commit: ad3b7d9f0d]
This commit is contained in:
Bill(Shuzhou) Liu
2023-03-07 09:08:27 -06:00
committed by Shuzhou Liu
parent c5054f75f4
commit e18c56efd8
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
projects/amdsmi/rocm_smi/src/rocm_smi.cc
+1 -1
View File
@@ -637,7 +637,7 @@ rsmi_dev_ecc_count_get(uint32_t dv_ind, rsmi_gpu_block_t block,
ret = GetDevValueVec(type, dv_ind, &val_vec);
if (ret == RSMI_STATUS_FILE_ERROR) {
if (ret == RSMI_STATUS_FILE_ERROR || val_vec.size() != 2) {
return RSMI_STATUS_NOT_SUPPORTED;
}
if (ret != RSMI_STATUS_SUCCESS) {