diff options
| author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2018-03-12 17:06:54 +0100 |
|---|---|---|
| committer | Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> | 2018-03-12 17:06:54 +0100 |
| commit | df7a84a8baf1caf01ded61356b68d5d978b37789 (patch) | |
| tree | 64172984762db5d8b70bb0368452ddac45d09e62 /drivers/video/fbdev/vermilion/vermilion.c | |
| parent | a2a348014aad8bdf1466e027aa1dad2f099b7de6 (diff) | |
| download | linux-stable-df7a84a8baf1caf01ded61356b68d5d978b37789.tar.gz linux-stable-df7a84a8baf1caf01ded61356b68d5d978b37789.tar.bz2 linux-stable-df7a84a8baf1caf01ded61356b68d5d978b37789.zip | |
video: fbdev: vermilion: use 64-bit arithmetic instead of 32-bit
Cast _pitch_ to u64 in order to give the compiler complete information
about the proper arithmetic to use. Notice that this variable is
being used in a context that expects an expression of type u64
(64 bits, unsigned).
The expression pitch * var->yres_virtual is currently being evaluated
using 32-bit arithmetic and the result of the operation is being stored
into variable mem, which is a variable of type u64. Based on that,
chances are there is a potential integer overflow as a result of the
operation.
Addresses-Coverity-ID: 200655 ("Unintentional integer overflow")
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Diffstat (limited to 'drivers/video/fbdev/vermilion/vermilion.c')
| -rw-r--r-- | drivers/video/fbdev/vermilion/vermilion.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/video/fbdev/vermilion/vermilion.c b/drivers/video/fbdev/vermilion/vermilion.c index 6f8d444eb0e3..5172fa581147 100644 --- a/drivers/video/fbdev/vermilion/vermilion.c +++ b/drivers/video/fbdev/vermilion/vermilion.c @@ -651,7 +651,7 @@ static int vmlfb_check_var_locked(struct fb_var_screeninfo *var, } pitch = ALIGN((var->xres * var->bits_per_pixel) >> 3, 0x40); - mem = pitch * var->yres_virtual; + mem = (u64)pitch * var->yres_virtual; if (mem > vinfo->vram_contig_size) { return -ENOMEM; } |