zram: free secondary algorithms names

sergey-senozhatsky · gregkh · commit 6272936fd242 · 2024-10-17T15:24:14.000+02:00
[ Upstream commit 684826f ] We need to kfree() secondary algorithms names when reset zram device that had multi-streams, otherwise we leak memory. [senozhatsky@chromium.org: kfree(NULL) is legal] Link: https://lkml.kernel.org/r/20240917013021.868769-1-senozhatsky@chromium.org Link: https://lkml.kernel.org/r/20240911025600.3681789-1-senozhatsky@chromium.org Fixes: 001d927 ("zram: add recompression algorithm sysfs knob") Signed-off-by: Sergey Senozhatsky <senozhatsky@chromium.org> Cc: Minchan Kim <minchan@kernel.org> Cc: <stable@vger.kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
diff --git a/drivers/block/zram/zram_drv.c b/drivers/block/zram/zram_drv.c
@@ -1983,6 +1983,11 @@ static void zram_destroy_comps(struct zram *zram)
 		zcomp_destroy(comp);
 		zram->num_active_comps--;
 	}
+
+	for (prio = ZRAM_SECONDARY_COMP; prio < ZRAM_MAX_COMPS; prio++) {
+		kfree(zram->comp_algs[prio]);
+		zram->comp_algs[prio] = NULL;
+	}
 }
 
 static void zram_reset_device(struct zram *zram)

Original file line number	Diff line number	Diff line change
`@@ -1983,6 +1983,11 @@ static void zram_destroy_comps(struct zram *zram)`
`1983`	`1983`	`zcomp_destroy(comp);`
`1984`	`1984`	`zram->num_active_comps--;`
`1985`	`1985`	`}`
	`1986`	`+`
	`1987`	`+ for (prio = ZRAM_SECONDARY_COMP; prio < ZRAM_MAX_COMPS; prio++) {`
	`1988`	`+ kfree(zram->comp_algs[prio]);`
	`1989`	`+ zram->comp_algs[prio] = NULL;`
	`1990`	`+ }`
`1986`	`1991`	`}`
`1987`	`1992`
`1988`	`1993`	`static void zram_reset_device(struct zram *zram)`