winrt: Avoid unnecessary asserts in LowEnergyController

We do not want to assert when a user tries to write an invalid value.
Setting the correct error and returning early is the proper way to
handle this situation.

Change-Id: Ia358023ed186bc4b7c5be0e6319182dbe563f576
Reviewed-by: Maurice Kalinowski <maurice.kalinowski@qt.io>

1 files changed, 3 insertions, 1 deletions

diff --git a/src/bluetooth/qlowenergycontroller_winrt.cpp b/src/bluetooth/qlowenergycontroller_winrt.cpp
index 868cadb3..10eef246 100644
--- a/src/bluetooth/qlowenergycontroller_winrt.cpp
+++ b/src/bluetooth/qlowenergycontroller_winrt.cpp
@@ -968,7 +968,9 @@ void QLowEnergyControllerPrivate::writeDescriptor(
             } else if (intValue == 0) {
                 value = GattClientCharacteristicConfigurationDescriptorValue_None;
             } else {
-                Q_ASSERT(false);
+                qCDebug(QT_BT_WINRT) << "Descriptor" << descHandle << "write operation failed: Invalid value";
+                service->setError(QLowEnergyService::DescriptorWriteError);
+                return S_OK;
             }
             ComPtr<IAsyncOperation<enum GattCommunicationStatus>> writeOp;
             HRESULT hr = characteristic->WriteClientCharacteristicConfigurationDescriptorAsync(value, &writeOp);