diff options
author | xinhui pan <xinhui.pan@amd.com> | 2019-03-11 06:10:57 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2019-03-19 16:36:51 -0400 |
commit | b076296b0fb39f142bdc183d4bab91e7204c53bb (patch) | |
tree | 833d1a8a74aa139707db10228be603431d133fe2 /drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | |
parent | 5caf466a6e42beee88a707105975a1d2324b659c (diff) |
drm/amdgpu: Fix ras debugfs data parse
Unzero char is accepted by sscanf, so when data is structure but
unexpectedly return error invalid;
Signed-off-by: xinhui pan <xinhui.pan@amd.com>
Reviewed-by: Feifei Xu <Feifei.Xu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c index 74a65a61fd23..1acaa8473629 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | |||
@@ -241,7 +241,7 @@ static int amdgpu_ras_debugfs_ctrl_parse_data(struct file *f, | |||
241 | op = 1; | 241 | op = 1; |
242 | else if (sscanf(str, "inject %32s %8s", block_name, err) == 2) | 242 | else if (sscanf(str, "inject %32s %8s", block_name, err) == 2) |
243 | op = 2; | 243 | op = 2; |
244 | else if (sscanf(str, "%32s", block_name) == 1) | 244 | else if (str[0] && str[1] && str[2] && str[3]) |
245 | /* ascii string, but commands are not matched. */ | 245 | /* ascii string, but commands are not matched. */ |
246 | return -EINVAL; | 246 | return -EINVAL; |
247 | 247 | ||