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