Commit 47c07bff authored by Harald Freudenberger's avatar Harald Freudenberger Committed by Heiko Carstens

s390/zcrypt: fix smatch warnings

Fix these smatch warnings:

zcrypt_api.c:986 _zcrypt_send_ep11_cprb() error: uninitialized symbol 'pref_weight'.
zcrypt_api.c:1008 _zcrypt_send_ep11_cprb() error: uninitialized symbol 'weight'.
zcrypt_api.c:676 zcrypt_rsa_modexpo() error: uninitialized symbol 'pref_weight'.
zcrypt_api.c:694 zcrypt_rsa_modexpo() error: uninitialized symbol 'weight'.
zcrypt_api.c:760 zcrypt_rsa_crt() error: uninitialized symbol 'pref_weight'.
zcrypt_api.c:778 zcrypt_rsa_crt() error: uninitialized symbol 'weight'.
zcrypt_api.c:824 _zcrypt_send_cprb() warn: always true condition '(tdom >= 0) => (0-u16max >= 0)'
zcrypt_api.c:846 _zcrypt_send_cprb() error: uninitialized symbol 'pref_weight'.
zcrypt_api.c:867 _zcrypt_send_cprb() error: uninitialized symbol 'weight'.
zcrypt_api.c:1065 zcrypt_rng() error: uninitialized symbol 'pref_weight'.
zcrypt_api.c:1079 zcrypt_rng() error: uninitialized symbol 'weight'.
zcrypt_cex4.c:251 ep11_card_op_modes_show() warn: should '(1 << ep11_op_modes[i]->mode_bit)' be a 64 bit type?
zcrypt_cex4.c:346 ep11_queue_op_modes_show() warn: should '(1 << ep11_op_modes[i]->mode_bit)' be a 64 bit type?
Reported-by: default avatarkernel test robot <lkp@intel.com>
Reported-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarHarald Freudenberger <freude@linux.ibm.com>
Signed-off-by: default avatarHeiko Carstens <heiko.carstens@de.ibm.com>
parent c6337c6e
...@@ -634,7 +634,7 @@ static long zcrypt_rsa_modexpo(struct ap_perms *perms, ...@@ -634,7 +634,7 @@ static long zcrypt_rsa_modexpo(struct ap_perms *perms,
{ {
struct zcrypt_card *zc, *pref_zc; struct zcrypt_card *zc, *pref_zc;
struct zcrypt_queue *zq, *pref_zq; struct zcrypt_queue *zq, *pref_zq;
unsigned int weight, pref_weight; unsigned int weight = 0, pref_weight = 0;
unsigned int func_code; unsigned int func_code;
int qid = 0, rc = -ENODEV; int qid = 0, rc = -ENODEV;
struct module *mod; struct module *mod;
...@@ -718,7 +718,7 @@ static long zcrypt_rsa_crt(struct ap_perms *perms, ...@@ -718,7 +718,7 @@ static long zcrypt_rsa_crt(struct ap_perms *perms,
{ {
struct zcrypt_card *zc, *pref_zc; struct zcrypt_card *zc, *pref_zc;
struct zcrypt_queue *zq, *pref_zq; struct zcrypt_queue *zq, *pref_zq;
unsigned int weight, pref_weight; unsigned int weight = 0, pref_weight = 0;
unsigned int func_code; unsigned int func_code;
int qid = 0, rc = -ENODEV; int qid = 0, rc = -ENODEV;
struct module *mod; struct module *mod;
...@@ -803,7 +803,7 @@ static long _zcrypt_send_cprb(struct ap_perms *perms, ...@@ -803,7 +803,7 @@ static long _zcrypt_send_cprb(struct ap_perms *perms,
struct zcrypt_card *zc, *pref_zc; struct zcrypt_card *zc, *pref_zc;
struct zcrypt_queue *zq, *pref_zq; struct zcrypt_queue *zq, *pref_zq;
struct ap_message ap_msg; struct ap_message ap_msg;
unsigned int weight, pref_weight; unsigned int weight = 0, pref_weight = 0;
unsigned int func_code; unsigned int func_code;
unsigned short *domain, tdom; unsigned short *domain, tdom;
int qid = 0, rc = -ENODEV; int qid = 0, rc = -ENODEV;
...@@ -822,7 +822,7 @@ static long _zcrypt_send_cprb(struct ap_perms *perms, ...@@ -822,7 +822,7 @@ static long _zcrypt_send_cprb(struct ap_perms *perms,
* domain but a control only domain, use the default domain as target. * domain but a control only domain, use the default domain as target.
*/ */
tdom = *domain; tdom = *domain;
if (tdom >= 0 && tdom < AP_DOMAINS && if (tdom < AP_DOMAINS &&
!ap_test_config_usage_domain(tdom) && !ap_test_config_usage_domain(tdom) &&
ap_test_config_ctrl_domain(tdom) && ap_test_config_ctrl_domain(tdom) &&
ap_domain_index >= 0) ap_domain_index >= 0)
...@@ -931,7 +931,7 @@ static long _zcrypt_send_ep11_cprb(struct ap_perms *perms, ...@@ -931,7 +931,7 @@ static long _zcrypt_send_ep11_cprb(struct ap_perms *perms,
struct zcrypt_queue *zq, *pref_zq; struct zcrypt_queue *zq, *pref_zq;
struct ep11_target_dev *targets; struct ep11_target_dev *targets;
unsigned short target_num; unsigned short target_num;
unsigned int weight, pref_weight; unsigned int weight = 0, pref_weight = 0;
unsigned int func_code; unsigned int func_code;
struct ap_message ap_msg; struct ap_message ap_msg;
int qid = 0, rc = -ENODEV; int qid = 0, rc = -ENODEV;
...@@ -1040,7 +1040,7 @@ static long zcrypt_rng(char *buffer) ...@@ -1040,7 +1040,7 @@ static long zcrypt_rng(char *buffer)
{ {
struct zcrypt_card *zc, *pref_zc; struct zcrypt_card *zc, *pref_zc;
struct zcrypt_queue *zq, *pref_zq; struct zcrypt_queue *zq, *pref_zq;
unsigned int weight, pref_weight; unsigned int weight = 0, pref_weight = 0;
unsigned int func_code; unsigned int func_code;
struct ap_message ap_msg; struct ap_message ap_msg;
unsigned int domain; unsigned int domain;
......
...@@ -250,7 +250,7 @@ static ssize_t ep11_card_op_modes_show(struct device *dev, ...@@ -250,7 +250,7 @@ static ssize_t ep11_card_op_modes_show(struct device *dev,
ep11_get_card_info(ac->id, &ci, zc->online); ep11_get_card_info(ac->id, &ci, zc->online);
for (i = 0; ep11_op_modes[i].mode_txt; i++) { for (i = 0; ep11_op_modes[i].mode_txt; i++) {
if (ci.op_mode & (1 << ep11_op_modes[i].mode_bit)) { if (ci.op_mode & (1ULL << ep11_op_modes[i].mode_bit)) {
if (n > 0) if (n > 0)
buf[n++] = ' '; buf[n++] = ' ';
n += scnprintf(buf + n, PAGE_SIZE - n, n += scnprintf(buf + n, PAGE_SIZE - n,
...@@ -345,7 +345,7 @@ static ssize_t ep11_queue_op_modes_show(struct device *dev, ...@@ -345,7 +345,7 @@ static ssize_t ep11_queue_op_modes_show(struct device *dev,
&di); &di);
for (i = 0; ep11_op_modes[i].mode_txt; i++) { for (i = 0; ep11_op_modes[i].mode_txt; i++) {
if (di.op_mode & (1 << ep11_op_modes[i].mode_bit)) { if (di.op_mode & (1ULL << ep11_op_modes[i].mode_bit)) {
if (n > 0) if (n > 0)
buf[n++] = ' '; buf[n++] = ' ';
n += scnprintf(buf + n, PAGE_SIZE - n, n += scnprintf(buf + n, PAGE_SIZE - n,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment