library: Drop dead code in subsession_get_current_user() 42/279342/1
authorKarol Lewandowski <k.lewandowsk@samsung.com>
Sat, 6 Aug 2022 13:00:30 +0000 (15:00 +0200)
committerKarol Lewandowski <k.lewandowsk@samsung.com>
Sat, 6 Aug 2022 13:00:30 +0000 (15:00 +0200)
We don't need to explicitly check type of returned gvariant
as this will be handled by g_dbus_method_call_sync() (as invoked
via method_call_sync()).

Change-Id: I5327b670ec002f3be00e8fe40e8ba20314ccf3fe

libsessiond/src/lib.c

index b2612da..01834cf 100644 (file)
@@ -891,11 +891,7 @@ EXPORT_API int subsession_get_current_user(int session_uid, subsession_user_t us
                return ret;
 
        g_autofree char *get_user = NULL;
-       if (!typechecked_g_variant_get(out, "(s)", &get_user) || !get_user) {
-               LOGE("DBus method %s returned incorrect %s", dbus_method_call.GetCurrentUser, !get_user ? "value" : "type");
-               return SUBSESSION_ERROR_IO_ERROR;
-       }
-
+       g_variant_get(out, "(s)", &get_user);
        subsession_user_copy(user, get_user);
 
        return SUBSESSION_ERROR_NONE;