mm: do_xip_mapping_read: fix length calculation
authorMartin Schwidefsky <schwidefsky@de.ibm.com>
Fri, 3 Apr 2009 04:35:12 +0000 (04:35 +0000)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sat, 2 May 2009 17:23:59 +0000 (10:23 -0700)
commite655a1eaa988bc55739f47d62ab0658c1364458a
tree6b793dbbdaaea766a29aa0d3de283bf661557dde
parent398e94c43b3339ce64f71d1ea827c3c4fed5ef1e
mm: do_xip_mapping_read: fix length calculation

upstream commit: 58984ce21d315b70df1a43644df7416ea7c9bfd8

The calculation of the value nr in do_xip_mapping_read is incorrect.  If
the copy required more than one iteration in the do while loop the copies
variable will be non-zero.  The maximum length that may be passed to the
call to copy_to_user(buf+copied, xip_mem+offset, nr) is len-copied but the
check only compares against (nr > len).

This bug is the cause for the heap corruption Carsten has been chasing
for so long:
mm/filemap_xip.c