From: Takashi Iwai Date: Wed, 11 Mar 2020 07:47:38 +0000 (+0100) Subject: mfd: aat2870: Use scnprintf() for avoiding potential buffer overflow X-Git-Tag: v5.10.7~2853^2~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c2b5fdfba2a915b5001d3ec63f4b3fc651c22da4;p=platform%2Fkernel%2Flinux-rpi.git mfd: aat2870: Use scnprintf() for avoiding potential buffer overflow There is still one call of sprintf() without checking the proper buffer overflow in aat2870_dump_reg(). Replace it with scnprintf() call for covering that. Signed-off-by: Takashi Iwai Signed-off-by: Lee Jones --- diff --git a/drivers/mfd/aat2870-core.c b/drivers/mfd/aat2870-core.c index 78ee4b2..a17cf75 100644 --- a/drivers/mfd/aat2870-core.c +++ b/drivers/mfd/aat2870-core.c @@ -221,7 +221,7 @@ static ssize_t aat2870_dump_reg(struct aat2870_data *aat2870, char *buf) count += sprintf(buf, "aat2870 registers\n"); for (addr = 0; addr < AAT2870_REG_NUM; addr++) { - count += sprintf(buf + count, "0x%02x: ", addr); + count += snprintf(buf + count, PAGE_SIZE - count, "0x%02x: ", addr); if (count >= PAGE_SIZE - 1) break;