diff options
| author | Tanu Kaskinen <tanu.kaskinen@linux.intel.com> | 2014-08-12 12:48:22 +0300 |
|---|---|---|
| committer | Tanu Kaskinen <tanu.kaskinen@linux.intel.com> | 2014-08-24 12:43:11 +0300 |
| commit | 10da0578c1f69a030b9ad96b4a07e92329bda55c (patch) | |
| tree | 426cf2e5b9e9aafba781d9b1f2c89a29e90f0d40 /src | |
| parent | 638d0a51e2217405b6fbfb33fe67b2399a767fbc (diff) | |
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.
Diffstat (limited to 'src')
| -rw-r--r-- | src/pulse/ext-device-manager.c | 1 |
1 files changed, 1 insertions, 0 deletions
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); |