aboutsummaryrefslogtreecommitdiff
path: root/src/qt/bitcoinunits.cpp
diff options
context:
space:
mode:
authorWladimir J. van der Laan <laanwj@gmail.com>2011-07-27 20:49:14 +0200
committerWladimir J. van der Laan <laanwj@gmail.com>2011-07-27 20:52:00 +0200
commit384625c1a62968ed84fd9664a2c819d2d8ffd575 (patch)
tree5276d26e03b57de0faed9aec7ddbaa37597c258c /src/qt/bitcoinunits.cpp
parentdd61035645aea85b393965b4a047ce7f1ad65960 (diff)
downloadbitcoin-384625c1a62968ed84fd9664a2c819d2d8ffd575.tar.xz
also accept numbers without dot/decimals for parsing, fixes transaction filter row
Diffstat (limited to 'src/qt/bitcoinunits.cpp')
-rw-r--r--src/qt/bitcoinunits.cpp30
1 files changed, 23 insertions, 7 deletions
diff --git a/src/qt/bitcoinunits.cpp b/src/qt/bitcoinunits.cpp
index 7cd50232dd..9a9a4890dc 100644
--- a/src/qt/bitcoinunits.cpp
+++ b/src/qt/bitcoinunits.cpp
@@ -119,17 +119,33 @@ QString BitcoinUnits::formatWithUnit(int unit, qint64 amount, bool plussign)
bool BitcoinUnits::parse(int unit, const QString &value, qint64 *val_out)
{
- if(!valid(unit))
- return false; // Refuse to parse invalid unit
+ if(!valid(unit) || value.isEmpty())
+ return false; // Refuse to parse invalid unit or empty string
int num_decimals = decimals(unit);
QStringList parts = value.split(".");
- if(parts.size() != 2 || parts.at(1).size() > num_decimals)
- return false; // Max num decimals
+
+ if(parts.size() > 2)
+ {
+ return false; // More than one dot
+ }
+ QString whole = parts[0];
+ QString decimals;
+
+ if(parts.size() > 1)
+ {
+ decimals = parts[1];
+ }
+ if(decimals.size() > num_decimals)
+ {
+ return false; // Exceeds max precision
+ }
bool ok = false;
- QString str = parts[0] + parts[1].leftJustified(num_decimals, '0');
- if(str.size()>18)
- return false; // Bounds check
+ QString str = whole + decimals.leftJustified(num_decimals, '0');
+ if(str.size() > 18)
+ {
+ return false; // Longer numbers will exceed 63 bits
+ }
qint64 retvalue = str.toLongLong(&ok);
if(val_out)
{