[rfa] remote-fileio.c, remote_fileio_func_fstat, uninitialized st.st_ino.
Corinna Vinschen
vinschen@redhat.com
Thu Mar 3 08:29:00 GMT 2011
On Mar 3 08:36, Joel Brobecker wrote:
> > 2011-03-02 Michael Snyder <...>
> >
> > * remote-fileio.c (remote_fileio_func_fstat): Initialize st_ino
> > to zero.
>
> FWIW: The initialization is made for the case where the file descriptor
> is not a file, so we should be ablet to deduce that the st_ino (inode ID)
> is irrelevant and not used on the consumer side... This would suggest
> that your change is correct. (you should still wait for Corinna to
> comment)
The descriptors are console descriptors so the inode number doesn't
matter and the patch is ok.
I'm just wondering, can we assume that the stat structure has always an
st_ino member? THere are checks for st_blocks and st_blksize in place
already. Is the same required for st_ino?
Corinna
--
Corinna Vinschen
Cygwin Project Co-Leader
Red Hat
More information about the Gdb-patches
mailing list