mbox series

[kvm-unit-tests,0/2] Introduce strtoll/strtoull

Message ID 20211013164259.88281-1-drjones@redhat.com (mailing list archive)
Headers show
Series Introduce strtoll/strtoull | expand

Message

Andrew Jones Oct. 13, 2021, 4:42 p.m. UTC
A recent posting by Daniele Ahmed inspired me to write a patch adding
strtoll/strtoull. While doing that I noticed check_mul_overflow wasn't
working and found copy+paste errors with it and check_sub_overflow.

Andrew Jones (2):
  compiler.h: Fix typos in mul and sub overflow checks
  lib: Introduce strtoll/strtoull

 lib/linux/compiler.h |  4 ++--
 lib/stdlib.h         |  2 ++
 lib/string.c         | 51 ++++++++++++++++++++++++++++++++------------
 3 files changed, 41 insertions(+), 16 deletions(-)

Comments

Paolo Bonzini Oct. 15, 2021, 9:15 a.m. UTC | #1
On 13/10/21 18:42, Andrew Jones wrote:
> A recent posting by Daniele Ahmed inspired me to write a patch adding
> strtoll/strtoull. While doing that I noticed check_mul_overflow wasn't
> working and found copy+paste errors with it and check_sub_overflow.
> 
> Andrew Jones (2):
>    compiler.h: Fix typos in mul and sub overflow checks
>    lib: Introduce strtoll/strtoull
> 
>   lib/linux/compiler.h |  4 ++--
>   lib/stdlib.h         |  2 ++
>   lib/string.c         | 51 ++++++++++++++++++++++++++++++++------------
>   3 files changed, 41 insertions(+), 16 deletions(-)
> 

Queued, thanks.

Paolo