[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 00/24] QemuOpts util/cutils: Fix and clean up num
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PATCH v2 00/24] QemuOpts util/cutils: Fix and clean up number conversions |
Date: |
Tue, 21 Feb 2017 21:13:44 +0100 |
QemuOpts has its own code to convert strings to numbers, and being
QemuOpts, it gets it wrong. util/cutils is less wrong. Fix it up
some, and reuse it for QemuOpts.
v2:
* PATCH 01+04: Use g_assert_cmphex() instead of g_assert_cmpuint() in
a few places
* PATCH 01+09: Cover invalid input and leading whitespace in tests
[Marc-André]
* PATCH 04: Prettier literal numbers
* PATCH 07+08: Rename to @libc_errno instead of @eno [Paolo]
* PATCH 15+18: Rename to qemu_strtosz_MiB() instead of _mebi() [Paolo]
* Commit messages massaged a bit
* R-bys retained
Markus Armbruster (24):
test-qemu-opts: Cover qemu_opts_parse()
option: Assert value string isn't null
test-cutils: Add missing qemu_strtol()... endptr checks
test-cutils: Clean up qemu_strtoul() result checks
util/cutils: Rewrite documentation of qemu_strtol() & friends
util/cutils: Rename qemu_strtoll(), qemu_strtoull()
util/cutils: Clean up variable names around qemu_strtol()
util/cutils: Clean up control flow around qemu_strtol() a bit
option: Fix to reject invalid and overflowing numbers
test-cutils: Add missing qemu_strtosz()... endptr checks
test-cutils: Cover qemu_strtosz() invalid input
test-cutils: Cover qemu_strtosz() with trailing crap
test-cutils: Cover qemu_strtosz() around range limits
util/cutils: New qemu_strtosz_metric()
util/cutils: Rename qemu_strtosz() to qemu_strtosz_MiB()
util/cutils: New qemu_strtosz()
util/cutils: Drop QEMU_STRTOSZ_DEFSUFFIX_* macros
test-cutils: Use qemu_strtosz() more often
test-cutils: Drop suffix from test_qemu_strtosz_simple()
qemu-img: Wrap cvtnum() around qemu_strtosz()
util/cutils: Let qemu_strtosz*() optionally reject trailing crap
util/cutils: Return qemu_strtosz*() error and value separately
util/cutils: Change qemu_strtosz*() from int64_t to uint64_t
option: Fix checking of sizes for overflow and trailing crap
hmp.c | 11 +-
hw/misc/ivshmem.c | 9 +-
include/qemu/cutils.h | 29 +--
monitor.c | 7 +-
qapi/opts-visitor.c | 11 +-
qemu-img.c | 62 ++---
qemu-io-cmds.c | 16 +-
qobject/qdict.c | 2 +-
qtest.c | 34 +--
target/i386/cpu.c | 9 +-
tests/test-cutils.c | 642 ++++++++++++++++++++++++++++++++-----------------
tests/test-qemu-opts.c | 306 ++++++++++++++++++++++-
util/cutils.c | 239 +++++++++---------
util/log.c | 4 +-
util/qemu-option.c | 87 +++----
15 files changed, 976 insertions(+), 492 deletions(-)
--
2.7.4
- [Qemu-devel] [PATCH v2 00/24] QemuOpts util/cutils: Fix and clean up number conversions,
Markus Armbruster <=
- [Qemu-devel] [PATCH v2 04/24] test-cutils: Clean up qemu_strtoul() result checks, Markus Armbruster, 2017/02/21
- [Qemu-devel] [PATCH v2 17/24] util/cutils: Drop QEMU_STRTOSZ_DEFSUFFIX_* macros, Markus Armbruster, 2017/02/21
- [Qemu-devel] [PATCH v2 01/24] test-qemu-opts: Cover qemu_opts_parse(), Markus Armbruster, 2017/02/21
- [Qemu-devel] [PATCH v2 07/24] util/cutils: Clean up variable names around qemu_strtol(), Markus Armbruster, 2017/02/21
- [Qemu-devel] [PATCH v2 11/24] test-cutils: Cover qemu_strtosz() invalid input, Markus Armbruster, 2017/02/21
- [Qemu-devel] [PATCH v2 10/24] test-cutils: Add missing qemu_strtosz()... endptr checks, Markus Armbruster, 2017/02/21
- [Qemu-devel] [PATCH v2 05/24] util/cutils: Rewrite documentation of qemu_strtol() & friends, Markus Armbruster, 2017/02/21
- [Qemu-devel] [PATCH v2 12/24] test-cutils: Cover qemu_strtosz() with trailing crap, Markus Armbruster, 2017/02/21
- [Qemu-devel] [PATCH v2 08/24] util/cutils: Clean up control flow around qemu_strtol() a bit, Markus Armbruster, 2017/02/21