blankie
/
wlroots
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

backend/drm: fix memory leak in realloc crtcs 

If *changed_outputs is not supplied by the calling function, track the local
allocation with a bool variable and free the allocation at the end of the
function.
This commit is contained in:
Rouven Czerwinski 2019-02-21 08:13:04 +01:00
parent 18600f457b
commit e1834ace28
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
backend/drm/drm.c
View File

@ -888,8 +888,9 @@ static void dealloc_crtc(struct wlr_drm_connector *conn) {
static void realloc_crtcs(struct wlr_drm_backend *drm, bool *changed_outputs) { static void realloc_crtcs(struct wlr_drm_backend *drm, bool *changed_outputs) {
size_t num_outputs = wl_list_length(&drm->outputs); size_t num_outputs = wl_list_length(&drm->outputs);
bool changed_local = changed_outputs ? false : true;
if (changed_outputs == NULL) { if (changed_local) {
changed_outputs = calloc(num_outputs, sizeof(bool)); changed_outputs = calloc(num_outputs, sizeof(bool));
if (changed_outputs == NULL) { if (changed_outputs == NULL) {
wlr_log(WLR_ERROR, "Allocation failed"); wlr_log(WLR_ERROR, "Allocation failed");
@ -952,7 +953,7 @@ static void realloc_crtcs(struct wlr_drm_backend *drm, bool *changed_outputs) {
connectors[crtc[i]]->desired_enabled) { connectors[crtc[i]]->desired_enabled) {
wlr_log(WLR_DEBUG, "Could not match a CRTC for connected output %d", wlr_log(WLR_DEBUG, "Could not match a CRTC for connected output %d",
crtc[i]); crtc[i]);
return; goto free_changed_outputs;
} }
} }
@ -1021,6 +1022,11 @@ static void realloc_crtcs(struct wlr_drm_backend *drm, bool *changed_outputs) {
wlr_output_damage_whole(&conn->output); wlr_output_damage_whole(&conn->output);
} }
free_changed_outputs:
if (changed_local) {
free(changed_outputs);
}
} }
static uint32_t get_possible_crtcs(int fd, drmModeRes *res, static uint32_t get_possible_crtcs(int fd, drmModeRes *res,