libs/newlib-cygwin
libs/newlib-cygwin
4
0
Fork 0
mirror of git://sourceware.org/git/newlib-cygwin.git synced 2025-02-02 12:30:24 +08:00
Code Issues Projects Releases Wiki Activity

Cygwin: fix handling of known reparse points that are not symlinks

Browse Source 
Commit aa467e6e, "Cygwin: add AF_UNIX reparse points to path
handling", changed check_reparse_point_target so that it could return
a positive value on a known reparse point that is not a symlink.  But
some of the code in check_reparse_point that handles this positive
return value was executed unconditionally, when it should have been
executed only for symlinks.

As a result, posixify could be called on a buffer containing garbage,
and check_reparse_point could erroneously return a positive value on a
non-symlink.  This is now fixed so that posixify is only called if the
reparse point is a symlink, and check_reparse_point returns 0 if the
reparse point is not a symlink.

Also fix symlink_info::check to handle this last case, in which
check_reparse_point returns 0 on a known reparse point.
This commit is contained in:
Ken Brown 2020-09-28 16:34:44 -04:00
parent 032f622533
commit 412c761a28
1 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
winsup/cygwin/path.cc
View File

@ -2655,11 +2655,15 @@ symlink_info::check_reparse_point (HANDLE h, bool remote)
/* ret is > 0, so it's a known reparse point, path in symbuf. */ /* ret is > 0, so it's a known reparse point, path in symbuf. */
path_flags |= ret; path_flags |= ret;
if (ret & PATH_SYMLINK) if (ret & PATH_SYMLINK)
sys_wcstombs (srcbuf, SYMLINK_MAX + 7, symbuf.Buffer, {
symbuf.Length / sizeof (WCHAR)); sys_wcstombs (srcbuf, SYMLINK_MAX + 7, symbuf.Buffer,
/* A symlink is never a directory. */ symbuf.Length / sizeof (WCHAR));
fileattr &= ~FILE_ATTRIBUTE_DIRECTORY; /* A symlink is never a directory. */
return posixify (srcbuf); fileattr &= ~FILE_ATTRIBUTE_DIRECTORY;
return posixify (srcbuf);
}
else
return 0;
} }
int int
@ -3274,6 +3278,9 @@ restart:
&= ~FILE_ATTRIBUTE_DIRECTORY; &= ~FILE_ATTRIBUTE_DIRECTORY;
break; break;
} }
else if (res == 0 && (path_flags & PATH_REP))
/* Known reparse point but not a symlink. */
goto file_not_symlink;
else else
{ {
/* Volume moint point or unrecognized reparse point type. /* Volume moint point or unrecognized reparse point type.