• Dan Carpenter's avatar
    genwqe: Prevent an integer overflow in the ioctl · 06666ce1
    Dan Carpenter authored
    commit 110080ce upstream.
    
    There are a couple potential integer overflows here.
    
    	round_up(m->size + (m->addr & ~PAGE_MASK), PAGE_SIZE);
    
    The first thing is that the "m->size + (...)" addition could overflow,
    and the second is that round_up() overflows to zero if the result is
    within PAGE_SIZE of the type max.
    
    In this code, the "m->size" variable is an u64 but we're saving the
    result in "map_size" which is an unsigned long and genwqe_user_vmap()
    takes an unsigned long as well.  So I have used ULONG_MAX as the upper
    bound.  From a practical perspective unsigned long is fine/better than
    trying to change all the types to u64.
    
    Fixes: eaf4722d ("GenWQE Character device and DDCB queue")
    Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
    Cc: stable <stable@vger.kernel.org>
    Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
    06666ce1
card_dev.c 34.3 KB