From 5560ca0e51919bc5aeccb22584e24219040dc78b Mon Sep 17 00:00:00 2001

From: Jakub Filak <jfilak@redhat.com>

Date: Tue, 24 Mar 2015 20:57:34 +0100

Subject: [PATCH] cli: use the DBus methods for getting problem information

The dump directory is no longer accessible by non-root users and we also

want to get rid of direct access to allow administrators (wheel members)

see problem data without the need to ChownProblem directory before.

Related: #1224984

Signed-off-by: Jakub Filak <jfilak@redhat.com>

---

 src/cli/abrt-cli-core.c | 74 ++++++++++++++++++++++++-------------------------

 src/cli/abrt-cli-core.h |  4 ++-

 src/cli/list.c          | 45 +++++++++++-------------------

 src/cli/process.c       |  6 +---

 src/cli/status.c        | 66 +++++++++++++------------------------------

 5 files changed, 77 insertions(+), 118 deletions(-)

diff --git a/src/cli/abrt-cli-core.c b/src/cli/abrt-cli-core.c

index 23a74a8..77a37f7 100644

--- a/src/cli/abrt-cli-core.c

+++ b/src/cli/abrt-cli-core.c

@@ -39,24 +39,22 @@ vector_of_problem_data_t *new_vector_of_problem_data(void)

     return g_ptr_array_new_with_free_func((void (*)(void*)) &problem_data_free);

 }

-static int

-append_problem_data(struct dump_dir *dd, void *arg)

+vector_of_problem_data_t *fetch_crash_infos(void)

 {

-    vector_of_problem_data_t *vpd = arg;

-

-    problem_data_t *problem_data = create_problem_data_from_dump_dir(dd);

-    problem_data_add(problem_data, CD_DUMPDIR, dd->dd_dirname,

-                            CD_FLAG_TXT + CD_FLAG_ISNOTEDITABLE + CD_FLAG_LIST);

-    g_ptr_array_add(vpd, problem_data);

-    return 0;

-}

+    GList *problems = get_problems_over_dbus(/*don't authorize*/false);

+    if (problems == ERR_PTR)

+        return NULL;

-vector_of_problem_data_t *fetch_crash_infos(GList *dir_list)

-{

     vector_of_problem_data_t *vpd = new_vector_of_problem_data();

-    for (GList *li = dir_list; li; li = li->next)

-        for_each_problem_in_dir(li->data, getuid(), append_problem_data, vpd);

+    for (GList *iter = problems; iter; iter = g_list_next(iter))

+    {

+        problem_data_t *problem_data = get_full_problem_data_over_dbus((const char *)(iter->data));

+        if (problem_data == ERR_PTR)

+            continue;

+

+        g_ptr_array_add(vpd, problem_data);

+    }

     return vpd;

 }

@@ -74,36 +72,38 @@ static bool isxdigit_str(const char *str)

     return true;

 }

-struct name_resolution_param {

-    const char *shortcut;

-    unsigned strlen_shortcut;

-    char *found_name;

-};

-

-static int find_dir_by_hash(struct dump_dir *dd, void *arg)

+char *find_problem_by_hash(const char *hash, GList *problems)

 {

-    struct name_resolution_param *param = arg;

-    char hash_str[SHA1_RESULT_LEN*2 + 1];

-    str_to_sha1str(hash_str, dd->dd_dirname);

-    if (strncasecmp(param->shortcut, hash_str, param->strlen_shortcut) == 0)

+    unsigned hash_len = strlen(hash);

+    if (!isxdigit_str(hash) || hash_len < 5)

+        return NULL;

+

+    char *found_name = NULL;

+    for (GList *iter = problems; iter; iter = g_list_next(iter))

     {

-        if (param->found_name)

-            error_msg_and_die(_("'%s' identifies more than one problem directory"), param->shortcut);

-        param->found_name = xstrdup(dd->dd_dirname);

+        char hash_str[SHA1_RESULT_LEN*2 + 1];

+        str_to_sha1str(hash_str, (const char *)(iter->data));

+        if (strncasecmp(hash, hash_str, hash_len) == 0)

+        {

+            if (found_name)

+                error_msg_and_die(_("'%s' identifies more than one problem directory"), hash);

+            found_name = xstrdup((const char *)(iter->data));

+        }

     }

-    return 0;

+

+    return found_name;

 }

 char *hash2dirname(const char *hash)

 {

-    unsigned hash_len = strlen(hash);

-    if (!isxdigit_str(hash) || hash_len < 5)

+    /* Try loading by dirname hash */

+    GList *problems = get_problems_over_dbus(/*don't authorize*/false);

+    if (problems == ERR_PTR)

         return NULL;

-    /* Try loading by dirname hash */

-    struct name_resolution_param param = { hash, hash_len, NULL };

-    GList *dir_list = get_problem_storages();

-    for (GList *li = dir_list; li; li = li->next)

-        for_each_problem_in_dir(li->data, getuid(), find_dir_by_hash, ¶m;;

-    return param.found_name;

+    char *found_name = find_problem_by_hash(hash, problems);

+

+    g_list_free_full(problems, free);

+

+    return found_name;

 }

diff --git a/src/cli/abrt-cli-core.h b/src/cli/abrt-cli-core.h

index 83d0b5d..33b2ea6 100644

--- a/src/cli/abrt-cli-core.h

+++ b/src/cli/abrt-cli-core.h

@@ -28,9 +28,11 @@ problem_data_t *get_problem_data(vector_of_problem_data_t *vector, unsigned i);

 void free_vector_of_problem_data(vector_of_problem_data_t *vector);

 vector_of_problem_data_t *new_vector_of_problem_data(void);

-vector_of_problem_data_t *fetch_crash_infos(GList *dir_list);

+vector_of_problem_data_t *fetch_crash_infos(void);

 /* Returns malloced string, or NULL if not found: */

+char *find_problem_by_hash(const char *hash, GList *problems);

+/* Returns malloced string, or NULL if not found: */

 char *hash2dirname(const char *hash);

diff --git a/src/cli/list.c b/src/cli/list.c

index ccb5f3b..1594906 100644

--- a/src/cli/list.c

+++ b/src/cli/list.c

@@ -30,33 +30,28 @@

  *       ~/.abrt/spool and /var/tmp/abrt? needs more _meditation_.

  */

-static problem_data_t *load_problem_data(const char *dump_dir_name)

+static problem_data_t *load_problem_data(const char *problem_id)

 {

-    /* First, try loading by dirname */

-    int sv_logmode = logmode;

-    logmode = 0; /* suppress EPERM/EACCES errors in opendir */

-    struct dump_dir *dd = dd_opendir(dump_dir_name, /*flags:*/ DD_OPEN_READONLY);

-    logmode = sv_logmode;

+    char *name2 = NULL;

+

+    /* First, check if there is a problem with the passed id */

+    GList *problems = get_problems_over_dbus(/*don't authorize*/false);

+    GList *item = g_list_find_custom(problems, problem_id, (GCompareFunc)strcmp);

     /* (git requires at least 5 char hash prefix, we do the same) */

-    if (!dd && errno == ENOENT)

+    if (item == NULL)

     {

         /* Try loading by dirname hash */

-        char *name2 = hash2dirname(dump_dir_name);

-        if (name2)

-            dd = dd_opendir(name2, /*flags:*/ DD_OPEN_READONLY);

-        free(name2);

-    }

+        name2 = find_problem_by_hash(problem_id, problems);

+        if (name2 == NULL)

+            return NULL;

-    if (!dd)

-        return NULL;

+        problem_id = name2;

+    }

-    problem_data_t *problem_data = create_problem_data_from_dump_dir(dd);

-    problem_data_add(problem_data, CD_DUMPDIR, dd->dd_dirname,

-                            CD_FLAG_TXT + CD_FLAG_ISNOTEDITABLE + CD_FLAG_LIST);

-    dd_close(dd);

+    problem_data_t *problem_data = get_full_problem_data_over_dbus(problem_id);

-    return problem_data;

+    return (problem_data == ERR_PTR ? NULL : problem_data);

 }

 /** Prints basic information about a crash to stdout. */

@@ -176,7 +171,7 @@ static bool print_crash_list(vector_of_problem_data_t *crash_list, int detailed,

 int cmd_list(int argc, const char **argv)

 {

     const char *program_usage_string = _(

-        "& list [options] [DIR]..."

+        "& list [options]"

         );

     int opt_not_reported = 0;

@@ -194,15 +189,8 @@ int cmd_list(int argc, const char **argv)

     };

     parse_opts(argc, (char **)argv, program_options, program_usage_string);

-    argv += optind;

-

-    GList *D_list = NULL;

-    while (*argv)

-        D_list = g_list_append(D_list, xstrdup(*argv++));

-    if (!D_list)

-        D_list = get_problem_storages();

-    vector_of_problem_data_t *ci = fetch_crash_infos(D_list);

+    vector_of_problem_data_t *ci = fetch_crash_infos();

     g_ptr_array_sort_with_data(ci, &cmp_problem_data, (char *) FILENAME_LAST_OCCURRENCE);

@@ -212,7 +200,6 @@ int cmd_list(int argc, const char **argv)

     print_crash_list(ci, opt_detailed, opt_not_reported, opt_since, opt_until, CD_TEXT_ATT_SIZE_BZ);

     free_vector_of_problem_data(ci);

-    list_free_with_free(D_list);

 #if SUGGEST_AUTOREPORTING != 0

     load_abrt_conf();

diff --git a/src/cli/process.c b/src/cli/process.c

index 7f4fff5..39462f9 100644

--- a/src/cli/process.c

+++ b/src/cli/process.c

@@ -152,18 +152,14 @@ int cmd_process(int argc, const char **argv)

     };

     parse_opts(argc, (char **)argv, program_options, program_usage_string);

-    argv += optind;

-    GList *D_list = get_problem_storages();

-

-    vector_of_problem_data_t *ci = fetch_crash_infos(D_list);

+    vector_of_problem_data_t *ci = fetch_crash_infos();

     g_ptr_array_sort_with_data(ci, &cmp_problem_data, (char *) FILENAME_LAST_OCCURRENCE);

     process_crashes(ci, opt_since);

     free_vector_of_problem_data(ci);

-    list_free_with_free(D_list);

     return 0;

 }

diff --git a/src/cli/status.c b/src/cli/status.c

index 1de2d41..68bdd0e 100644

--- a/src/cli/status.c

+++ b/src/cli/status.c

@@ -21,53 +21,36 @@

 #include <sys/types.h>

 #include "problem_api.h"

-struct time_range {

-    unsigned count;

-    unsigned long since;

-};

-

-static int count_dir_if_newer_than(struct dump_dir *dd, void *arg)

-{

-    struct time_range *me = arg;

-

-    if (dd_exist(dd, FILENAME_REPORTED_TO))

-        return 0;

-

-    char *time_str = dd_load_text(dd, FILENAME_LAST_OCCURRENCE);

-    long val = atol(time_str);

-    free(time_str);

-    if (val < me->since)

-        return 0;

-

-    me->count++;

-    return 0;

-}

-

-static void count_problems_in_dir(gpointer data, gpointer arg)

+static unsigned int count_problem_dirs(unsigned long since)

 {

-    char *path = data;

-    struct time_range *me = arg;

+    unsigned count = 0;

-    log_info("scanning '%s' for problems since %lu", path, me->since);

+    GList *problems = get_problems_over_dbus(/*don't authorize*/false);

+    for (GList *iter = problems; iter != NULL; iter = g_list_next(iter))

+    {

+        const char *problem_id = (const char *)iter->data;

+        if (test_exist_over_dbus(problem_id, FILENAME_REPORTED_TO))

+            continue;

-    for_each_problem_in_dir(path, getuid(), count_dir_if_newer_than, me);

-}

+        char *time_str = load_text_over_dbus(problem_id, FILENAME_LAST_OCCURRENCE);

+        if (time_str == NULL)

+            continue;

-static unsigned int count_problem_dirs(GList *paths, unsigned long since)

-{

-    struct time_range me;

-    me.count = 0;

-    me.since = since;

+        long val = atol(time_str);

+        free(time_str);

+        if (val < since)

+            return 0;

-    g_list_foreach(paths, count_problems_in_dir, &me);

+        count++;

+    }

-    return me.count;

+    return count;

 }

 int cmd_status(int argc, const char **argv)

 {

     const char *program_usage_string = _(

-        "& status [DIR]..."

+        "& status"

         );

     int opt_bare = 0; /* must be _int_, OPT_BOOL expects that! */

@@ -81,17 +64,8 @@ int cmd_status(int argc, const char **argv)

     };

     parse_opts(argc, (char **)argv, program_options, program_usage_string);

-    argv += optind;

-

-    GList *problem_dir_list = NULL;

-    while (*argv)

-        problem_dir_list = g_list_append(problem_dir_list, xstrdup(*argv++));

-    if (!problem_dir_list)

-        problem_dir_list = get_problem_storages();

-

-    unsigned int problem_count = count_problem_dirs(problem_dir_list, opt_since);

-    list_free_with_free(problem_dir_list);

+    unsigned int problem_count = count_problem_dirs(opt_since);

     /* show only if there is at least 1 problem or user set the -v */

     if (problem_count > 0 || g_verbose > 0)

-- 

2.4.3