diff options
| author | Michal Zidek <mzidek@redhat.com> | 2013-01-25 16:41:07 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-02-06 14:29:06 +0100 |
| commit | 1117cf65b89b6cbd3a3b1e34589f57ab71a35d60 (patch) | |
| tree | 68f8bf45267eef43eb14de2ed687cd80a72eaf10 /src | |
| parent | 242521606b86e2178ce9f2db0e84c05f650031b8 (diff) | |
| download | sssd-1117cf65b89b6cbd3a3b1e34589f57ab71a35d60.tar.gz sssd-1117cf65b89b6cbd3a3b1e34589f57ab71a35d60.tar.xz sssd-1117cf65b89b6cbd3a3b1e34589f57ab71a35d60.zip | |
Unchecked return value in files.c
Found by coverity.
https://fedorahosted.org/sssd/ticket/1791
Diffstat (limited to 'src')
| -rw-r--r-- | src/tools/files.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/tools/files.c b/src/tools/files.c index 62fd00368..572617296 100644 --- a/src/tools/files.c +++ b/src/tools/files.c @@ -691,6 +691,7 @@ copy_dir(struct copy_ctx *cctx, const struct stat *src_dir_stat) { errno_t ret; + errno_t dret; int dest_dir_fd = -1; DIR *dir = NULL; struct dirent *ent; @@ -779,7 +780,14 @@ copy_dir(struct copy_ctx *cctx, ret = EOK; done: - if (dir) closedir(dir); + if (dir) { + dret = closedir(dir); + if (dret != 0) { + dret = errno; + DEBUG(SSSDBG_MINOR_FAILURE, + ("Failed to close directory: %s.\n", strerror(dret))); + } + } if (dest_dir_fd != -1) { close(dest_dir_fd); |