ARM: 7160/1: setup: avoid overflowing {elf,arch}_name from proc_info_list
authorWill Deacon <will.deacon@arm.com>
Fri, 11 Nov 2011 10:35:58 +0000 (11:35 +0100)
committerRussell King <rmk+kernel@arm.linux.org.uk>
Sat, 12 Nov 2011 11:33:14 +0000 (11:33 +0000)
commita34dbfb03f472ad3e063d3b1d1e88f7ef8f88bf1
treec3e5418493546663bf0b9ba00810621b9ec7b0d9
parent798681bf2127e60f16e9decc5e3a0b901f3ff3fe
ARM: 7160/1: setup: avoid overflowing {elf,arch}_name from proc_info_list

setup_processor copies the arch_name and elf_name fields out of the
selected proc_info_list into two fixed size buffers.

Since the proc_info_list structure is defined in a proc_*.S assembly
file, this can lead to subtle errors if the strings defined there are
too long (for example, corrupting the machine ID).

This patch uses snprintf instead of sprintf to ensure that these buffers
are not overrun.

Signed-off-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
arch/arm/kernel/setup.c