Subject: Re: A patch for _opening com1
From: [email-masked] (Kohtala Marko)

2 Apr 1997 02:52:17 -0800
Sender: [email-masked]
Approved: [email-masked]
Distribution: cygnus
Message-ID: <[email-masked]>
Reply-To: [email-masked]
Original-To: [email-masked]
Original-Cc: [email-masked], [email-masked]
In-Reply-To: <[email-masked]>
	([email-masked])
Original-Sender: [email-masked]

> Hi Marko, you wrote:
> : +      if (GetFileType (h) != FILE_TYPE_DISK)
> : +	{
> : +	  /* It is not a disk file, can not be a symlink */

> If I remember right, there's also FILE_TYPE_REMOTE, which also holds disk
> files and possibly symlinks.

You are right. This FILE_TYPE_REMOVE is a flag bit in the return value
and the Win32 Programmer's Reference fails to tell so. I found it
mentioned in machintosh porting reference and in the WINBASE.H header
file though...

So the patch should be like

diff -up b17.1/winsup/path.cc\~ b17.1/winsup/path.cc
--- b17.1/winsup/path.cc~	Wed Dec  4 05:23:01 1996
+++ b17.1/winsup/path.cc	Tue Apr  1 17:14:21 1997
@@ -1056,7 +1056,14 @@ symlink_check (const char *path, char *b
       char cookie_buf[sizeof (SYMLINK_COOKIE) - 1];
       DWORD done;
 
-      if (! ReadFile (h, cookie_buf, sizeof (cookie_buf), &done, 0))
+      if ((GetFileType (h) & ~FILE_TYPE_REMOVE) != FILE_TYPE_DISK)
+	{
+	  /* It is not a disk file, can not be a symlink */
+	  *syml = 0;
+	  *exec = 0;
+	  res = 0;
+	}
+      else if (! ReadFile (h, cookie_buf, sizeof (cookie_buf), &done, 0))
 	{
 	  set_errno (EIO);
 	}

-- 
---
Marko Kohtala - [email-masked], [email-masked]
-
For help on using this list, send a message to
"[email-masked]" with one line of text: "help".