Re: [spice-xpi 5/5] Add glib Windows logging

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




----- Mensaje original -----
> On Mon, Mar 25, 2013 at 12:18:32PM +0100, Marc-André Lureau wrote:
> > On Mon, Mar 25, 2013 at 11:50 AM, Christophe Fergeau
> > <cfergeau@xxxxxxxxxx> wrote:
> > > I've considered that, but this means I'd have to sanitize the
> > > filename for
> > > \, '..', and probably some other things I didn't think of. As
> > > this is just
> > > a debugging help, I've chosen to go with a hardcoded filename
> > > which does
> > > not need additional care. I can change that if you feel strongly
> > > about it.
> > 
> > Imho, there is no need for that. If the user provide a wrong
> > filename,
> > it should fail.
> 
> Well, setting it to ..\..\..\bootmgr could lead to interesting
> results, and
> I'd like to protect against such issues if I add support for an
> arbitrary
> filename.

This is clearly not your responsibility at this point. This is the job of the OS.
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]