diff options
author | Per Åstrand <per.astrand@arm.com> | 2022-05-18 09:09:01 +0200 |
---|---|---|
committer | Kristofer Jonsson <kristofer.jonsson@arm.com> | 2022-05-19 06:07:52 +0000 |
commit | 419993512e11643d26fbc56d3465022a696d2108 (patch) | |
tree | 3475be1b61f4871eb1e4ea93a4da27151246bd9e /applications | |
parent | b05507295d2c13713c03bff54b9a5effe413b7de (diff) | |
download | ethos-u-core-platform-419993512e11643d26fbc56d3465022a696d2108.tar.gz |
Handle network indices out of range
Check that getting the network succeeds before parsing the the buffer to
avoid hardfault.
Change-Id: I582412fb0e7459bf45fdb790df2d48401c1a2d0b
Diffstat (limited to 'applications')
-rw-r--r-- | applications/message_handler/message_handler.cpp | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/applications/message_handler/message_handler.cpp b/applications/message_handler/message_handler.cpp index f109dc8..4a3938e 100644 --- a/applications/message_handler/message_handler.cpp +++ b/applications/message_handler/message_handler.cpp @@ -383,12 +383,16 @@ void IncomingMessageHandler::sendNetworkInfoRsp(uint64_t userArg, ethosu_core_ne void *buffer; size_t size; - getNetwork(network, buffer, size); - bool failed = parser.parseModel(buffer, - rsp.desc, - InferenceProcess::makeArray(rsp.ifm_size, rsp.ifm_count, ETHOSU_CORE_BUFFER_MAX), - InferenceProcess::makeArray(rsp.ofm_size, rsp.ofm_count, ETHOSU_CORE_BUFFER_MAX)); - rsp.status = failed ? ETHOSU_CORE_STATUS_ERROR : ETHOSU_CORE_STATUS_OK; + + bool failed = getNetwork(network, buffer, size); + + if (!failed) { + failed = parser.parseModel(buffer, + rsp.desc, + InferenceProcess::makeArray(rsp.ifm_size, rsp.ifm_count, ETHOSU_CORE_BUFFER_MAX), + InferenceProcess::makeArray(rsp.ofm_size, rsp.ofm_count, ETHOSU_CORE_BUFFER_MAX)); + } + rsp.status = failed ? ETHOSU_CORE_STATUS_ERROR : ETHOSU_CORE_STATUS_OK; if (!outputMessageQueue.write(ETHOSU_CORE_MSG_NETWORK_INFO_RSP, rsp)) { printf("ERROR: Msg: Failed to write network info response. No mailbox message sent\n"); |