Re: [PATCH] m501fb: don't return zero on failure path in sm501fb_start()

2017-05-02 Thread Bartlomiej Zolnierkiewicz
On Saturday, April 29, 2017 09:33:39 PM Alexey Khoroshilov wrote: > If fbmem iomemory mapping failed, sm501fb_start() breaks off > initialization, deallocates resources, but returns zero. > As a result, double deallocation can happen in sm501fb_stop(). > > Found by Linux Driver Verification projec

[PATCH] m501fb: don't return zero on failure path in sm501fb_start()

2017-04-29 Thread Alexey Khoroshilov
If fbmem iomemory mapping failed, sm501fb_start() breaks off initialization, deallocates resources, but returns zero. As a result, double deallocation can happen in sm501fb_stop(). Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshilov --- drivers/video