[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qemu_find_file: check name as a straight path e
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] qemu_find_file: check name as a straight path even if it has no '/' |
Date: |
Tue, 10 Jul 2012 15:18:50 +0100 |
Ping^3? This patch has been on the list over six weeks now...
-- PMM
On 28 June 2012 12:23, Peter Maydell <address@hidden> wrote:
> Ping^2?
>
> -- PMM
>
> On 19 June 2012 16:13, Peter Maydell <address@hidden> wrote:
>> Ping? (patch still applies cleanly to current master; patchwork URL
>> http://patchwork.ozlabs.org/patch/161324/ )
>>
>> -- PMM
>>
>> On 25 May 2012 13:07, Peter Maydell <address@hidden> wrote:
>>> Make qemu_find_file() check for the passed in name as a straight
>>> pathname even if it doesn't have any path separator character in it.
>>> This means that "-bios foo", "-dtb foo" etc will find a file 'foo'
>>> in the current directory.
>>> This removes an inconsistency with -kernel and -initrd, which both
>>> accept plain filenames as meaning files in the current directory.
>>> It's also less confusing for the user than an undocumented restriction
>>> that "this option accepts a filename, except for the special case
>>> where the filename you pass happens not to have a '/' in it, in
>>> which case we'll ignore it."
>>>
>>> Signed-off-by: Peter Maydell <address@hidden>
>>> ---
>>> vl.c | 5 ++---
>>> 1 files changed, 2 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/vl.c b/vl.c
>>> index 23ab3a3..4639526 100644
>>> --- a/vl.c
>>> +++ b/vl.c
>>> @@ -1801,9 +1801,8 @@ char *qemu_find_file(int type, const char *name)
>>> const char *subdir;
>>> char *buf;
>>>
>>> - /* If name contains path separators then try it as a straight path. */
>>> - if ((strchr(name, '/') || strchr(name, '\\'))
>>> - && access(name, R_OK) == 0) {
>>> + /* Try the name as a straight path first */
>>> + if (access(name, R_OK) == 0) {
>>> return g_strdup(name);
>>> }
>>> switch (type) {
>>> --
>>> 1.7.1
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH] qemu_find_file: check name as a straight path even if it has no '/',
Peter Maydell <=