[PATCH] Cygwin: Update release/3.4.8 for latest commit

2023-07-12 Thread Mark Geisert
--- winsup/cygwin/release/3.4.8 | 3 +++ 1 file changed, 3 insertions(+) diff --git a/winsup/cygwin/release/3.4.8 b/winsup/cygwin/release/3.4.8 index d1e34ce3c..3113be8cb 100644 --- a/winsup/cygwin/release/3.4.8 +++ b/winsup/cygwin/release/3.4.8 @@ -3,3 +3,6 @@ Bug Fixes - Make safe for c89

[PATCH 5/5] Cygwin: add AT_EMPTY_PATH fix to release message

2023-07-12 Thread Corinna Vinschen
From: Corinna Vinschen Signed-off-by: Corinna Vinschen --- winsup/cygwin/release/3.4.8 | 4 1 file changed, 4 insertions(+) diff --git a/winsup/cygwin/release/3.4.8 b/winsup/cygwin/release/3.4.8 index d1e34ce3c633..581ed5a140ff 100644 --- a/winsup/cygwin/release/3.4.8 +++

[PATCH 1/5] Cygwin: gen_full_path_at: drop never reached code

2023-07-12 Thread Corinna Vinschen
From: Corinna Vinschen The check if the local variable p is NULL is useless. The preceeding code always sets p to a valid pointer, or it crashes if path_ret is invalid (which would be a bug in Cygwin). Fixes:c57b57e5c43a ("* cygwin.din: Sort.") Signed-off-by: Corinna Vinschen ---

[PATCH 3/5] Cygwin: use new _AT_NULL_PATHNAME_ALLOWED flag

2023-07-12 Thread Corinna Vinschen
From: Corinna Vinschen Convert gen_full_path_at to take flag values from the caller, rather than just a bool indicating that empty paths are allowed. This is in preparation of a better AT_EMPTY_PATH handling in a followup patch. Signed-off-by: Corinna Vinschen --- winsup/cygwin/syscalls.cc |

[PATCH 4/5] Cygwin: Fix and streamline AT_EMPTY_PATH handling

2023-07-12 Thread Corinna Vinschen
From: Corinna Vinschen The GLIBC extension AT_EMPTY_PATH allows the functions fchownat and fstatat to operate on dirfd alone, if the given pathname is an empty string. This also allows to operate on any file type, not only directories. Commit fa84aa4dd2fb4 broke this. It only allows dirfd to

[PATCH 2/5] Define _AT_NULL_PATHNAME_ALLOWED

2023-07-12 Thread Corinna Vinschen
From: Corinna Vinschen Cygwin needs an internal flag to allow specifying an empty pathname in utimesat (GLIBC extension). We define it in _default_fcntl.h to make sure we never introduce a value collision accidentally. While at it, define the values as 16 bit hex values. Signed-off-by: Corinna

[PATCH 0/5] Fix AT_EMPTY_PATH handling

2023-07-12 Thread Corinna Vinschen
From: Corinna Vinschen The GLIBC extension AT_EMPTY_PATH allows the functions fchownat and fstatat to operate on dirfd alone, if the given pathname is an empty string. This also allows to operate on any file type, not only directories. Commit fa84aa4dd2fb4 broke this. It only allows dirfd to

Re: [PATCH] fchmodat/fstatat: fix regression with empty `pathname`

2023-07-12 Thread Corinna Vinschen
Hi Johannes, On Jul 4 20:38, Corinna Vinschen wrote: > On Jul 4 17:45, Johannes Schindelin wrote: > > [...] > > BTW a colleague and I were wondering whether we really want to set > > `errno=ENOTDIR` in `gen_full_path_at()` for empty paths when > > `AT_EMPTY_PATH` is _not_ specified. As far as