xwayland: Always free reply from xcb_get_property_reply()
Signed-off-by: Bryce Harrington <bryce@osg.samsung.com> Reviewed-by: Marek Chalupa <mchqwerty@gmail.com>
This commit is contained in:
@@ -119,8 +119,9 @@ weston_wm_get_incr_chunk(struct weston_wm *wm)
|
|||||||
} else {
|
} else {
|
||||||
weston_log("transfer complete\n");
|
weston_log("transfer complete\n");
|
||||||
close(wm->data_source_fd);
|
close(wm->data_source_fd);
|
||||||
free(reply);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
free(reply);
|
||||||
}
|
}
|
||||||
|
|
||||||
struct x11_data_source {
|
struct x11_data_source {
|
||||||
@@ -192,8 +193,10 @@ weston_wm_get_selection_targets(struct weston_wm *wm)
|
|||||||
}
|
}
|
||||||
|
|
||||||
source = malloc(sizeof *source);
|
source = malloc(sizeof *source);
|
||||||
if (source == NULL)
|
if (source == NULL) {
|
||||||
|
free(reply);
|
||||||
return;
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
wl_signal_init(&source->base.destroy_signal);
|
wl_signal_init(&source->base.destroy_signal);
|
||||||
source->base.accept = data_source_accept;
|
source->base.accept = data_source_accept;
|
||||||
@@ -237,12 +240,13 @@ weston_wm_get_selection_data(struct weston_wm *wm)
|
|||||||
if (reply->type == wm->atom.incr) {
|
if (reply->type == wm->atom.incr) {
|
||||||
dump_property(wm, wm->atom.wl_selection, reply);
|
dump_property(wm, wm->atom.wl_selection, reply);
|
||||||
wm->incr = 1;
|
wm->incr = 1;
|
||||||
free(reply);
|
|
||||||
} else {
|
} else {
|
||||||
dump_property(wm, wm->atom.wl_selection, reply);
|
dump_property(wm, wm->atom.wl_selection, reply);
|
||||||
wm->incr = 0;
|
wm->incr = 0;
|
||||||
weston_wm_write_property(wm, reply);
|
weston_wm_write_property(wm, reply);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
free(reply);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void
|
static void
|
||||||
|
|||||||
Reference in New Issue
Block a user