This is the mail archive of the cygwin@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [BUG] open(): Opening with flags O_RDONLY | O_APPEND positions the file pointer at the end of the file


On Wed, Jul 10, 2002 at 12:50:14PM -0400, Chris Faylor wrote:
> On Wed, Jul 10, 2002 at 06:48:30PM +0200, Corinna Vinschen wrote:
> >On Wed, Jul 10, 2002 at 12:36:13PM -0400, Chris Faylor wrote:
> >> On Wed, Jul 10, 2002 at 03:39:43PM +0200, Pavel Tsekov wrote:
> >> >Hello, there :)
> >> >
> >> >Attached is a testcase which displays the problem. On Linux it will
> >> >properly return 13, while on cygwin it returns 0.
> >> >
> >> >I found this while trying to understand why MC doesn't extract
> >> >properly files from say .zip files.
> >> >
> >> >I don't know exactly why they call O_RDONLY combined with O_APPEND but
> >> >I will mail the mc-devel list ASAP.
> >> >
> >> >Still according to the Linux man page and SUSv2, O_APPEND should be
> >> >taken into account only when writing to the file.
> >> >
> >> >Having in mind that fhandler_base::write() calls SetFilePointer
> >> >before each write, I wonder why fhandler_disk_base::open calls
> >> >SetFilePointer when it detects O_APPEND ?
> >> 
> >> Good question.
> >
> >What if
> >
> > 	fd = open(O_APPEND);
> >	pos = lseek(fd, 0, SEEK_CUR);
> >
> >?  If open() doesn't move the pointer, `pos' is incorrectly set to 0.
> 
> Actually, I'd argue that, in that case, lseek should have some O_APPEND
> logic of its own.

Just checked the following on Linux:

    #include <stdio.h>
    #include <fcntl.h>

    int
    main()
    {
      int fd;
      off_t pos;

      fd = open ("x.c", O_APPEND);
      if (fd >= 0)
	{
	  pos = lseek (fd, 0, SEEK_CUR);
	  printf ("pos1: %ld\n", pos);
	  pos = lseek (fd, 0, SEEK_END);
	  printf ("pos2: %ld\n", pos);
	  pos = lseek (fd, 0, SEEK_SET);
	  printf ("pos3: %ld\n", pos);
	  close (fd);
	}
      return 0;
    }

Output:

pos1: 0
pos2: 382
pos3: 0

So open() doesn't set the pointer.  And lseek() doesn't take O_APPEND
into account either.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin@cygwin.com
Red Hat, Inc.

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]