diff options
author | Andrew Waterman <andrew@sifive.com> | 2017-05-01 16:44:47 -0700 |
---|---|---|
committer | Andrew Waterman <andrew@sifive.com> | 2017-05-01 16:44:47 -0700 |
commit | 63c98b41ae2d36808ebbb57e7f8e871ba247d444 (patch) | |
tree | 3649d1d9fedb439aecfd8e98cece60f98ca30357 /riscv/devices.cc | |
parent | 75f2a05df9cdff6f3faba748065b3184b9f01b01 (diff) | |
download | spike-63c98b41ae2d36808ebbb57e7f8e871ba247d444.zip spike-63c98b41ae2d36808ebbb57e7f8e871ba247d444.tar.gz spike-63c98b41ae2d36808ebbb57e7f8e871ba247d444.tar.bz2 |
Fix segfault when accessing bad memory addresses
Diffstat (limited to 'riscv/devices.cc')
-rw-r--r-- | riscv/devices.cc | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/riscv/devices.cc b/riscv/devices.cc index af4dc7d..15115c8 100644 --- a/riscv/devices.cc +++ b/riscv/devices.cc @@ -21,13 +21,10 @@ bool bus_t::store(reg_t addr, size_t len, const uint8_t* bytes) return it->second->store(addr - -it->first, len, bytes); } -bus_t::descriptor bus_t::find_device(reg_t addr) +std::pair<reg_t, abstract_device_t*> bus_t::find_device(reg_t addr) { auto it = devices.lower_bound(-addr); - if (it == devices.end()) { - bus_t::descriptor desc = {0, 0}; - return desc; - } - bus_t::descriptor desc = {-it->first, it->second}; - return desc; + if (it == devices.end() || addr < -it->first) + return std::make_pair((reg_t)0, (abstract_device_t*)NULL); + return std::make_pair(-it->first, it->second); } |