[newlib-cygwin] Cygwin: POSIX msg queues: destroy msg queue fhandler in case of error
Corinna Vinschen
corinna@sourceware.org
Fri May 21 13:34:38 GMT 2021
https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=bce3563e988137a23a6867d749b727825a2ad96a
commit bce3563e988137a23a6867d749b727825a2ad96a
Author: Corinna Vinschen <corinna@vinschen.de>
Date: Fri May 21 15:26:51 2021 +0200
Cygwin: POSIX msg queues: destroy msg queue fhandler in case of error
...and don't leave a descriptor dangling
Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
Diff:
---
winsup/cygwin/posix_ipc.cc | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/winsup/cygwin/posix_ipc.cc b/winsup/cygwin/posix_ipc.cc
index fc483bdc8..8101a28f9 100644
--- a/winsup/cygwin/posix_ipc.cc
+++ b/winsup/cygwin/posix_ipc.cc
@@ -392,7 +392,6 @@ mq_open (const char *name, int oflag, ...)
fh = (fhandler_mqueue *) build_fh_dev (*mqueue_dev, name);
if (!fh)
__leave;
- fdm = fh;
mqinfo = fh->mqinfo_create ((HANDLE) _get_osfhandle (fd), filesize,
mode, nonblock);
@@ -426,6 +425,7 @@ mq_open (const char *name, int oflag, ...)
if (fchmod (fd, mode) == -1)
__leave;
close (fd);
+ fdm = fh;
return (mqd_t) fdm;
}
@@ -468,7 +468,6 @@ mq_open (const char *name, int oflag, ...)
fh = (fhandler_mqueue *) build_fh_dev (*mqueue_dev, name);
if (!fh)
__leave;
- fdm = fh;
mqinfo = fh->mqinfo_open ((HANDLE) _get_osfhandle (fd), statbuff.st_size,
statbuff.st_mode, nonblock);
@@ -476,6 +475,7 @@ mq_open (const char *name, int oflag, ...)
__leave;
close (fd);
+ fdm = fh;
return (mqd_t) fdm;
}
__except (EFAULT) {}
@@ -486,6 +486,11 @@ mq_open (const char *name, int oflag, ...)
unlink (mqname);
if (fd >= 0)
close (fd);
+ if (fh)
+ {
+ fh->close ();
+ delete fh;
+ }
return (mqd_t) -1;
}
More information about the Cygwin-cvs
mailing list