[PATCH v4 18/32] commit_lock_file(): if close fails, roll back
From: Michael Haggerty <hidden>
Date: 2016-06-15 23:02:27
Subsystem:
the rest · Maintainer:
Linus Torvalds
If closing an open lockfile fails, then we cannot be sure of the contents of the lockfile, so there is nothing sensible to do but delete it. This change also leaves the lock_file object in a defined state in this error path (namely, unlocked). Signed-off-by: Michael Haggerty <redacted> --- lockfile.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/lockfile.c b/lockfile.c
index 716b674..213783a 100644
--- a/lockfile.c
+++ b/lockfile.c@@ -310,8 +310,12 @@ int commit_lock_file(struct lock_file *lk) if (!lk->filename[0]) die("BUG: attempt to commit unlocked object"); - if (lk->fd >= 0 && close_lock_file(lk)) + if (lk->fd >= 0 && close_lock_file(lk)) { + int save_errno = errno; + rollback_lock_file(lk); + errno = save_errno; return -1; + } strcpy(result_file, lk->filename); /* remove ".lock": */
--
2.1.0