i965: fix problem with constant out of bounds access (v2)
authorDave Airlie <airlied@redhat.com>
Wed, 29 May 2013 23:51:41 +0000 (09:51 +1000)
committerDave Airlie <airlied@redhat.com>
Thu, 30 May 2013 02:59:34 +0000 (12:59 +1000)
commit98dfd59a0445666060c97b0dccaf0e9f030b547a
tree2e40353b8096a048e14cb4d1ade37738623b3603
parent02fe736cc0e6866daa50aaae1ed7b977522eaf65
i965: fix problem with constant out of bounds access (v2)

This is my attempt at fixing this as the CVE is making RH security team
care enough to make me look at this. (please upstream, security fixes are
more important than whatever else you are doing, if for no other reason than
it saves me having to fix stuff I've no real clue about).

Since Frank's original fix was denied, here is my attempt to just
alias all constants that are out of bounds < 0 or > nr_params to constant 0,
hopefully this provides the undefined behaviour idr requires..

CVE-2013-1872

v2: drop the last hunk which was a separate fix (now in master).
hopefully fix the indentations.

NOTE: This is a candidate for stable branches.
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Signed-off-by: Dave Airlie <airlied@redhat.com>
src/mesa/drivers/dri/i965/brw_fs.cpp