From 824b3d8e8448de71ed40d1c3d13b99802eac0350 Mon Sep 17 00:00:00 2001 From: Ilya Dryomov Date: Mon, 16 Dec 2013 18:57:22 +0200 Subject: [PATCH] FileJournal: use pclose() to close a popen() stream In FileJournal::_check_disk_write_cache(), use pclose() instead of fclose() to close a stream, created by popen(). Signed-off-by: Ilya Dryomov --- src/os/FileJournal.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/os/FileJournal.cc b/src/os/FileJournal.cc index b42d9f8b39c57..d482efaa58227 100644 --- a/src/os/FileJournal.cc +++ b/src/os/FileJournal.cc @@ -224,9 +224,9 @@ void FileJournal::_check_disk_write_cache() const } close_f: - if (::fclose(fp)) { + if (pclose(fp)) { int ret = -errno; - derr << "_check_disk_write_cache: fclose error: " << cpp_strerror(ret) + derr << "_check_disk_write_cache: pclose failed: " << cpp_strerror(ret) << dendl; } done: -- 2.39.5