diff options
| author | Eric Blake <eblake@redhat.com> | 2023-05-22 14:04:39 -0500 |
|---|---|---|
| committer | Eric Blake <eblake@redhat.com> | 2023-06-02 12:29:27 -0500 |
| commit | b87ac96651054fa89baab4e3a88a7feee7f92314 (patch) | |
| tree | 912cb79075e7871787546deba4e37625b902c1ce /net/vmnet-shared.c | |
| parent | 3c5f2467984c23aae5a64548dcb15efae18e207e (diff) | |
| download | qemu-b87ac96651054fa89baab4e3a88a7feee7f92314.zip qemu-b87ac96651054fa89baab4e3a88a7feee7f92314.tar.gz qemu-b87ac96651054fa89baab4e3a88a7feee7f92314.tar.bz2 | |
cutils: Use parse_uint in qemu_strtosz for negative rejection
Rather than open-coding two different ways to check for an unwanted
negative sign, reuse the same code in both functions. That way, if we
decide down the road to accept "-0" instead of rejecting it, we have
fewer places to change. Also, it means we now get ERANGE instead of
EINVAL for negative values in qemu_strtosz, which is reasonable for
what it represents. This in turn changes the expected output of a
couple of iotests.
The change is not quite complete: negative fractional scaled values
can trip us up. This will be fixed in a later patch addressing other
issues with fractional scaled values.
Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Hanna Czenczek <hreitz@redhat.com>
Message-Id: <20230522190441.64278-18-eblake@redhat.com>
Diffstat (limited to 'net/vmnet-shared.c')
0 files changed, 0 insertions, 0 deletions