|
 |
c313de |
From 3d08d15b8fb214233e6b426bffe8f8b89969529a Mon Sep 17 00:00:00 2001
|
|
|
c313de |
Message-Id: <3d08d15b8fb214233e6b426bffe8f8b89969529a@dist-git>
|
|
|
c313de |
From: Shi Lei <shi_lei@massclouds.com>
|
|
|
c313de |
Date: Fri, 21 Jun 2019 09:25:42 +0200
|
|
|
c313de |
Subject: [PATCH] util: file: introduce VIR_AUTOCLOSE macro to close fd of the
|
|
|
c313de |
file automatically
|
|
|
c313de |
MIME-Version: 1.0
|
|
|
c313de |
Content-Type: text/plain; charset=UTF-8
|
|
|
c313de |
Content-Transfer-Encoding: 8bit
|
|
|
c313de |
|
|
|
c313de |
Signed-off-by: Shi Lei <shi_lei@massclouds.com>
|
|
|
c313de |
(cherry picked from commit 09d35afd2c3058688290d5d818343d0f6aa2dd6e)
|
|
|
c313de |
|
|
|
c313de |
https://bugzilla.redhat.com/show_bug.cgi?id=1697627
|
|
|
c313de |
|
|
|
c313de |
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
|
|
|
c313de |
Message-Id: <93cb7516e852b96a82eac8dd71acdccc81ef13f9.1561068591.git.jdenemar@redhat.com>
|
|
|
c313de |
Reviewed-by: Ján Tomko <jtomko@redhat.com>
|
|
|
c313de |
---
|
|
|
c313de |
src/util/virfile.h | 18 ++++++++++++++++--
|
|
|
c313de |
1 file changed, 16 insertions(+), 2 deletions(-)
|
|
|
c313de |
|
|
|
c313de |
diff --git a/src/util/virfile.h b/src/util/virfile.h
|
|
|
c313de |
index 51c221e069..fa03269289 100644
|
|
|
c313de |
--- a/src/util/virfile.h
|
|
|
c313de |
+++ b/src/util/virfile.h
|
|
|
c313de |
@@ -53,6 +53,11 @@ int virFileClose(int *fdptr, virFileCloseFlags flags)
|
|
|
c313de |
int virFileFclose(FILE **file, bool preserve_errno) ATTRIBUTE_RETURN_CHECK;
|
|
|
c313de |
FILE *virFileFdopen(int *fdptr, const char *mode) ATTRIBUTE_RETURN_CHECK;
|
|
|
c313de |
|
|
|
c313de |
+static inline void virForceCloseHelper(int *fd)
|
|
|
c313de |
+{
|
|
|
c313de |
+ ignore_value(virFileClose(fd, VIR_FILE_CLOSE_PRESERVE_ERRNO));
|
|
|
c313de |
+}
|
|
|
c313de |
+
|
|
|
c313de |
/* For use on normal paths; caller must check return value,
|
|
|
c313de |
and failure sets errno per close. */
|
|
|
c313de |
# define VIR_CLOSE(FD) virFileClose(&(FD), 0)
|
|
|
c313de |
@@ -63,8 +68,7 @@ FILE *virFileFdopen(int *fdptr, const char *mode) ATTRIBUTE_RETURN_CHECK;
|
|
|
c313de |
|
|
|
c313de |
/* For use on cleanup paths; errno is unaffected by close,
|
|
|
c313de |
and no return value to worry about. */
|
|
|
c313de |
-# define VIR_FORCE_CLOSE(FD) \
|
|
|
c313de |
- ignore_value(virFileClose(&(FD), VIR_FILE_CLOSE_PRESERVE_ERRNO))
|
|
|
c313de |
+# define VIR_FORCE_CLOSE(FD) virForceCloseHelper(&(FD))
|
|
|
c313de |
# define VIR_FORCE_FCLOSE(FILE) ignore_value(virFileFclose(&(FILE), true))
|
|
|
c313de |
|
|
|
c313de |
/* Similar VIR_FORCE_CLOSE() but ignores EBADF errors since they are expected
|
|
|
c313de |
@@ -79,6 +83,16 @@ FILE *virFileFdopen(int *fdptr, const char *mode) ATTRIBUTE_RETURN_CHECK;
|
|
|
c313de |
VIR_FILE_CLOSE_PRESERVE_ERRNO | \
|
|
|
c313de |
VIR_FILE_CLOSE_DONT_LOG))
|
|
|
c313de |
|
|
|
c313de |
+/**
|
|
|
c313de |
+ * VIR_AUTOCLOSE:
|
|
|
c313de |
+ *
|
|
|
c313de |
+ * Macro to automatically force close the fd by calling virForceCloseHelper
|
|
|
c313de |
+ * when the fd goes out of scope. It's used to eliminate VIR_FORCE_CLOSE
|
|
|
c313de |
+ * in cleanup sections.
|
|
|
c313de |
+ */
|
|
|
c313de |
+# define VIR_AUTOCLOSE __attribute__((cleanup(virForceCloseHelper))) int
|
|
|
c313de |
+
|
|
|
c313de |
+
|
|
|
c313de |
/* Opaque type for managing a wrapper around a fd. */
|
|
|
c313de |
struct _virFileWrapperFd;
|
|
|
c313de |
|
|
|
c313de |
--
|
|
|
c313de |
2.22.0
|
|
|
c313de |
|