[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] block: handle filenames with colons better
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-devel] [PATCH] block: handle filenames with colons better |
Date: |
Thu, 16 Aug 2012 23:24:11 +0400 |
User-agent: |
Mozilla/5.0 (X11; Linux i686 on x86_64; rv:10.0.5) Gecko/20120624 Icedove/10.0.5 |
On 16.08.2012 18:58, Iustin Pop wrote:
> Commit 947995c (block: protect path_has_protocol from filenames with
> colons) introduced a way to handle filenames with colons based on
> whether the path contains a slash or not. IMHO this is not optimal,
> since we shouldn't rely on the contents of the path but rather on
> whether the given path exists as a file or not.
>
> As such, this patch tries to handle both files with and without
> slashes by falling back to opening them as files if no drivers
> supporting the protocol has been identified.
I for one dislike this idea entirely: I think there should be a
way to stop qemu from trying to open something as a file. It
opens a security hole after all, "what if" such a file will actually
exist?
If I can vote, I'm voting against this with both hands.
Thanks,
/mjt