Blame SOURCES/cyrus-imapd-close_backup_fd_on_error.patch

0f9cf8
From 725e1efbd923c6d15ba639e17bfd0baabc619daa Mon Sep 17 00:00:00 2001
0f9cf8
From: Pavel Zhukov <pzhukov@redhat.com>
0f9cf8
Date: Mon, 1 Oct 2018 15:55:35 +0200
0f9cf8
Subject: [PATCH] Close file descriptior in case of error
0f9cf8
0f9cf8
Make static code analizers happy.
0f9cf8
If stat() failed for some reason it may lead backup fd unclosed.
0f9cf8
---
0f9cf8
 backup/lcb.c | 1 +
0f9cf8
 1 file changed, 1 insertion(+)
0f9cf8
0f9cf8
diff --git a/backup/lcb.c b/backup/lcb.c
0f9cf8
index 8c4a0e31a..9a04b08f2 100644
0f9cf8
--- a/backup/lcb.c
0f9cf8
+++ b/backup/lcb.c
0f9cf8
@@ -182,6 +182,7 @@ HIDDEN int backup_real_open(struct backup **backupp,
0f9cf8
         if (r) {
0f9cf8
             syslog(LOG_ERR, "IOERROR: (f)stat %s: %m", backup->data_fname);
0f9cf8
             r = IMAP_IOERROR;
0f9cf8
+            close(fd);
0f9cf8
             goto error;
0f9cf8
         }
0f9cf8