diff options
| author | Dan Carpenter <[email protected]> | 2016-04-25 09:04:54 +0000 |
|---|---|---|
| committer | Boris Brezillon <[email protected]> | 2016-06-01 13:59:36 +0000 |
| commit | 58a2ab3af722550b2e4e8155eb08660e16c20ee6 (patch) | |
| tree | 5685d16d65ee605b438678179238303128a964da | |
| parent | drm: atmel-hlcdc: fix atmel_hlcdc_crtc_reset() implementation (diff) | |
| download | kernel-58a2ab3af722550b2e4e8155eb08660e16c20ee6.tar.gz kernel-58a2ab3af722550b2e4e8155eb08660e16c20ee6.zip | |
drm: atmel-hlcdc: fix a NULL check
If kmalloc() returned NULL we would end up dereferencing "state" a
couple lines later.
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Boris Brezillon <[email protected]>
| -rw-r--r-- | drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c index ac4e0632c2de..bd12231ab0cd 100644 --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c @@ -414,8 +414,9 @@ atmel_hlcdc_crtc_duplicate_state(struct drm_crtc *crtc) return NULL; state = kmalloc(sizeof(*state), GFP_KERNEL); - if (state) - __drm_atomic_helper_crtc_duplicate_state(crtc, &state->base); + if (!state) + return NULL; + __drm_atomic_helper_crtc_duplicate_state(crtc, &state->base); cur = drm_crtc_state_to_atmel_hlcdc_crtc_state(crtc->state); state->output_mode = cur->output_mode; |