diff options
| author | Jiasheng Jiang <[email protected]> | 2021-12-14 01:41:26 +0000 |
|---|---|---|
| committer | Thomas Zimmermann <[email protected]> | 2021-12-14 08:18:28 +0000 |
| commit | fea3fdf975dd9f3e5248afaab8fe023db313f005 (patch) | |
| tree | ac28b92c4b22b25f9fa222e1dc74628f8c56334f /drivers/gpu/drm/ast/ast_mode.c | |
| parent | drm: simpledrm: fix wrong unit with pixel clock (diff) | |
| download | kernel-fea3fdf975dd9f3e5248afaab8fe023db313f005.tar.gz kernel-fea3fdf975dd9f3e5248afaab8fe023db313f005.zip | |
drm/ast: potential dereference of null pointer
The return value of kzalloc() needs to be checked.
To avoid use of null pointer '&ast_state->base' in case of the
failure of alloc.
Fixes: f0adbc382b8b ("drm/ast: Allocate initial CRTC state of the correct size")
Signed-off-by: Jiasheng Jiang <[email protected]>
Signed-off-by: Thomas Zimmermann <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
Diffstat (limited to 'drivers/gpu/drm/ast/ast_mode.c')
| -rw-r--r-- | drivers/gpu/drm/ast/ast_mode.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c index 1e30eaeb0e1b..d5c98f79d58d 100644 --- a/drivers/gpu/drm/ast/ast_mode.c +++ b/drivers/gpu/drm/ast/ast_mode.c @@ -1121,7 +1121,10 @@ static void ast_crtc_reset(struct drm_crtc *crtc) if (crtc->state) crtc->funcs->atomic_destroy_state(crtc, crtc->state); - __drm_atomic_helper_crtc_reset(crtc, &ast_state->base); + if (ast_state) + __drm_atomic_helper_crtc_reset(crtc, &ast_state->base); + else + __drm_atomic_helper_crtc_reset(crtc, NULL); } static struct drm_crtc_state * |