[Qemu-devel] [PATCH 0/2] qemu-img: fix coverity nits

Peter Maydell posted 2 patches 7 years, 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1486744104-15590-1-git-send-email-peter.maydell@linaro.org
Test checkpatch passed
Test docker passed
Test s390x passed
qemu-img.c | 40 ++++++++++++++++++++--------------------
1 file changed, 20 insertions(+), 20 deletions(-)
[Qemu-devel] [PATCH 0/2] qemu-img: fix coverity nits
Posted by Peter Maydell 7 years, 1 month ago
Coverity points out a couple of nits in qemu-img:
 * misuse of strtoul()
 * variable set but not used

These patches fix the issues.

Peter Maydell (2):
  qemu-img: Use qemu_strtoul() rather than raw strtoul()
  qemu-img: Avoid setting ret to unused value in img_convert()

 qemu-img.c | 40 ++++++++++++++++++++--------------------
 1 file changed, 20 insertions(+), 20 deletions(-)

-- 
2.7.4


Re: [Qemu-devel] [PATCH 0/2] qemu-img: fix coverity nits
Posted by Max Reitz 7 years, 1 month ago
On 10.02.2017 17:28, Peter Maydell wrote:
> Coverity points out a couple of nits in qemu-img:
>  * misuse of strtoul()
>  * variable set but not used
> 
> These patches fix the issues.
> 
> Peter Maydell (2):
>   qemu-img: Use qemu_strtoul() rather than raw strtoul()
>   qemu-img: Avoid setting ret to unused value in img_convert()
> 
>  qemu-img.c | 40 ++++++++++++++++++++--------------------
>  1 file changed, 20 insertions(+), 20 deletions(-)

Thanks, applied to my block tree:

https://github.com/XanClic/qemu/commits/block

Max