diff options
| author | Takashi Yano <takashi.yano@nifty.ne.jp> | 2022-01-18 22:17:37 +0900 |
|---|---|---|
| committer | Takashi Yano <takashi.yano@nifty.ne.jp> | 2022-01-18 22:17:37 +0900 |
| commit | ff539cc0f9ba4e52682a2647e4347ead4f2f8787 (patch) | |
| tree | 5518884c17f17e5995ec55068b055b06df5ace3f /winsup | |
| parent | 0cae2b802b2d8e78af45bd32e5feea86d721c5d2 (diff) | |
| download | newlib-ff539cc0f9ba4e52682a2647e4347ead4f2f8787.zip newlib-ff539cc0f9ba4e52682a2647e4347ead4f2f8787.tar.gz newlib-ff539cc0f9ba4e52682a2647e4347ead4f2f8787.tar.bz2 | |
Cygwin: fhandler_base: Fix double free caused when open() fails.
- When open fails, archetype stored in archetypes[] is not cleared.
This causes double free when next open fail. This patch fixes the
issue.
Addresses:
https://cygwin.com/pipermail/cygwin/2022-January/250518.html
Diffstat (limited to 'winsup')
| -rw-r--r-- | winsup/cygwin/fhandler.cc | 4 | ||||
| -rw-r--r-- | winsup/cygwin/release/3.3.4 | 3 |
2 files changed, 5 insertions, 2 deletions
diff --git a/winsup/cygwin/fhandler.cc b/winsup/cygwin/fhandler.cc index fc7c042..7d42713 100644 --- a/winsup/cygwin/fhandler.cc +++ b/winsup/cygwin/fhandler.cc @@ -440,8 +440,8 @@ fhandler_base::open_with_arch (int flags, mode_t mode) if (!(res = (archetype && archetype->io_handle) || open (flags, mode & 07777))) { - if (archetype) - delete archetype; + if (archetype && archetype->usecount == 0) + cygheap->fdtab.delete_archetype (archetype); } else if (archetype) { diff --git a/winsup/cygwin/release/3.3.4 b/winsup/cygwin/release/3.3.4 index e09ffef..3fa9b9e 100644 --- a/winsup/cygwin/release/3.3.4 +++ b/winsup/cygwin/release/3.3.4 @@ -30,3 +30,6 @@ Bug Fixes - Fix a problem creating a dir "foo", if a file (but not a Cygwin symlink) "foo.lnk" already exists. Addresses: https://github.com/msys2/msys2-runtime/issues/81 + +- Fix double free for archetype, which is caused when open() fails. + Addresses: https://cygwin.com/pipermail/cygwin/2022-January/250518.html |