mtd: fix oops in dataflash driver
authorWill Newton <will.newton@gmail.com>
Fri, 30 Mar 2012 10:51:02 +0000 (11:51 +0100)
committerDavid Woodhouse <David.Woodhouse@intel.com>
Mon, 7 May 2012 19:29:50 +0000 (20:29 +0100)
commit7a84477c4acebf6299b6a8bd6a1d5894eb838ffa
tree7ead0857a87874bb3362953d5fb55749f88903de
parent66f75a5d028beaf67c931435fdc3e7823125730c
mtd: fix oops in dataflash driver

I'm seeing an oops in mtd_dataflash.c with Linux 3.3. What appears to
be happening is that otp_select_filemode calls mtd_read_fact_prot_reg
with -1 for offset and length and a NULL buffer to test if OTP
operations are supported. This finds its way down to otp_read in
mtd_dataflash.c and causes an oops when memcpying the returned data
into the NULL buf.

None of the checks in otp_read catches the negative length and offset.
Changing the length of the dummy read to 0 prevents the oops.

Cc: stable@kernel.org [3.3+]
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
drivers/mtd/mtdchar.c