On 05/02/2011 05:21 PM, Kirill A. Shutemov wrote: > On Fri, Apr 29, 2011 at 09:24:37PM -0400, Nitin Gupta wrote: >> Hi, >> >> On 04/28/2011 11:57 AM, Sergey Evlashev wrote: >>> From: Kirill A. Shutemov<[email protected]> >>> >>> reset_devices() uses in ramzswap_ioctl_init_device() error path. So we >>> have to check if rzs->table was really allocated before freeing it. >>> >>> Signed-off-by: Kirill A. Shutemov<[email protected]> >>> Signed-off-by: Sergey Evlashev<[email protected]> >>> Reported-by: Phil Carmody<[email protected]> >> >> >> Thanks for this fix but I'm no longer maintaining this old ramzswap >> code. Is there any reason why you can't switch to new zram code. > > zram has the same problem. See zram_reset_device(). >
zram_reset_device() is called only when zram->init_done flag is set and if this flag is set then we know that the table is already allocated. Thanks, Nitin _______________________________________________ linux-mm-cc mailing list [email protected] http://lists.laptop.org/listinfo/linux-mm-cc
