qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v2 4/5] block: qemu-iotests - fix image cleanup


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH v2 4/5] block: qemu-iotests - fix image cleanup when using spaced pathnames
Date: Thu, 10 Apr 2014 08:48:10 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0

On 04/10/2014 08:43 AM, Eric Blake wrote:
> On 04/10/2014 06:53 AM, Jeff Cody wrote:
> 
>>>> +++ b/tests/qemu-iotests/common.rc
>>>> @@ -178,10 +178,10 @@ _rm_test_img()
>>>>      local img=$1
>>>
>>> Since we are quoting $img, should we quote $1 as well?
>>>

> 
> http://austingroupbugs.net/view.php?id=351
> 
> But even with the notion of an assignment-context argument added to a
> future version of POSIX, the reality is that given the present standard,
> it's safer to either use "" to ensure no word splitting:

Well, if you were trying to be portable to multiple shells, then it
would matter.  But as this script is explicitly being run under
/bin/bash, and as bash already has support for declaration utilities
where local is one such utility, your script as written is safe without
"" in the arguments to local.  So I'm fine whether you choose to change
it in a respin or to leave it as written in this version.

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

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]