From 3829489cd5f74f4b9f7e1567fee941123aa77987 Mon Sep 17 00:00:00 2001
From: Jakub Filak <jfilak@redhat.com>
Date: Wed, 27 Aug 2014 08:45:24 +0200
Subject: [PATCH 21/24] a-a-s-p-data: reduce amount of error messages

Read each GPG key only once. The GPG key dirs may contain many symlinks
and if their target cannot be read, then we print an error message for
every symlink pointing to the unreadable file. What's worse, the error
messages show a path to the target, so users see several identical
messages in the system logs.

Related to rhbz#1133674

Signed-off-by: Jakub Filak <jfilak@redhat.com>
---
 src/daemon/abrt-action-save-package-data.c | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/src/daemon/abrt-action-save-package-data.c b/src/daemon/abrt-action-save-package-data.c
index 6dbcfc2..cc86327 100644
--- a/src/daemon/abrt-action-save-package-data.c
+++ b/src/daemon/abrt-action-save-package-data.c
@@ -91,16 +91,22 @@ static void load_gpg_keys(void)
     if (strcmp(gpg_keys_dir, "") != 0)
     {
         log_debug("Reading gpg keys from '%s'", gpg_keys_dir);
+        GHashTable *done_set = g_hash_table_new(g_str_hash, g_str_equal);
         GList *gpg_files = get_file_list(gpg_keys_dir, NULL /* we don't care about the file ext */);
-        GList *tmp_gpp_files = gpg_files;
-        while (tmp_gpp_files)
+        for (GList *iter = gpg_files; iter; iter = g_list_next(iter))
         {
-            log_debug("Loading gpg key '%s'", fo_get_fullpath((file_obj_t *)tmp_gpp_files->data));
-            settings_setOpenGPGPublicKeys = g_list_append(settings_setOpenGPGPublicKeys, xstrdup(fo_get_fullpath((file_obj_t *)(tmp_gpp_files->data)) ));
-            tmp_gpp_files = g_list_next(tmp_gpp_files);
+            const char *key_path = fo_get_fullpath((file_obj_t *)iter->data);
+
+            if (g_hash_table_contains(done_set, key_path))
+                continue;
+
+            g_hash_table_insert(done_set, (gpointer)key_path, NULL);
+            log_debug("Loading gpg key '%s'", key_path);
+            settings_setOpenGPGPublicKeys = g_list_append(settings_setOpenGPGPublicKeys, xstrdup(key_path));
         }
 
         g_list_free_full(gpg_files, (GDestroyNotify)free_file_obj);
+        g_hash_table_destroy(done_set);
     }
 }
 
-- 
2.1.0