[ostree] gpg: Fix _ostree_gpg_verifier_add_keyring()



commit f162c0b6d6c6d993bd6d2e3a038c836a6244414b
Author: Matthew Barnes <mbarnes redhat com>
Date:   Fri May 1 15:19:20 2015 -0400

    gpg: Fix _ostree_gpg_verifier_add_keyring()
    
    The function never fails, but its API makes it look like it can.
    
    Fortunately it's private, so just fix it.

 src/libostree/ostree-gpg-verifier.c |    9 +++------
 src/libostree/ostree-gpg-verifier.h |    7 +++----
 src/libostree/ostree-repo.c         |    4 +---
 3 files changed, 7 insertions(+), 13 deletions(-)
---
diff --git a/src/libostree/ostree-gpg-verifier.c b/src/libostree/ostree-gpg-verifier.c
index 96ca7c6..42581b3 100644
--- a/src/libostree/ostree-gpg-verifier.c
+++ b/src/libostree/ostree-gpg-verifier.c
@@ -259,16 +259,13 @@ out:
   return result;
 }
 
-gboolean
+void
 _ostree_gpg_verifier_add_keyring (OstreeGpgVerifier  *self,
-                                  GFile              *path,
-                                  GCancellable       *cancellable,
-                                  GError            **error)
+                                  GFile              *path)
 {
-  g_return_val_if_fail (path != NULL, FALSE);
+  g_return_if_fail (G_IS_FILE (path));
 
   self->keyrings = g_list_append (self->keyrings, g_object_ref (path));
-  return TRUE;
 }
 
 gboolean
diff --git a/src/libostree/ostree-gpg-verifier.h b/src/libostree/ostree-gpg-verifier.h
index b505d85..54be424 100644
--- a/src/libostree/ostree-gpg-verifier.h
+++ b/src/libostree/ostree-gpg-verifier.h
@@ -51,8 +51,7 @@ gboolean      _ostree_gpg_verifier_add_keyring_dir (OstreeGpgVerifier   *self,
                                                     GCancellable        *cancellable,
                                                     GError             **error);
 
-gboolean      _ostree_gpg_verifier_add_keyring (OstreeGpgVerifier   *self,
-                                                GFile               *path,
-                                                GCancellable        *cancellable,
-                                                GError             **error);
+void _ostree_gpg_verifier_add_keyring (OstreeGpgVerifier *self,
+                                       GFile             *path);
+
 G_END_DECLS
diff --git a/src/libostree/ostree-repo.c b/src/libostree/ostree-repo.c
index 5bdba3c..1ec18af 100644
--- a/src/libostree/ostree-repo.c
+++ b/src/libostree/ostree-repo.c
@@ -3376,9 +3376,7 @@ _ostree_repo_gpg_verify_with_metadata (OstreeRepo          *self,
     }
   if (extra_keyring != NULL)
     {
-      if (!_ostree_gpg_verifier_add_keyring (verifier, extra_keyring,
-                                             cancellable, error))
-        goto out;
+      _ostree_gpg_verifier_add_keyring (verifier, extra_keyring);
     }
 
   if (metadata)


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]