# \[PATCH\] drivers/video/fbdev/core/fbmem.c: add pointer judgment

- Subsystems: FRAMEBUFFER LAYER, FRAMEBUFFER CORE, THE REST
- Maintainers: Helge Deller, Simona Vetter, Linus Torvalds
- List: linux-fbdev
- 2 messages, 2 authors
- Page: https://ratatoskr.run/linux-fbdev/2021/05/1383361/t

## \[PATCH\] drivers/video/fbdev/core/fbmem.c: add pointer judgment

- From: songqiang \<hidden\>
- Date: 2021-05-19 12:05:17 
- Message: https://ratatoskr.run/linux-fbdev/2021/05/1383361.md

```text
Signed-off-by: songqiang <redacted>
---
 drivers/video/fbdev/core/fbmem.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c
index 072780b0e570..6036ab849475 100644
--- a/drivers/video/fbdev/core/fbmem.c
+++ b/drivers/video/fbdev/core/fbmem.c
@@ -1859,6 +1859,9 @@ void fb_set_suspend(struct fb_info *info, int state)
 {
 	WARN_CONSOLE_UNLOCKED();
 
+	if (!info) {
+		return;
+	}
 	if (state) {
 		fbcon_suspended(info);
 		info->state = FBINFO_STATE_SUSPENDED;
-- 
2.20.1



```

## Re: \[PATCH\] drivers/video/fbdev/core/fbmem.c: add pointer judgment

- From: Matthew Wilcox \<willy@infradead.org\>
- Date: 2021-05-19 12:18:51 
- Message: https://ratatoskr.run/linux-fbdev/2021/05/1383370.md

```text
On Wed, May 19, 2021 at 08:00:28PM +0800, songqiang wrote:
> Signed-off-by: songqiang <redacted>
> ---

You need to explain:

 - Why you think this patch is needed
   - Did you observe a problem at runtime?
   - Is this the output from some checking tool?
 - Why this is the right way to address the problem

```