| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <202112211125.pOtaOLOt-lkp@intel.com>
Date: Tue, 21 Dec 2021 11:54:56 +0800
From: kernel test robot <lkp@...el.com>
To: Kai Ye <yekai13@...wei.com>
Cc: kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org,
Herbert Xu <herbert@...dor.apana.org.au>
Subject: drivers/crypto/hisilicon/qm.c:4135:8: warning: %x in format string
(no. 2) requires 'unsigned int *' but the argument type is 'signed int *'.
[invalidScanfArgType_int]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 86085fe79e3c1a66e32f2acae0ae64f4cceb8d28
commit: 3bbf0783636be8fd672907df25904288f14566f2 crypto: hisilicon/qm - supports to inquiry each function's QoS
date: 6 months ago
compiler: ia64-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>
cppcheck warnings: (new ones prefixed by >>)
drivers/crypto/hisilicon/qm.c:4077:8: warning: %ld in format string (no. 1) requires 'long *' but the argument type is 'unsigned long *'. [invalidScanfArgType_int]
ret = sscanf(buf, "%ld", val);
^
>> drivers/crypto/hisilicon/qm.c:4135:8: warning: %x in format string (no. 2) requires 'unsigned int *' but the argument type is 'signed int *'. [invalidScanfArgType_int]
ret = sscanf(tbuf_bdf, "%d:%x:%d.%d", &tmp1, &bus, &device, &function);
^
cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
drivers/crypto/hisilicon/qm.c:4077:8: warning: %ld in format string (no. 1) requires 'long *' but the argument type is 'unsigned long *'. [invalidScanfArgType_int]
ret = sscanf(buf, "%ld", val);
^
>> drivers/crypto/hisilicon/qm.c:4135:8: warning: %x in format string (no. 2) requires 'unsigned int *' but the argument type is 'signed int *'. [invalidScanfArgType_int]
ret = sscanf(tbuf_bdf, "%d:%x:%d.%d", &tmp1, &bus, &device, &function);
^
>> drivers/crypto/hisilicon/qm.c:4122:8: warning: sscanf() without field width limits can crash with huge input data. [invalidscanf]
ret = sscanf(tbuf, "%s %s", tbuf_bdf, val_buf);
^
>> drivers/crypto/hisilicon/qm.c:4031:9: warning: Uninitialized variable: ret [uninitvar]
return ret;
^
vim +4135 drivers/crypto/hisilicon/qm.c
3bbf0783636be8 Kai Ye 2021-06-11 4002
3bbf0783636be8 Kai Ye 2021-06-11 4003 static int qm_vf_read_qos(struct hisi_qm *qm)
3bbf0783636be8 Kai Ye 2021-06-11 4004 {
3bbf0783636be8 Kai Ye 2021-06-11 4005 int cnt = 0;
3bbf0783636be8 Kai Ye 2021-06-11 4006 int ret;
3bbf0783636be8 Kai Ye 2021-06-11 4007
3bbf0783636be8 Kai Ye 2021-06-11 4008 /* reset mailbox qos val */
3bbf0783636be8 Kai Ye 2021-06-11 4009 qm->mb_qos = 0;
3bbf0783636be8 Kai Ye 2021-06-11 4010
3bbf0783636be8 Kai Ye 2021-06-11 4011 /* vf ping pf to get function qos */
3bbf0783636be8 Kai Ye 2021-06-11 4012 if (qm->ops->ping_pf) {
3bbf0783636be8 Kai Ye 2021-06-11 4013 ret = qm->ops->ping_pf(qm, QM_VF_GET_QOS);
3bbf0783636be8 Kai Ye 2021-06-11 4014 if (ret) {
3bbf0783636be8 Kai Ye 2021-06-11 4015 pci_err(qm->pdev, "failed to send cmd to PF to get qos!\n");
3bbf0783636be8 Kai Ye 2021-06-11 4016 return ret;
3bbf0783636be8 Kai Ye 2021-06-11 4017 }
3bbf0783636be8 Kai Ye 2021-06-11 4018 }
3bbf0783636be8 Kai Ye 2021-06-11 4019
3bbf0783636be8 Kai Ye 2021-06-11 4020 while (true) {
3bbf0783636be8 Kai Ye 2021-06-11 4021 msleep(QM_WAIT_DST_ACK);
3bbf0783636be8 Kai Ye 2021-06-11 4022 if (qm->mb_qos)
3bbf0783636be8 Kai Ye 2021-06-11 4023 break;
3bbf0783636be8 Kai Ye 2021-06-11 4024
3bbf0783636be8 Kai Ye 2021-06-11 4025 if (++cnt > QM_MAX_VF_WAIT_COUNT) {
3bbf0783636be8 Kai Ye 2021-06-11 4026 pci_err(qm->pdev, "PF ping VF timeout!\n");
3bbf0783636be8 Kai Ye 2021-06-11 4027 return -ETIMEDOUT;
3bbf0783636be8 Kai Ye 2021-06-11 4028 }
3bbf0783636be8 Kai Ye 2021-06-11 4029 }
3bbf0783636be8 Kai Ye 2021-06-11 4030
3bbf0783636be8 Kai Ye 2021-06-11 @4031 return ret;
3bbf0783636be8 Kai Ye 2021-06-11 4032 }
3bbf0783636be8 Kai Ye 2021-06-11 4033
3bbf0783636be8 Kai Ye 2021-06-11 4034 static ssize_t qm_algqos_read(struct file *filp, char __user *buf,
3bbf0783636be8 Kai Ye 2021-06-11 4035 size_t count, loff_t *pos)
3bbf0783636be8 Kai Ye 2021-06-11 4036 {
3bbf0783636be8 Kai Ye 2021-06-11 4037 struct hisi_qm *qm = filp->private_data;
3bbf0783636be8 Kai Ye 2021-06-11 4038 char tbuf[QM_DBG_READ_LEN];
3bbf0783636be8 Kai Ye 2021-06-11 4039 u32 qos_val, ir;
3bbf0783636be8 Kai Ye 2021-06-11 4040 int ret;
3bbf0783636be8 Kai Ye 2021-06-11 4041
3bbf0783636be8 Kai Ye 2021-06-11 4042 /* Mailbox and reset cannot be operated at the same time */
3bbf0783636be8 Kai Ye 2021-06-11 4043 if (test_and_set_bit(QM_RESETTING, &qm->misc_ctl)) {
3bbf0783636be8 Kai Ye 2021-06-11 4044 pci_err(qm->pdev, "dev resetting, read alg qos failed!\n");
3bbf0783636be8 Kai Ye 2021-06-11 4045 return -EAGAIN;
3bbf0783636be8 Kai Ye 2021-06-11 4046 }
3bbf0783636be8 Kai Ye 2021-06-11 4047
3bbf0783636be8 Kai Ye 2021-06-11 4048 if (qm->fun_type == QM_HW_PF) {
3bbf0783636be8 Kai Ye 2021-06-11 4049 ir = qm_get_shaper_vft_qos(qm, 0);
3bbf0783636be8 Kai Ye 2021-06-11 4050 } else {
3bbf0783636be8 Kai Ye 2021-06-11 4051 ret = qm_vf_read_qos(qm);
3bbf0783636be8 Kai Ye 2021-06-11 4052 if (ret)
3bbf0783636be8 Kai Ye 2021-06-11 4053 goto err_get_status;
3bbf0783636be8 Kai Ye 2021-06-11 4054 ir = qm->mb_qos;
3bbf0783636be8 Kai Ye 2021-06-11 4055 }
3bbf0783636be8 Kai Ye 2021-06-11 4056
3bbf0783636be8 Kai Ye 2021-06-11 4057 qos_val = ir / QM_QOS_RATE;
3bbf0783636be8 Kai Ye 2021-06-11 4058 ret = scnprintf(tbuf, QM_DBG_READ_LEN, "%u\n", qos_val);
3bbf0783636be8 Kai Ye 2021-06-11 4059
3bbf0783636be8 Kai Ye 2021-06-11 4060 ret = simple_read_from_buffer(buf, count, pos, tbuf, ret);
3bbf0783636be8 Kai Ye 2021-06-11 4061
3bbf0783636be8 Kai Ye 2021-06-11 4062 err_get_status:
3bbf0783636be8 Kai Ye 2021-06-11 4063 clear_bit(QM_RESETTING, &qm->misc_ctl);
3bbf0783636be8 Kai Ye 2021-06-11 4064 return ret;
3bbf0783636be8 Kai Ye 2021-06-11 4065 }
3bbf0783636be8 Kai Ye 2021-06-11 4066
72b010dc33b959 Kai Ye 2021-06-11 4067 static ssize_t qm_qos_value_init(const char *buf, unsigned long *val)
72b010dc33b959 Kai Ye 2021-06-11 4068 {
72b010dc33b959 Kai Ye 2021-06-11 4069 int buflen = strlen(buf);
72b010dc33b959 Kai Ye 2021-06-11 4070 int ret, i;
72b010dc33b959 Kai Ye 2021-06-11 4071
72b010dc33b959 Kai Ye 2021-06-11 4072 for (i = 0; i < buflen; i++) {
72b010dc33b959 Kai Ye 2021-06-11 4073 if (!isdigit(buf[i]))
72b010dc33b959 Kai Ye 2021-06-11 4074 return -EINVAL;
72b010dc33b959 Kai Ye 2021-06-11 4075 }
72b010dc33b959 Kai Ye 2021-06-11 4076
72b010dc33b959 Kai Ye 2021-06-11 4077 ret = sscanf(buf, "%ld", val);
72b010dc33b959 Kai Ye 2021-06-11 4078 if (ret != QM_QOS_VAL_NUM)
72b010dc33b959 Kai Ye 2021-06-11 4079 return -EINVAL;
72b010dc33b959 Kai Ye 2021-06-11 4080
72b010dc33b959 Kai Ye 2021-06-11 4081 return 0;
72b010dc33b959 Kai Ye 2021-06-11 4082 }
72b010dc33b959 Kai Ye 2021-06-11 4083
72b010dc33b959 Kai Ye 2021-06-11 4084 static ssize_t qm_algqos_write(struct file *filp, const char __user *buf,
72b010dc33b959 Kai Ye 2021-06-11 4085 size_t count, loff_t *pos)
72b010dc33b959 Kai Ye 2021-06-11 4086 {
72b010dc33b959 Kai Ye 2021-06-11 4087 struct hisi_qm *qm = filp->private_data;
72b010dc33b959 Kai Ye 2021-06-11 4088 char tbuf[QM_DBG_READ_LEN];
72b010dc33b959 Kai Ye 2021-06-11 4089 int tmp1, bus, device, function;
72b010dc33b959 Kai Ye 2021-06-11 4090 char tbuf_bdf[QM_DBG_READ_LEN] = {0};
72b010dc33b959 Kai Ye 2021-06-11 4091 char val_buf[QM_QOS_VAL_MAX_LEN] = {0};
72b010dc33b959 Kai Ye 2021-06-11 4092 unsigned int fun_index;
72b010dc33b959 Kai Ye 2021-06-11 4093 unsigned long val = 0;
72b010dc33b959 Kai Ye 2021-06-11 4094 int len, ret;
72b010dc33b959 Kai Ye 2021-06-11 4095
72b010dc33b959 Kai Ye 2021-06-11 4096 if (qm->fun_type == QM_HW_VF)
72b010dc33b959 Kai Ye 2021-06-11 4097 return -EINVAL;
72b010dc33b959 Kai Ye 2021-06-11 4098
72b010dc33b959 Kai Ye 2021-06-11 4099 /* Mailbox and reset cannot be operated at the same time */
72b010dc33b959 Kai Ye 2021-06-11 4100 if (test_and_set_bit(QM_RESETTING, &qm->misc_ctl)) {
72b010dc33b959 Kai Ye 2021-06-11 4101 pci_err(qm->pdev, "dev resetting, write alg qos failed!\n");
72b010dc33b959 Kai Ye 2021-06-11 4102 return -EAGAIN;
72b010dc33b959 Kai Ye 2021-06-11 4103 }
72b010dc33b959 Kai Ye 2021-06-11 4104
72b010dc33b959 Kai Ye 2021-06-11 4105 if (*pos != 0) {
72b010dc33b959 Kai Ye 2021-06-11 4106 ret = 0;
72b010dc33b959 Kai Ye 2021-06-11 4107 goto err_get_status;
72b010dc33b959 Kai Ye 2021-06-11 4108 }
72b010dc33b959 Kai Ye 2021-06-11 4109
72b010dc33b959 Kai Ye 2021-06-11 4110 if (count >= QM_DBG_READ_LEN) {
72b010dc33b959 Kai Ye 2021-06-11 4111 ret = -ENOSPC;
72b010dc33b959 Kai Ye 2021-06-11 4112 goto err_get_status;
72b010dc33b959 Kai Ye 2021-06-11 4113 }
72b010dc33b959 Kai Ye 2021-06-11 4114
72b010dc33b959 Kai Ye 2021-06-11 4115 len = simple_write_to_buffer(tbuf, QM_DBG_READ_LEN - 1, pos, buf, count);
72b010dc33b959 Kai Ye 2021-06-11 4116 if (len < 0) {
72b010dc33b959 Kai Ye 2021-06-11 4117 ret = len;
72b010dc33b959 Kai Ye 2021-06-11 4118 goto err_get_status;
72b010dc33b959 Kai Ye 2021-06-11 4119 }
72b010dc33b959 Kai Ye 2021-06-11 4120
72b010dc33b959 Kai Ye 2021-06-11 4121 tbuf[len] = '\0';
72b010dc33b959 Kai Ye 2021-06-11 @4122 ret = sscanf(tbuf, "%s %s", tbuf_bdf, val_buf);
72b010dc33b959 Kai Ye 2021-06-11 4123 if (ret != QM_QOS_PARAM_NUM) {
72b010dc33b959 Kai Ye 2021-06-11 4124 ret = -EINVAL;
72b010dc33b959 Kai Ye 2021-06-11 4125 goto err_get_status;
72b010dc33b959 Kai Ye 2021-06-11 4126 }
72b010dc33b959 Kai Ye 2021-06-11 4127
72b010dc33b959 Kai Ye 2021-06-11 4128 ret = qm_qos_value_init(val_buf, &val);
72b010dc33b959 Kai Ye 2021-06-11 4129 if (val == 0 || val > QM_QOS_MAX_VAL || ret) {
72b010dc33b959 Kai Ye 2021-06-11 4130 pci_err(qm->pdev, "input qos value is error, please set 1~1000!\n");
72b010dc33b959 Kai Ye 2021-06-11 4131 ret = -EINVAL;
72b010dc33b959 Kai Ye 2021-06-11 4132 goto err_get_status;
72b010dc33b959 Kai Ye 2021-06-11 4133 }
72b010dc33b959 Kai Ye 2021-06-11 4134
72b010dc33b959 Kai Ye 2021-06-11 @4135 ret = sscanf(tbuf_bdf, "%d:%x:%d.%d", &tmp1, &bus, &device, &function);
72b010dc33b959 Kai Ye 2021-06-11 4136 if (ret != QM_QOS_BDF_PARAM_NUM) {
72b010dc33b959 Kai Ye 2021-06-11 4137 pci_err(qm->pdev, "input pci bdf value is error!\n");
72b010dc33b959 Kai Ye 2021-06-11 4138 ret = -EINVAL;
72b010dc33b959 Kai Ye 2021-06-11 4139 goto err_get_status;
72b010dc33b959 Kai Ye 2021-06-11 4140 }
72b010dc33b959 Kai Ye 2021-06-11 4141
72b010dc33b959 Kai Ye 2021-06-11 4142 fun_index = device * 8 + function;
72b010dc33b959 Kai Ye 2021-06-11 4143
72b010dc33b959 Kai Ye 2021-06-11 4144 ret = qm_func_shaper_enable(qm, fun_index, val);
72b010dc33b959 Kai Ye 2021-06-11 4145 if (ret) {
72b010dc33b959 Kai Ye 2021-06-11 4146 pci_err(qm->pdev, "failed to enable function shaper!\n");
72b010dc33b959 Kai Ye 2021-06-11 4147 ret = -EINVAL;
72b010dc33b959 Kai Ye 2021-06-11 4148 goto err_get_status;
72b010dc33b959 Kai Ye 2021-06-11 4149 }
72b010dc33b959 Kai Ye 2021-06-11 4150
72b010dc33b959 Kai Ye 2021-06-11 4151 ret = count;
72b010dc33b959 Kai Ye 2021-06-11 4152
72b010dc33b959 Kai Ye 2021-06-11 4153 err_get_status:
72b010dc33b959 Kai Ye 2021-06-11 4154 clear_bit(QM_RESETTING, &qm->misc_ctl);
72b010dc33b959 Kai Ye 2021-06-11 4155 return ret;
72b010dc33b959 Kai Ye 2021-06-11 4156 }
72b010dc33b959 Kai Ye 2021-06-11 4157
:::::: The code at line 4135 was first introduced by commit
:::::: 72b010dc33b9598883bc84d40b0a9d07c16f5e39 crypto: hisilicon/qm - supports writing QoS int the host
:::::: TO: Kai Ye <yekai13@...wei.com>
:::::: CC: Herbert Xu <herbert@...dor.apana.org.au>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
Powered by blists - more mailing lists