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