mgr/smb: fix a resource error unpacking str instead of list

Add special handling for the case where a string is passed instead of a
list. Without this fix a string will be converted into a list of single
letter items, something pretty much no one ever wants. Raise an
exception instead.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
(cherry picked from commit e760668778ddc8227d0c3aa7baa10ec39bb0b885)

diff --git a/src/pybind/mgr/smb/resourcelib.py b/src/pybind/mgr/smb/resourcelib.py
index 7128dba2673769173502b1fdf1981ff23a6c468a..d10b26941fa0e54115bcb89baa36a3ae5f6d50c9 100644 (file)
--- a/src/pybind/mgr/smb/resourcelib.py
+++ b/src/pybind/mgr/smb/resourcelib.py
@@ -390,6 +390,10 @@ class Resource:
             return _fs(value)
 
         if fld.takes(list):
+            if isinstance(value, str):
+                raise ResourceTypeError(
+                    f'{fld.name} expects a list not a string'
+                )
             subtype = fld.list_element_type()
             return [
                 self._object_sub_from_simplified(subtype, v) for v in value