diff options
author | Alex Blasche <alexander.blasche@theqtcompany.com> | 2015-12-18 14:12:29 +0100 |
---|---|---|
committer | Alex Blasche <alexander.blasche@theqtcompany.com> | 2015-12-18 13:24:02 +0000 |
commit | 2bb7b51463767233e4490dd25433c6c183da9192 (patch) | |
tree | 59e4e305cc474fd7a5ad7f5d2600322fec9e137d /src/plugins/canbus/socketcan/socketcanbackend.cpp | |
parent | 8e8e27abaeb6fbf5bc9fdf6ca937aff74d21244c (diff) |
Don't attempt to write an invalid QCanBusFrame
Change-Id: I9b73c9bdfe933a74302b4871aa40d5a025496b8c
Reviewed-by: Rolf Eike Beer <eb@emlix.com>
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
Diffstat (limited to 'src/plugins/canbus/socketcan/socketcanbackend.cpp')
-rw-r--r-- | src/plugins/canbus/socketcan/socketcanbackend.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/plugins/canbus/socketcan/socketcanbackend.cpp b/src/plugins/canbus/socketcan/socketcanbackend.cpp index 49d641f..0105dae 100644 --- a/src/plugins/canbus/socketcan/socketcanbackend.cpp +++ b/src/plugins/canbus/socketcan/socketcanbackend.cpp @@ -319,6 +319,11 @@ bool SocketCanBackend::writeFrame(const QCanBusFrame &newData) if (state() != ConnectedState) return false; + if (!newData.isValid()) { + setError(tr("Cannot write invalid QCanBusFrame"), QCanBusDevice::WriteError); + return false; + } + canid_t canId = newData.frameId(); if (newData.hasExtendedFrameFormat()) canId |= CAN_EFF_FLAG; |