dbacc54 qemu-io: Use purely string blockdev options

1 file Authored by Max Reitz 5 years ago, Committed by Michael Roth 5 years ago,
1 file changed. 2 lines added. 2 lines removed.
    qemu-io: Use purely string blockdev options
    
    Currently, qemu-io only uses string-valued blockdev options (as all are
    converted directly from QemuOpts) -- with one exception: -U adds the
    force-share option as a boolean.  This in itself is already a bit
    questionable, but a real issue is that it also assumes the value already
    existing in the options QDict would be a boolean, which is wrong.
    
    That has the following effect:
    
    $ ./qemu-io -r -U --image-opts \
        driver=file,filename=/dev/null,force-share=off
    [1]    15200 segmentation fault (core dumped)  ./qemu-io -r -U
    --image-opts driver=file,filename=/dev/null,force-share=off
    
    Since @opts is converted from QemuOpts, the value must be a string, and
    we have to compare it as such.  Consequently, it makes sense to also set
    it as a string instead of a boolean.
    
    Cc: qemu-stable@nongnu.org
    Signed-off-by: Max Reitz <mreitz@redhat.com>
    Message-id: 20180502202051.15493-2-mreitz@redhat.com
    Reviewed-by: Eric Blake <eblake@redhat.com>
    Signed-off-by: Max Reitz <mreitz@redhat.com>
    (cherry picked from commit 2a01c01f9ecb43af4c0a85fe6adc429ffc9c31b5)
    Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
    
        
file modified
+2 -2