drm/tegra: Zero-initialize iosys_map

cyndis · thierryreding · commit 3868ff006b57 · 2023-10-11T22:52:44.000+02:00
UBSAN reports an invalid load for bool, as the iosys_map is read later without being initialized. Zero-initialize it to avoid this. Reported-by: Ashish Mhetre <amhetre@nvidia.com> Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com> Signed-off-by: Thierry Reding <treding@nvidia.com> Link: https://patchwork.freedesktop.org/patch/msgid/20230901115910.701518-2-cyndis@kapsi.fi
diff --git a/drivers/gpu/drm/tegra/gem.c b/drivers/gpu/drm/tegra/gem.c
@@ -177,7 +177,7 @@ static void tegra_bo_unpin(struct host1x_bo_mapping *map)
 static void *tegra_bo_mmap(struct host1x_bo *bo)
 {
 	struct tegra_bo *obj = host1x_to_tegra_bo(bo);
-	struct iosys_map map;
+	struct iosys_map map = { 0 };
 	int ret;
 
 	if (obj->vaddr)

Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,7 @@ static void tegra_bo_unpin(struct host1x_bo_mapping *map)`
`177`	`177`	`static void tegra_bo_mmap(struct host1x_bo bo)`
`178`	`178`	`{`
`179`	`179`	`struct tegra_bo *obj = host1x_to_tegra_bo(bo);`
`180`		`- struct iosys_map map;`
	`180`	`+ struct iosys_map map = { 0 };`
`181`	`181`	`int ret;`
`182`	`182`
`183`	`183`	`if (obj->vaddr)`