|
Thomas Zimmermann |
dfae32 |
From 58c0cc2d90f1e37c4eb63ae7f164c83830833f78 Mon Sep 17 00:00:00 2001
|
|
Thomas Zimmermann |
dfae32 |
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
|
Thomas Zimmermann |
dfae32 |
Date: Mon, 3 May 2021 13:57:05 +0200
|
|
Thomas Zimmermann |
dfae32 |
Subject: Revert "video: hgafb: fix potential NULL pointer dereference"
|
|
Thomas Zimmermann |
dfae32 |
Git-commit: 58c0cc2d90f1e37c4eb63ae7f164c83830833f78
|
|
Thomas Zimmermann |
dfae32 |
Patch-mainline: v5.13-rc3
|
|
Thomas Zimmermann |
dfae32 |
References: bsc#1129770
|
|
Thomas Zimmermann |
dfae32 |
|
|
Thomas Zimmermann |
dfae32 |
This reverts commit ec7f6aad57ad29e4e66cc2e18e1e1599ddb02542.
|
|
Thomas Zimmermann |
dfae32 |
|
|
Thomas Zimmermann |
dfae32 |
Because of recent interactions with developers from @umn.edu, all
|
|
Thomas Zimmermann |
dfae32 |
commits from them have been recently re-reviewed to ensure if they were
|
|
Thomas Zimmermann |
dfae32 |
correct or not.
|
|
Thomas Zimmermann |
dfae32 |
|
|
Thomas Zimmermann |
dfae32 |
Upon review, this commit was found to be incorrect for the reasons
|
|
Thomas Zimmermann |
dfae32 |
below, so it must be reverted. It will be fixed up "correctly" in a
|
|
Thomas Zimmermann |
dfae32 |
later kernel change.
|
|
Thomas Zimmermann |
dfae32 |
|
|
Thomas Zimmermann |
dfae32 |
This patch "looks" correct, but the driver keeps on running and will
|
|
Thomas Zimmermann |
dfae32 |
fail horribly right afterward if this error condition ever trips.
|
|
Thomas Zimmermann |
dfae32 |
|
|
Thomas Zimmermann |
dfae32 |
So points for trying to resolve an issue, but a huge NEGATIVE value for
|
|
Thomas Zimmermann |
dfae32 |
providing a "fake" fix for the problem as nothing actually got resolved
|
|
Thomas Zimmermann |
dfae32 |
at all. I'll go fix this up properly...
|
|
Thomas Zimmermann |
dfae32 |
|
|
Thomas Zimmermann |
dfae32 |
Cc: Kangjie Lu <kjlu@umn.edu>
|
|
Thomas Zimmermann |
dfae32 |
Cc: Aditya Pakki <pakki001@umn.edu>
|
|
Thomas Zimmermann |
dfae32 |
Cc: Ferenc Bakonyi <fero@drama.obuda.kando.hu>
|
|
Thomas Zimmermann |
dfae32 |
Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
|
|
Thomas Zimmermann |
dfae32 |
Fixes: ec7f6aad57ad ("video: hgafb: fix potential NULL pointer dereference")
|
|
Thomas Zimmermann |
dfae32 |
Cc: stable <stable@vger.kernel.org>
|
|
Thomas Zimmermann |
dfae32 |
Link: https://lore.kernel.org/r/20210503115736.2104747-39-gregkh@linuxfoundation.org
|
|
Thomas Zimmermann |
dfae32 |
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
|
Thomas Zimmermann |
dfae32 |
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
|
|
Thomas Zimmermann |
dfae32 |
---
|
|
Thomas Zimmermann |
dfae32 |
drivers/video/fbdev/hgafb.c | 2 --
|
|
Thomas Zimmermann |
dfae32 |
1 file changed, 2 deletions(-)
|
|
Thomas Zimmermann |
dfae32 |
|
|
Thomas Zimmermann |
dfae32 |
diff --git a/drivers/video/fbdev/hgafb.c b/drivers/video/fbdev/hgafb.c
|
|
Thomas Zimmermann |
dfae32 |
index 8bbac7182ad3..fca29f219f8b 100644
|
|
Thomas Zimmermann |
dfae32 |
--- a/drivers/video/fbdev/hgafb.c
|
|
Thomas Zimmermann |
dfae32 |
+++ b/drivers/video/fbdev/hgafb.c
|
|
Thomas Zimmermann |
dfae32 |
@@ -285,8 +285,6 @@ static int hga_card_detect(void)
|
|
Thomas Zimmermann |
dfae32 |
hga_vram_len = 0x08000;
|
|
Thomas Zimmermann |
dfae32 |
|
|
Thomas Zimmermann |
dfae32 |
hga_vram = ioremap(0xb0000, hga_vram_len);
|
|
Thomas Zimmermann |
dfae32 |
- if (!hga_vram)
|
|
Thomas Zimmermann |
dfae32 |
- goto error;
|
|
Thomas Zimmermann |
dfae32 |
|
|
Thomas Zimmermann |
dfae32 |
if (request_region(0x3b0, 12, "hgafb"))
|
|
Thomas Zimmermann |
dfae32 |
release_io_ports = 1;
|
|
Thomas Zimmermann |
dfae32 |
--
|
|
Thomas Zimmermann |
dfae32 |
2.37.3
|
|
Thomas Zimmermann |
dfae32 |
|