sign-dummy: optimize ostree_sign_dummy_data_verify
authorDenis Pynkin <denis.pynkin@collabora.com>
Mon, 17 Feb 2020 21:46:51 +0000 (00:46 +0300)
committerDenis Pynkin <denis.pynkin@collabora.com>
Wed, 25 Mar 2020 12:23:55 +0000 (15:23 +0300)
Return TRUE as soon as any signature verified.

Signed-off-by: Denis Pynkin <denis.pynkin@collabora.com>
src/libostree/ostree-sign-dummy.c

index b9d8abf488df863ccace62782c8b50f49a712e66..722d461b0e36de4f4ccf1b126a69dd3f4fb86780 100644 (file)
@@ -156,7 +156,6 @@ gboolean ostree_sign_dummy_data_verify (OstreeSign *self,
   if (!g_variant_is_of_type (signatures, (GVariantType *) OSTREE_SIGN_METADATA_DUMMY_TYPE))
     return glnx_throw (error, "signature: dummy: wrong type passed for verification");
 
-  gboolean verified = FALSE;
   for (gsize i = 0; i < g_variant_n_children(signatures); i++)
     {
       g_autoptr (GVariant) child = g_variant_get_child_value (signatures, i);
@@ -169,12 +168,10 @@ gboolean ostree_sign_dummy_data_verify (OstreeSign *self,
       g_debug("Stored signature %d: %s", (gint)i, sign->pk_ascii);
 
       if (!g_strcmp0(sign_ascii, sign->pk_ascii))
-        verified = TRUE;
+        return TRUE;
       else
         return glnx_throw (error, "signature: dummy: incorrect signature %" G_GSIZE_FORMAT, i);
     }
-  if (!verified)
-    return glnx_throw (error, "signature: dummy: no signatures");
 
-  return TRUE;
+  return glnx_throw (error, "signature: dummy: no signatures");
 }