NumericBox unsafe double conversion

Stephan Bergmann sbergman at redhat.com
Fri Mar 22 04:27:46 PDT 2013


On 03/22/2013 11:44 AM, Zolnai Tamás wrote:
> Otherwise in ImplNumericGetValue() the problem of overflow/underflow can
> be avoid with these lines
> sal_Int64 nValue = aStr.toInt64()
> if( OUString::valueOf(nValue) != aStr )
> {
>      if( bNegative )
>          rValue = SAL_MIN_INT64;
>      else
>          rValue = SAL_MAX_INT64;
>      return sal_True;
> }

...but only if aStr is known to be in canonical form, esp. neither 
contains minus zero nor contains excessive leading zero digits.  None of 
that is guaranteed for ImplNumericGetValue in its current form (and also 
consider nDecDigits > 0, where input rStr="0" and nDecDigits=2, say, 
would lead to aStr="000" and the above comparison erroneously failing).

Stephan



More information about the LibreOffice mailing list