drm/dp/mst: fix a possible memory leak in fetch_monitor_name()

HBh25Y · Lyude · commit 6e03b13cc7d9 · 2022-05-17T15:56:18.000-04:00
drm_dp_mst_get_edid call kmemdup to create mst_edid. So mst_edid need to be freed after use. Signed-off-by: Hangyu Hua <hbh25y@gmail.com> Reviewed-by: Lyude Paul <lyude@redhat.com> Signed-off-by: Lyude Paul <lyude@redhat.com> Cc: stable@vger.kernel.org Link: https://patchwork.freedesktop.org/patch/msgid/20220516032042.13166-1-hbh25y@gmail.com
diff --git a/drivers/gpu/drm/dp/drm_dp_mst_topology.c b/drivers/gpu/drm/dp/drm_dp_mst_topology.c
@@ -4852,6 +4852,7 @@ static void fetch_monitor_name(struct drm_dp_mst_topology_mgr *mgr,
 
 	mst_edid = drm_dp_mst_get_edid(port->connector, mgr, port);
 	drm_edid_get_monitor_name(mst_edid, name, namelen);
+	kfree(mst_edid);
 }
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -4852,6 +4852,7 @@ static void fetch_monitor_name(struct drm_dp_mst_topology_mgr *mgr,`
`4852`	`4852`
`4853`	`4853`	`mst_edid = drm_dp_mst_get_edid(port->connector, mgr, port);`
`4854`	`4854`	`drm_edid_get_monitor_name(mst_edid, name, namelen);`
	`4855`	`+ kfree(mst_edid);`
`4855`	`4856`	`}`
`4856`	`4857`
`4857`	`4858`	`/**`