Re: [PATCH] Strip control codes in virBufferEscapeString

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

 



On 03/30/2015 09:50 AM, Daniel Veillard wrote:

>> NACK.  Stripping control codes from a volume name represents the wrong
>> name.  We need to escape the problematic bytes, rather than strip them.
> 
>   you can't escape them with a CharRef for sure
> 
> http://www.w3.org/TR/REC-xml/#wf-Legalchar
> Characters referred to using character references must match the
> production for Char.
> 
>   That time Ján  is right :-)

Ouch.  Then how do we represent the name of a storage volume, when the
file system allows arbitrary bytes including control characters, in the
volume name, but where we are restricted to only using valid XML?  Do we
just silently ignore such files as impossible volumes that libvirt
cannot manage?  (I'd rather omit such a volume from the list in the
pool, than silently munge its name into something incorrect)

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]