diff options
| author | Kotresh HR <khiremat@redhat.com> | 2015-01-27 17:42:53 +0530 |
|---|---|---|
| committer | Venky Shankar <vshankar@redhat.com> | 2015-01-27 18:44:08 -0800 |
| commit | ee3d92e83c64b654b42c9ecc5b0cc3ad80a1d51b (patch) | |
| tree | 288b997e63fda4c3b1a8b734a3d7d7a68bc09382 | |
| parent | 6b22f5ba834e7ea382bee88f359a10faac407a77 (diff) | |
feature/changelog: Fix changelog and history scan failure.
Fixes bad file descriptor issue while cleaning up
scratch directory during gf_changelog_register.
Change-Id: Ia6aa8d55dcc2209144b48b6583681a155d919c42
BUG: 1162057
Signed-off-by: Kotresh HR <khiremat@redhat.com>
Reviewed-on: http://review.gluster.org/9495
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Tested-by: Venky Shankar <vshankar@redhat.com>
| -rw-r--r-- | xlators/features/changelog/lib/src/gf-changelog.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/xlators/features/changelog/lib/src/gf-changelog.c b/xlators/features/changelog/lib/src/gf-changelog.c index a2257a66930..18eeac3e1a4 100644 --- a/xlators/features/changelog/lib/src/gf-changelog.c +++ b/xlators/features/changelog/lib/src/gf-changelog.c @@ -118,11 +118,12 @@ gf_changelog_open_dirs (gf_changelog_t *gfc) gfc->gfc_working_dir); ret = recursive_rmdir (gfc->gfc_current_dir); - if (ret) + if (ret) { gf_log (this->name, GF_LOG_ERROR, "Failed to rmdir: %s, err: %s", gfc->gfc_current_dir, strerror (errno)); goto out; + } ret = mkdir_p (gfc->gfc_current_dir, 0600, _gf_false); if (ret) @@ -141,11 +142,12 @@ gf_changelog_open_dirs (gf_changelog_t *gfc) gfc->gfc_working_dir); ret = recursive_rmdir (gfc->gfc_processing_dir); - if (ret) + if (ret) { gf_log (this->name, GF_LOG_ERROR, "Failed to rmdir: %s, err: %s", gfc->gfc_processing_dir, strerror (errno)); goto out; + } ret = mkdir_p (gfc->gfc_processing_dir, 0600, _gf_false); if (ret) |