We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 4513 - [PATCH] wayland: add dummy remove handler to wl_registry_listener
Summary: [PATCH] wayland: add dummy remove handler to wl_registry_listener
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: video (show other bugs)
Version: HG 2.0
Hardware: All Linux
: P2 normal
Assignee: Sam Lantinga
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-02-20 02:41 UTC by Sebastian Krzyszkowiak
Modified: 2019-03-12 19:05 UTC (History)
1 user (show)

See Also:

Attachments
0001-wayland-add-dummy-remove-handler-to-wl_registry_list.patch (1.18 KB, patch)
2019-02-20 02:41 UTC, Sebastian Krzyszkowiak 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sebastian Krzyszkowiak 2019-02-20 02:41:36 UTC 
Created attachment 3639 [details]
0001-wayland-add-dummy-remove-handler-to-wl_registry_list.patch

Passing NULL there causes SDL to crash every time the compositor sends a remove event - which is particularly unhelpful as the protocol allows to simply ignore the event, because removed objects can be still safely called on the client side and compositor will ignore such requests.
Comment 1 Sylvain 2019-03-12 19:05:30 UTC 
Fixed in https://hg.libsdl.org/SDL/rev/5b8e7e5bda1c Thanks !