Matej Habrnal fa1950
From 8ff7f7f65cf871b889c3a9a53cd1a432c63d2180 Mon Sep 17 00:00:00 2001
Matej Habrnal fa1950
From: Jakub Filak <jfilak@redhat.com>
Matej Habrnal fa1950
Date: Thu, 23 Apr 2015 13:12:01 +0200
Matej Habrnal fa1950
Subject: [PATCH] daemon: use libreport's function checking file name
Matej Habrnal fa1950
Matej Habrnal fa1950
Move the functions to libreport because we need the same functionality
Matej Habrnal fa1950
there too.
Matej Habrnal fa1950
Matej Habrnal fa1950
Related: #1214451
Matej Habrnal fa1950
Matej Habrnal fa1950
Signed-off-by: Jakub Filak <jfilak@redhat.com>
Matej Habrnal fa1950
---
Matej Habrnal fa1950
 src/daemon/abrt-server.c | 18 +-----------------
Matej Habrnal fa1950
 1 file changed, 1 insertion(+), 17 deletions(-)
Matej Habrnal fa1950
Matej Habrnal fa1950
diff --git a/src/daemon/abrt-server.c b/src/daemon/abrt-server.c
Matej Habrnal fa1950
index 9951468..287c510 100644
Matej Habrnal fa1950
--- a/src/daemon/abrt-server.c
Matej Habrnal fa1950
+++ b/src/daemon/abrt-server.c
Matej Habrnal fa1950
@@ -445,22 +445,6 @@ static int create_problem_dir(GHashTable *problem_info, unsigned pid)
Matej Habrnal fa1950
     exit(0);
Matej Habrnal fa1950
 }
Matej Habrnal fa1950
 
Matej Habrnal fa1950
-/* Checks if a string contains only printable characters. */
Matej Habrnal fa1950
-static gboolean printable_str(const char *str)
Matej Habrnal fa1950
-{
Matej Habrnal fa1950
-    do {
Matej Habrnal fa1950
-        if ((unsigned char)(*str) < ' ' || *str == 0x7f)
Matej Habrnal fa1950
-            return FALSE;
Matej Habrnal fa1950
-        str++;
Matej Habrnal fa1950
-    } while (*str);
Matej Habrnal fa1950
-    return TRUE;
Matej Habrnal fa1950
-}
Matej Habrnal fa1950
-
Matej Habrnal fa1950
-static gboolean is_correct_filename(const char *value)
Matej Habrnal fa1950
-{
Matej Habrnal fa1950
-    return printable_str(value) && !strchr(value, '/') && !strchr(value, '.');
Matej Habrnal fa1950
-}
Matej Habrnal fa1950
-
Matej Habrnal fa1950
 static gboolean key_value_ok(gchar *key, gchar *value)
Matej Habrnal fa1950
 {
Matej Habrnal fa1950
     char *i;
Matej Habrnal fa1950
@@ -479,7 +463,7 @@ static gboolean key_value_ok(gchar *key, gchar *value)
Matej Habrnal fa1950
      || strcmp(key, FILENAME_TYPE) == 0
Matej Habrnal fa1950
     )
Matej Habrnal fa1950
     {
Matej Habrnal fa1950
-        if (!is_correct_filename(value))
Matej Habrnal fa1950
+        if (!str_is_correct_filename(value))
Matej Habrnal fa1950
         {
Matej Habrnal fa1950
             error_msg("Value of '%s' ('%s') is not a valid directory name",
Matej Habrnal fa1950
                       key, value);
Matej Habrnal fa1950
-- 
Matej Habrnal fa1950
2.1.0
Matej Habrnal fa1950