shinys000114/linux
1
0
Fork 0
mirror of https://github.com/hardkernel/linux.git synced 2026-06-08 20:07:46 +09:00
Code Issues Packages Projects Releases Wiki Activity

ge2d: add return value check for kstrtoint call [1/1]

Browse Source 
PD#OTT-4875

Problem:
missing return value check for kstrtoint

Solution:
add this check for kstrtoint

Verify:
verified on g12b-w400

Change-Id: I0515b4b445c78d950201df6547e83578cac32d16
Signed-off-by: Jian Cao <jian.cao@amlogic.com>
This commit is contained in:
Jian Cao
2019-06-27 14:20:46 +08:00
committed by Chris KIM
parent 90e75d11b9
commit ba11b11d0b
1 changed files with 15 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
drivers/amlogic/media/common/ge2d/ge2d_main.c
View File

@@ -151,6 +151,11 @@ static ssize_t dump_reg_enable_store(struct class *cla,
int ret = 0;
ret = kstrtoint(buf, 0, &res);
if (ret) {
ge2d_log_err("kstrtoint err\n");
return -EINVAL;
}
ge2d_log_info("ge2d dump_reg_enbale: %d->%d\n",
ge2d_dump_reg_enable, res);
ge2d_dump_reg_enable = res;
@@ -173,6 +178,11 @@ static ssize_t dump_reg_cnt_store(struct class *cla,
int ret = 0;
ret = kstrtoint(buf, 0, &res);
if (ret) {
ge2d_log_err("kstrtoint err\n");
return -EINVAL;
}
ge2d_log_info("ge2d dump_reg: %d->%d\n", ge2d_dump_reg_cnt, res);
ge2d_dump_reg_cnt = res;
return count;
@@ -193,6 +203,11 @@ static ssize_t log_level_store(struct class *cla,
int ret = 0;
ret = kstrtoint(buf, 0, &res);
if (ret) {
ge2d_log_err("kstrtoint err\n");
return -EINVAL;
}
ge2d_log_info("ge2d log_level: %d->%d\n", ge2d_log_level, res);
ge2d_log_level = res;