data-device: check if there's a source before sending dnd_finish

This could cause a segfault in data_offer_destroy if the offer has version < 3.
This commit is contained in:
emersion 2019-02-24 09:24:58 +01:00 committed by Drew DeVault
parent 421283935b
commit d425206719
1 changed files with 5 additions and 5 deletions

View File

@ -96,7 +96,7 @@ static void data_offer_handle_receive(struct wl_client *client,
wlr_data_source_send(offer->source, mime_type, fd); wlr_data_source_send(offer->source, mime_type, fd);
} }
static void data_offer_dnd_finish(struct wlr_data_offer *offer) { static void data_offer_source_dnd_finish(struct wlr_data_offer *offer) {
struct wlr_data_source *source = offer->source; struct wlr_data_source *source = offer->source;
if (source->actions < 0) { if (source->actions < 0) {
return; return;
@ -142,7 +142,7 @@ static void data_offer_handle_finish(struct wl_client *client,
return; return;
} }
data_offer_dnd_finish(offer); data_offer_source_dnd_finish(offer);
data_offer_destroy(offer); data_offer_destroy(offer);
} }
@ -190,14 +190,14 @@ void data_offer_destroy(struct wlr_data_offer *offer) {
wl_list_remove(&offer->source_destroy.link); wl_list_remove(&offer->source_destroy.link);
wl_list_remove(&offer->link); wl_list_remove(&offer->link);
if (offer->type == WLR_DATA_OFFER_DRAG) { if (offer->type == WLR_DATA_OFFER_DRAG && offer->source) {
// If the drag destination has version < 3, wl_data_offer.finish // If the drag destination has version < 3, wl_data_offer.finish
// won't be called, so do this here as a safety net, because // won't be called, so do this here as a safety net, because
// we still want the version >= 3 drag source to be happy. // we still want the version >= 3 drag source to be happy.
if (wl_resource_get_version(offer->resource) < if (wl_resource_get_version(offer->resource) <
WL_DATA_OFFER_ACTION_SINCE_VERSION) { WL_DATA_OFFER_ACTION_SINCE_VERSION) {
data_offer_dnd_finish(offer); data_offer_source_dnd_finish(offer);
} else if (offer->source && offer->source->impl->dnd_finish) { } else if (offer->source->impl->dnd_finish) {
wlr_data_source_destroy(offer->source); wlr_data_source_destroy(offer->source);
} }
} }