mini-gmp: mpz_init_set_str fails on leading zeroes

Niels Möller nisse at lysator.liu.se
Sun Aug 14 14:40:41 UTC 2016


Austyn Krutsinger <akrutsinger at gmail.com> writes:

> I think the attached patch provides a reasonable solution. The only
> function effected is mpn_set_str_other() and since this particular function
> is unique to mini-gmp, 

I'm looking into this now. I'm leaning towards lettong mpn_set_str_other
return 1 for all-zero input, and let the caller, mpz_set_str, deal with
normalization (mpn_set_str doesn't need to, since it doesn't promise
normalization if the input has leading zeros).

I also found another corner case where the current implementation is
broken: Calling mpz_set_str with an empty string as input. Have to
check if the real gmp treats that as invalid or as zero.

Regards,
/Niels

-- 
Niels Möller. PGP-encrypted email is preferred. Keyid C0B98E26.
Internet email is subject to wholesale government surveillance.


More information about the gmp-bugs mailing list