summaryrefslogtreecommitdiffstats
path: root/lib/Plugins
diff options
context:
space:
mode:
authorDenys Vlasenko <vda.linux@googlemail.com>2010-01-11 07:22:13 +0100
committerDenys Vlasenko <vda.linux@googlemail.com>2010-01-11 07:22:13 +0100
commit14ef0cfe72faf6696df3ef8f42927e9458ccbeeb (patch)
treec5d3a3169fc31f212d507553beb67920be92ebcd /lib/Plugins
parentb1c4304104910c4bc066cd43f9784fe2f3ddf1ad (diff)
downloadabrt-14ef0cfe72faf6696df3ef8f42927e9458ccbeeb.tar.gz
abrt-14ef0cfe72faf6696df3ef8f42927e9458ccbeeb.tar.xz
abrt-14ef0cfe72faf6696df3ef8f42927e9458ccbeeb.zip
*: misc fixes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'lib/Plugins')
-rw-r--r--lib/Plugins/CCpp.cpp2
-rw-r--r--lib/Plugins/FileTransfer.cpp20
-rw-r--r--lib/Plugins/RunApp.cpp4
3 files changed, 11 insertions, 15 deletions
diff --git a/lib/Plugins/CCpp.cpp b/lib/Plugins/CCpp.cpp
index a7f005c..46c2f28 100644
--- a/lib/Plugins/CCpp.cpp
+++ b/lib/Plugins/CCpp.cpp
@@ -627,7 +627,7 @@ static bool DebuginfoCheckPolkit(int uid)
//parent
int status;
- if (waitpid(child_pid, &status, 0) > 0 && WEXITSTATUS(status) == 0)
+ if (waitpid(child_pid, &status, 0) > 0 && WIFEXITED(status) && WEXITSTATUS(status) == 0)
{
return true; //authorization OK
}
diff --git a/lib/Plugins/FileTransfer.cpp b/lib/Plugins/FileTransfer.cpp
index 60e1e66..9d7a59a 100644
--- a/lib/Plugins/FileTransfer.cpp
+++ b/lib/Plugins/FileTransfer.cpp
@@ -110,13 +110,11 @@ parameter "something" to each filename,
now used in create_zip, but can be useful for some future
archivers as well
*/
-static void traverse_directory(const char * directory, void * something,
- void (*func)(void *, const char *) )
+static void traverse_directory(const char *directory, void *something,
+ void (*func)(void *, const char *))
{
- DIR * dp;
- struct dirent * dirp;
- char complete_name[BUFSIZ];
- char * end;
+ DIR *dp;
+ struct dirent *dirp;
dp = opendir(directory);
if (dp == NULL)
@@ -127,14 +125,8 @@ static void traverse_directory(const char * directory, void * something,
{
if (is_regular_file(dirp, directory))
{
- end = stpcpy(complete_name, directory);
- if (end[-1] != '/')
- {
- *end++ = '/';
- }
- end = stpcpy(end, dirp->d_name);
-
- func(something, complete_name);
+ string complete_name = concat_path_file(directory, dirp->d_name);
+ func(something, complete_name.c_str());
}
}
closedir(dp);
diff --git a/lib/Plugins/RunApp.cpp b/lib/Plugins/RunApp.cpp
index c1f725b..e2147e2 100644
--- a/lib/Plugins/RunApp.cpp
+++ b/lib/Plugins/RunApp.cpp
@@ -40,6 +40,10 @@ void CActionRunApp::Run(const char *pActionDir, const char *pArgs)
vector_string_t args;
parse_args(pArgs, args, '"');
+ if (args.size() <= COMMAND)
+ {
+ return;
+ }
const char *cmd = args[COMMAND].c_str();
if (!cmd[0])
{