Merge request !1242 from 'supported_storage_changes_check'

iface-modem-messaging: Validate supported storages

https://gitlab.freedesktop.org/mobile-broadband/ModemManager/-/merge_requests/1242

1 files changed, 3 insertions, 5 deletions

diff --git a/src/mm-iface-modem-messaging.c b/src/mm-iface-modem-messaging.c
index c89477fb..a064f11e 100644
--- a/src/mm-iface-modem-messaging.c
+++ b/src/mm-iface-modem-messaging.c
@@ -428,11 +428,9 @@ handle_set_default_storage_auth_ready (MMBaseModem                    *self,
         return;
     }
 
-    if (ctx->storage == MM_SMS_STORAGE_UNKNOWN) {
-        mm_dbus_method_invocation_return_error (ctx->invocation,
-                                               MM_CORE_ERROR,
-                                               MM_CORE_ERROR_UNSUPPORTED,
-                                               "Cannot set default storage: Unknown storage");
+    /* Check if the requested storage is supported or not */
+    if (!mm_iface_modem_messaging_is_storage_supported_for_storing (ctx->self, ctx->storage, &error)) {
+        mm_dbus_method_invocation_take_error (ctx->invocation, error);
         handle_set_default_storage_context_free (ctx);
         return;
     }