summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichal Zidek <mzidek@redhat.com>2012-07-30 18:17:02 +0200
committerJakub Hrozek <jhrozek@redhat.com>2012-08-03 11:10:25 +0200
commit2e1b46d7acb03b22b27bbad2816b996db629d609 (patch)
treed399e472ea6e68e809ef64b23edd89af24d07bd5
parent1bb7735288ab75ddb8ce6875a6e1c7e2dd65a010 (diff)
downloadsssd-2e1b46d7acb03b22b27bbad2816b996db629d609.tar.gz
sssd-2e1b46d7acb03b22b27bbad2816b996db629d609.tar.xz
sssd-2e1b46d7acb03b22b27bbad2816b996db629d609.zip
Return value of fread in src/tools/sss_debuglevel.c no longer ignored.
https://fedorahosted.org/sssd/ticket/1426
-rw-r--r--src/tools/sss_debuglevel.c10
1 files changed, 9 insertions, 1 deletions
diff --git a/src/tools/sss_debuglevel.c b/src/tools/sss_debuglevel.c
index 603ae1669..908518b1d 100644
--- a/src/tools/sss_debuglevel.c
+++ b/src/tools/sss_debuglevel.c
@@ -319,6 +319,7 @@ fail:
errno_t get_sssd_pid(pid_t *out_pid)
{
int ret;
+ size_t fsize;
FILE *pid_file = NULL;
char pid_str[MAX_PID_LENGTH] = {'\0'};
@@ -333,7 +334,8 @@ errno_t get_sssd_pid(pid_t *out_pid)
goto done;
}
- fread(pid_str, sizeof(char), MAX_PID_LENGTH * sizeof(char), pid_file);
+ fsize = fread(pid_str, sizeof(char), MAX_PID_LENGTH * sizeof(char),
+ pid_file);
if (!feof(pid_file)) {
/* eof not reached */
ret = ferror(pid_file);
@@ -346,6 +348,12 @@ errno_t get_sssd_pid(pid_t *out_pid)
}
goto done;
}
+ if (fsize == 0) {
+ DEBUG(SSSDBG_CRIT_FAILURE, ("File \"%s\" contains no pid.\n",
+ SSSD_PIDFILE));
+ ret = EINVAL;
+ goto done;
+ }
pid_str[MAX_PID_LENGTH-1] = '\0';
*out_pid = parse_pid(pid_str);