commit 93a3d2b905f1012aa0eac8db36b31ce01c3571f3
author Christian Brabandt <cb@256bit.org> Fri Jul 05 09:54:30 2024 +0200
committer Christian Brabandt <cb@256bit.org> Fri Jul 05 09:58:58 2024 +0200
tree 61f9f63d223d5918d2b7e413f5466ebf9d94df34
parent 7a44f96fe7f1bc81d4a1a2f91979327e9cb1349f

patch 9.1.0531: resource leak in mch_get_random()

Problem:  resource leak in mch_get_random() (after v9.1.0518)
Solution: close file descriptor after reading successfully
          from /dev/urandom

Signed-off-by: Christian Brabandt <cb@256bit.org>

src/os_unix.c

diff --git a/src/os_unix.c b/src/os_unix.c
index b7da552..474d288 100644
--- a/src/os_unix.c
+++ b/src/os_unix.c
@@ -7740,7 +7740,10 @@
     if (fd == -1)
 	dev_urandom_state = FAIL;
     else if (read(fd, buf, len) == len)
+    {
 	dev_urandom_state = OK;
+	close(fd);
+    }
     else
     {
 	dev_urandom_state = FAIL;

src/version.c

diff --git a/src/version.c b/src/version.c
index 159fe3b..9cf6032 100644
--- a/src/version.c
+++ b/src/version.c
@@ -705,6 +705,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    531,
+/**/
     530,
 /**/
     529,