Avoiding redundant check

https://bugs.freedesktop.org/show_bug.cgi?id=93587

diff --git a/p11-kit/modules.c b/p11-kit/modules.c
index acb3707c641ae1713a4dc4d72ded51a00d6cdafd..63baeb1904de8da94e9fbbc3cc06b218a4fa0c57 100644 (file)
--- a/p11-kit/modules.c
+++ b/p11-kit/modules.c
@@ -2326,17 +2326,15 @@ p11_module_load_inlock_reentrant (CK_FUNCTION_LIST *module,
                }
 
                /* If this was newly allocated, add it to the list */
-               if (rv == CKR_OK && allocated) {
+               if (allocated) {
                        if (!p11_dict_set (gl.modules, allocated, allocated) ||
                            !p11_dict_set (gl.unmanaged_by_funcs, module, allocated))
                                return_val_if_reached (CKR_HOST_MEMORY);
                        allocated = NULL;
                }
 
-               if (rv == CKR_OK) {
                        /* WARNING: Reentrancy can occur here */
                        rv = prepare_module_inlock_reentrant (mod, flags, result);
-               }
 
                free (allocated);
        }