[PATCH weston 4/4] screenshooter: Add a missing return; in an error path

Jasper St. Pierre jstpierre at mecheye.net
Thu Aug 7 13:43:14 PDT 2014


---
 src/screenshooter.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/screenshooter.c b/src/screenshooter.c
index 9ae0d29..4403933 100644
--- a/src/screenshooter.c
+++ b/src/screenshooter.c
@@ -265,6 +265,7 @@ bind_shooter(struct wl_client *client,
 	if (client != shooter->client) {
 		wl_resource_post_error(resource, WL_DISPLAY_ERROR_INVALID_OBJECT,
 				       "screenshooter failed: permission denied");
+		return;
 	}
 
 	wl_resource_set_implementation(resource, &screenshooter_implementation,
-- 
2.0.4



More information about the wayland-devel mailing list