[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Fix double backslash problem in Windows
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH] Fix double backslash problem in Windows |
Date: |
Wed, 09 Jan 2008 20:11:07 +0100 |
Le mercredi 09 janvier 2008 à 11:53 -0600, Anthony Liguori a écrit :
> Laurent Vivier wrote:
> > Le mercredi 09 janvier 2008 à 11:40 +0100, andrzej zaborowski a écrit :
> >
> >> On 09/01/2008, Laurent Vivier <address@hidden> wrote:
> >>
> >>> Le mercredi 09 janvier 2008 à 10:31 +0100, Laurent Vivier a écrit :
> >>>
> >>>> Le mardi 08 janvier 2008 à 17:17 +0100, Hervé Poussineau a écrit :
> >>>>
> >>>>> Hi,
> >>>>>
> >>>>> On Windows, since December 2nd, files names provided in command line
> >>>>> have to double their backslash to work correctly, for example: "-hda
> >>>>> c:\\disks\\qemu.qcow" instead of -hda c:\disks\qemu.qcow"
> >>>>> This patch removes this need and reverts back to Qemu 0.9.0 behaviour
> >>>>>
> >>>>> Hervé
> >>>>>
> >>>>>
> >>>> I have introduced this behavior to be able to use command line like
> >>>> "qemu -hda my\ file", IMHO your patch should be #ifdef for window only.
> >>>>
> >>> In fact, this is a wrong example, this case is managed by the shell.
> >>> A good example is when we have a filename with a '"' in it:
> >>>
> >>> qemu -hda 2\\\"file
> >>>
> >>> to open file 2"file
> >>>
> >> And the correct behaviour for that would be to open the file 2\"file, while
> >>
> >> qemu -hda 2\"file
> >>
> >> should open 2"file. The only character that we may need to handle
> >> specially should be the comma, I don't know if anyone cares.
> >>
> >
> > You're right...
> > but "-hda" is an alias for "-drive file="%s",index=%d,media=disk".
> >
> > So when you type "qemu -hda 2\"file",
> > it becomes "qemu -drive file="2"file",index=0,media=disk"
> > which gives filename equal to "2file,index=0,media=disk" instead of
> > filename equal to 2"file with option index and media.
> >
>
> The proper solution is to escape the files before doing the snprintf().
What do you call "to escape the files" ?
Regards,
Laurent
--
----------------- address@hidden ------------------
"La perfection est atteinte non quand il ne reste rien à
ajouter mais quand il ne reste rien à enlever." Saint Exupéry
signature.asc
Description: Ceci est une partie de message numériquement signée
- Re: [Qemu-devel] Fix double backslash problem in Windows, (continued)
- [Qemu-devel] Re: [PATCH] Fix double backslash problem in Windows, consul, 2008/01/10
- Re: [Qemu-devel] Re: [PATCH] Fix double backslash problem in Windows, Laurent Vivier, 2008/01/10
- Re: [Qemu-devel] [PATCH] Fix double backslash problem in Windows, Andreas Färber, 2008/01/09
- Re: [Qemu-devel] [PATCH] Fix double backslash problem in Windows, Laurent Vivier, 2008/01/09
- Re: [Qemu-devel] [PATCH] Fix double backslash problem in Windows, andrzej zaborowski, 2008/01/09
- Re: [Qemu-devel] [PATCH] Fix double backslash problem in Windows, Anthony Liguori, 2008/01/09
- Re: [Qemu-devel] [PATCH] Fix double backslash problem in Windows, Johannes Schindelin, 2008/01/09
- Re: [Qemu-devel] [PATCH] Fix double backslash problem in Windows,
Laurent Vivier <=