Mail Archives: cygwin-developers/2002/07/22/12:40:46

Mailing-List: contact cygwin-developers-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-developers-subscribe AT cygwin DOT com>
List-Archive: <>
List-Post: <mailto:cygwin-developers AT cygwin DOT com>
List-Help: <mailto:cygwin-developers-help AT cygwin DOT com>, <>
Sender: cygwin-developers-owner AT cygwin DOT com
Delivered-To: mailing list cygwin-developers AT cygwin DOT com
Date: Mon, 22 Jul 2002 12:41:57 -0400
From: Jason Tishler <jason AT tishler DOT net>
Subject: path_conv::check() gets confused by recently removed files
To: Cygwin-Developers <cygwin-developers AT cygwin DOT com>
Mail-followup-to: Cygwin-Developers <cygwin-developers AT cygwin DOT com>
Message-id: <>
MIME-version: 1.0
User-Agent: Mutt/1.4i

Content-type: text/plain; charset=us-ascii
Content-transfer-encoding: 7BIT
Content-disposition: inline

I guess that I found the complement of the following:

while tracking down some regressions with Cygwin Python CVS. :,)

The attached test program,, demonstrates the problem:

    $ >file
    $ j7 file

The root cause of the problem is that symlink_info::check() ignores the
ERROR_ACCESS_DENIED case in the following:

    fileattr = GetFileAttributes (suffix.path);
    if (fileattr == INVALID_FILE_ATTRIBUTES)
      /* The GetFileAttributes call can fail for reasons that don't
         matter, so we just return 0.  For example, getting the
         attributes of \\HOST will typically fail.  */
      debug_printf ("GetFileAttributes (%s) failed", suffix.path);
      error = geterrno_from_win_error (GetLastError (), EACCES);

The above causes path_conv::check() to the lop off the tail component of
the recently deleted file so that path_conv::fileattr incorrectly
indicates a directory instead of a file.  This in turn, causes fstat()
to incorrectly indicate that the file descriptor is a directory instead
of a file.

Unfortunately, once again I do not know the best way to fix this
problem.  Should fstat() (and cousins) just return EACCES in this case?

BTW, this would fix the Cygwin Python regressions:

    static PyFileObject*
    dircheck(PyFileObject* f)
        struct stat buf;
        if (f->f_fp == NULL)
            return f;
        if (fstat(fileno(f->f_fp), &buf) == 0 &&

            S_ISDIR(buf.st_mode)) {
            char *msg = strerror(EISDIR);
            PyObject *exc = PyObject_CallFunction(PyExc_IOError, ...
            PyErr_SetObject(PyExc_IOError, exc);
            return NULL;
        return f;


Content-type: text/plain; charset=us-ascii;
Content-transfer-encoding: 7BIT
Content-disposition: attachment;

#include <stdio.h>
#include <errno.h>
#include <sys/stat.h>

main(int argc, char* argv[])
	char* file = argv[1];
	FILE* fp = fopen(file, "wb+");
	if (!fp)
		printf("fopen(%s) failed with errno = %d\n", file, errno);
		return 1;

	int status = remove(file);
	if (status < 0)
		printf("remove(%s) failed with errno = %d\n", file, errno);
		return 2;

	struct stat buf;
	status = fstat(fileno(fp), &buf);
	if (status < 0)
		printf("fstat() failed with errno = %d\n", errno);

	if (S_ISDIR(buf.st_mode))
		printf("not dir\n");

	return 0;


- Raw text -

  webmaster     delorie software   privacy  
  Copyright 2019   by DJ Delorie     Updated Jul 2019