aly/servatrice
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Added additional bound checks.

Browse source
This commit is contained in:
Mark Morschhäuser 2014-03-20 15:37:21 +01:00
parent fbf54b62bc
commit f964dcc656
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
common/rng_sfmt.cpp
View file

@ -61,7 +61,7 @@ RNG_SFMT::RNG_SFMT(QObject *parent)
unsigned int RNG_SFMT::getNumber(unsigned int min, unsigned int max) unsigned int RNG_SFMT::getNumber(unsigned int min, unsigned int max)
{ {
// This all makes no sense if min > max, which should never happen. // This all makes no sense if min > max, which should never happen.
if(min > max) { if(min > max || max >= UINT_MAX || min >= UINT_MAX) {
throw std::invalid_argument( throw std::invalid_argument(
QString("Invalid bounds for RNG: min > max! Values were: min = " + QString("Invalid bounds for RNG: min > max! Values were: min = " +
QString::number(min) + ", max = " + QString::number(min) + ", max = " +