diff options
author | Eric Anholt <eric@anholt.net> | 2012-01-04 12:31:40 -0800 |
---|---|---|
committer | Eric Anholt <eric@anholt.net> | 2012-01-27 13:21:19 -0800 |
commit | 028715ee7074691895051296105e8b4cbbfaabb8 (patch) | |
tree | 8400a293600aabf1feb16203a13cf600e42cf9b4 /nouveau/nouveau_channel.c | |
parent | c1d2946da8f13ea8cbda1e33aeccee4d06015fea (diff) |
intel: Avoid the need for most overflow checks by using a scratch page.
The overflow checks were all thoroughly untested, and a bunch of the
ones I'm deleting were pretty broken. Now, in the case of overflow,
you just decode data of 0xd0d0d0d0, and instr_out prints the warning
message instead. Note that this still has the same issue of being
under-tested, but at least it's one place instead of per-packet.
A couple of BUFFER_FAIL uses are left where the length to be decoded
could be (significantly) larger than a page, and the decode didn't
just call instr_out (which doesn't dereference data itself unless it's
safe).
Diffstat (limited to 'nouveau/nouveau_channel.c')
0 files changed, 0 insertions, 0 deletions