diff options
| author | Trent Piepho <xyzzy@speakeasy.org> | 2009-01-28 16:32:58 -0800 |
|---|---|---|
| committer | Trent Piepho <xyzzy@speakeasy.org> | 2009-01-28 16:32:58 -0800 |
| commit | 562378e1f982f3a817c81594e61d40ffa05fac7d (patch) | |
| tree | 29717f3f822e1720a6e922418392c364c0991839 /mailimport | |
| parent | 471270c843cfbff8d1b8944c0c54850c121dd6e2 (diff) | |
| download | mediapointer-dvb-s2-562378e1f982f3a817c81594e61d40ffa05fac7d.tar.gz mediapointer-dvb-s2-562378e1f982f3a817c81594e61d40ffa05fac7d.tar.bz2 | |
bttv: norm value should be unsigned
From: Trent Piepho <xyzzy@speakeasy.org>
The norm value in the driver is an index into an array and the the driver
doesn't allow it to be negative or otherwise invalid. It should be
unsigned but wasn't in all places.
Fix some structs and functions to have the norm be unsigned. Get rid of
useless checks for "< 0". Most of the driver code can't handle a norm
value that's out of range, so change some ">= BTTV_TVNORMS" checks to
BUG_ON(). There's no point in silently ignoring invalid driver state just
to crash because of it later.
Priority: normal
Reported-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Trent Piepho <xyzzy@speakeasy.org>
Diffstat (limited to 'mailimport')
0 files changed, 0 insertions, 0 deletions