From 10da0578c1f69a030b9ad96b4a07e92329bda55c Mon Sep 17 00:00:00 2001 From: Tanu Kaskinen Date: Tue, 12 Aug 2014 12:48:22 +0300 Subject: [PATCH] device-manager: Don't allow setting an empty description in the client API The module implementation treats an empty description as a protocol error, so we should validate the description already at client side. --- src/pulse/ext-device-manager.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/pulse/ext-device-manager.c b/src/pulse/ext-device-manager.c index 7fbb7de23..915806804 100644 --- a/src/pulse/ext-device-manager.c +++ b/src/pulse/ext-device-manager.c @@ -225,6 +225,7 @@ pa_operation *pa_ext_device_manager_set_device_description( PA_CHECK_VALIDITY_RETURN_NULL(c, !pa_detect_fork(), PA_ERR_FORKED); PA_CHECK_VALIDITY_RETURN_NULL(c, c->state == PA_CONTEXT_READY, PA_ERR_BADSTATE); PA_CHECK_VALIDITY_RETURN_NULL(c, c->version >= 14, PA_ERR_NOTSUPPORTED); + PA_CHECK_VALIDITY_RETURN_NULL(c, *description, PA_ERR_INVALID); o = pa_operation_new(c, NULL, (pa_operation_cb_t) cb, userdata);