Skip to content

Commit 146f278

Browse files
surajk8vsyrjala
authored andcommitted
drm/i915/display: Use intel_crtc_destroy_state instead kfree
intel_encoder_current_mode() seems to leak some resource because it uses kfree instead of intel_crtc_destroy_state let us fix that. Signed-off-by: Suraj Kandpal <[email protected]> Signed-off-by: Ville Syrjälä <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
1 parent 2250c7e commit 146f278

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/gpu/drm/i915/display/intel_display.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3999,7 +3999,7 @@ intel_encoder_current_mode(struct intel_encoder *encoder)
39993999
}
40004000

40014001
if (!intel_crtc_get_pipe_config(crtc_state)) {
4002-
kfree(crtc_state);
4002+
intel_crtc_destroy_state(&crtc->base, &crtc_state->uapi);
40034003
kfree(mode);
40044004
return NULL;
40054005
}
@@ -4008,7 +4008,7 @@ intel_encoder_current_mode(struct intel_encoder *encoder)
40084008

40094009
intel_mode_from_crtc_timings(mode, &crtc_state->hw.adjusted_mode);
40104010

4011-
kfree(crtc_state);
4011+
intel_crtc_destroy_state(&crtc->base, &crtc_state->uapi);
40124012

40134013
return mode;
40144014
}

0 commit comments

Comments
 (0)