s390: qeth: Fix potential array overrun in cmd/rc lookup
commit048a7f8b4ec085d5c56ad4a3bf450389a4aed5f9
authorJean Delvare <jdelvare@suse.de>
Wed, 26 Sep 2018 16:07:10 +0000 (26 18:07 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 28 Sep 2018 17:27:26 +0000 (28 10:27 -0700)
tree41815bbe3107dd8e9647df90df1332e8218d8f9e
parent065a2cdcbdf8eb9aefb66e1a24b2d684b8b8852b
s390: qeth: Fix potential array overrun in cmd/rc lookup

Functions qeth_get_ipa_msg and qeth_get_ipa_cmd_name are modifying
the last member of global arrays without any locking that I can see.
If two instances of either function are running at the same time,
it could cause a race ultimately leading to an array overrun (the
contents of the last entry of the array is the only guarantee that
the loop will ever stop).

Performing the lookups without modifying the arrays is admittedly
slower (two comparisons per iteration instead of one) but these
are operations which are rare (should only be needed in error
cases or when debugging, not during successful operation) and it
seems still less costly than introducing a mutex to protect the
arrays in question.

As a side bonus, it allows us to declare both arrays as const data.

Signed-off-by: Jean Delvare <jdelvare@suse.de>
Cc: Julian Wiedmann <jwi@linux.ibm.com>
Cc: Ursula Braun <ubraun@linux.ibm.com>
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Julian Wiedmann <jwi@linux.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/s390/net/qeth_core_main.c
drivers/s390/net/qeth_core_mpc.c
drivers/s390/net/qeth_core_mpc.h