video: mb862xxfb: prevent divide by zero bug
Do a sanity check on these before using them as divisors. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Acked-by: Anatolij Gustschin <agust@denx.de> Signed-off-by: Florian Tobias Schandinat <FlorianSchandinat@gmx.de>
This commit is contained in:
committed by
Florian Tobias Schandinat
parent
01817d194a
commit
2568236256
@@ -328,6 +328,8 @@ static int mb862xxfb_ioctl(struct fb_info *fbi, unsigned int cmd,
|
|||||||
case MB862XX_L1_SET_CFG:
|
case MB862XX_L1_SET_CFG:
|
||||||
if (copy_from_user(l1_cfg, argp, sizeof(*l1_cfg)))
|
if (copy_from_user(l1_cfg, argp, sizeof(*l1_cfg)))
|
||||||
return -EFAULT;
|
return -EFAULT;
|
||||||
|
if (l1_cfg->dh == 0 || l1_cfg->dw == 0)
|
||||||
|
return -EINVAL;
|
||||||
if ((l1_cfg->sw >= l1_cfg->dw) && (l1_cfg->sh >= l1_cfg->dh)) {
|
if ((l1_cfg->sw >= l1_cfg->dw) && (l1_cfg->sh >= l1_cfg->dh)) {
|
||||||
/* downscaling */
|
/* downscaling */
|
||||||
outreg(cap, GC_CAP_CSC,
|
outreg(cap, GC_CAP_CSC,
|
||||||
|
Reference in New Issue
Block a user