Commit 0c641bff authored by Rickard Strandqvist's avatar Rickard Strandqvist Committed by Tomi Valkeinen

video: fbdev: s3fb.c: Fix for possible null pointer dereference

There is otherwise a risk of a possible null pointer dereference.

Was largely found by using a static code analysis program called cppcheck.
Signed-off-by: default avatarRickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: default avatarTomi Valkeinen <tomi.valkeinen@ti.com>
parent d9a5f87c
...@@ -1401,9 +1401,10 @@ static int s3_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) ...@@ -1401,9 +1401,10 @@ static int s3_pci_probe(struct pci_dev *dev, const struct pci_device_id *id)
static void s3_pci_remove(struct pci_dev *dev) static void s3_pci_remove(struct pci_dev *dev)
{ {
struct fb_info *info = pci_get_drvdata(dev); struct fb_info *info = pci_get_drvdata(dev);
struct s3fb_info __maybe_unused *par = info->par; struct s3fb_info __maybe_unused *par;
if (info) { if (info) {
par = info->par;
#ifdef CONFIG_MTRR #ifdef CONFIG_MTRR
if (par->mtrr_reg >= 0) { if (par->mtrr_reg >= 0) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment