This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB 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: RFC: Check permissions of .gdbinit files


> Date: Mon, 30 May 2005 18:42:00 -0400
> From: Daniel Jacobowitz <drow@false.org>
> Cc: gdb-patches@sourceware.org
> 
> Bother; I thought about the portability for a while, but didn't quite
> consider this.  We're still OK though - the whole thing is surrounded
> by HAVE_GETUID, and MinGW does not have GETUID, if I understand
> correctly.

Maybe, I don't know.  Isn't the MinGW port linked against some
library, such as libgw32c.a, that implements more Posix stuff?

In any case, the other issue still remains: if they do have getuid and
S_IWOTH, non-readonly files will be reported world-writable.  So I'd
suggest to either disable this feature entirely on Windows platforms,
or write a Windows specific code that uses the Win32 API to get file
ownership (GetSecurityInfo or some such).

> > > +    error (_("source command requires pathname of file to source."));
> > 
> > I think the message text should begin with a capital letter (yes, I
> > know the original didn't do it, either).
> 
> I was pretty sure that the convention was lowercase letter, and no
> trailing period.  But I don't think this is documented anywhere, and
> the source is wildly inconsistent.  I could easily be remembering wrong
> :-)

I'm not aware of any such conventions.  Starting a sentence with a
capital letter sounds like the right choice, though.

> > Could you please tell why you replaced `fopen' with `open'+`fdopen'?
> 
> In order to have the file descriptor, for fstat.  It occurs to me now
> that "fileno" could be used for this; I am not sure how portable fileno
> is

`fileno' is defined by Posix, so we definitely can use it.

> > Last, but not least: if we decide to make such a change (which to my
> > HO sounds like a good idea, in general), we should describe this
> > subtlety (and the warning it could produce) in the user's manual.
> 
> Where would you suggest?  Ah, never mind, I see there is a section on
> .gdbinit already.

Right.


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