diff options
author | Max Kellermann <max@duempel.org> | 2014-09-04 14:38:55 +0200 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2014-09-04 17:37:31 +0200 |
commit | 421c4ae907e27661902f28c07c1c470c3dba3cf7 (patch) | |
tree | 5dc7c244df11cdfdb7ff1a3ccdf63f02011b6095 /src/DecoderThread.cxx | |
parent | 4907f610d6116949111fb6ff81c1489ec68b9d43 (diff) | |
download | mpd-421c4ae907e27661902f28c07c1c470c3dba3cf7.tar.gz mpd-421c4ae907e27661902f28c07c1c470c3dba3cf7.tar.xz mpd-421c4ae907e27661902f28c07c1c470c3dba3cf7.zip |
protocol/ArgParser: fix integer overflow in parse_range()
Casting std::numeric_limits<unsigned>::max() to "long" leads to an
overflow if sizeof(unsigned)==sizeof(long), and the result will be -1.
This happens on some 32 bit architectures, for example ARM and WIN32.
Workaround: use std::numeric_limits<int>::max(), which is the largest
signed integer. Since sizeof(long)>=sizeof(int), this will never
overflow.
Fixes Mantis ticket 0004080.
Diffstat (limited to 'src/DecoderThread.cxx')
0 files changed, 0 insertions, 0 deletions