[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 5/5] 9p: forbid empty extension string
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2 5/5] 9p: forbid empty extension string |
Date: |
Mon, 29 Aug 2016 14:35:07 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 |
On 08/28/2016 12:34 PM, Greg Kurz wrote:
>>
>> For the sake of simplicity, I'd rather have the target names to follow the
>> same rules as other file names, and return ENOENT directly (the link you
>> provide states it is a valid option).
>>
>> Peter,
>>
>> Since you suggested to do explicit error checking on empty file names, do
>> you have an opinion on the case of symlinks with an empty target ?
Failing with ENOENT for both "" and for "a/b" seems reasonable to me (a
directory entry named "a/b" can never exist, just as "" cannot exist).
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v2 3/5] 9p: forbid . and .. in file names, (continued)
- [Qemu-devel] [PATCH v2 3/5] 9p: forbid . and .. in file names, Greg Kurz, 2016/08/26
- [Qemu-devel] [PATCH v2 4/5] 9p: handle walk of ".." in the root directory, Greg Kurz, 2016/08/26
- [Qemu-devel] [PATCH v2 5/5] 9p: forbid empty extension string, Greg Kurz, 2016/08/26
- Re: [Qemu-devel] [PATCH v2 5/5] 9p: forbid empty extension string, Eric Blake, 2016/08/26
- Re: [Qemu-devel] [PATCH v2 5/5] 9p: forbid empty extension string, Michael S. Tsirkin, 2016/08/26
- Re: [Qemu-devel] [PATCH v2 5/5] 9p: forbid empty extension string, Greg Kurz, 2016/08/28
- Re: [Qemu-devel] [PATCH v2 5/5] 9p: forbid empty extension string, Greg Kurz, 2016/08/28
- Re: [Qemu-devel] [PATCH v2 5/5] 9p: forbid empty extension string,
Eric Blake <=
- Re: [Qemu-devel] [PATCH v2 5/5] 9p: forbid empty extension string, Greg Kurz, 2016/08/30
- Re: [Qemu-devel] [PATCH v2 5/5] 9p: forbid empty extension string, Peter Maydell, 2016/08/28