SocketCAN: Set unused can{fd}_frame fields to zero when writing

At the moment, this is most important for canfd_frame.flags.
For other reserved fields this might be important later, too.

Change-Id: I66013a5ddac1f2f21a03c28cdbe9ac6719a604ce
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>
Reviewed-by: Rolf Eike Beer <eb@emlix.com>
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

1 files changed, 2 insertions, 0 deletions

diff --git a/src/plugins/canbus/socketcan/socketcanbackend.cpp b/src/plugins/canbus/socketcan/socketcanbackend.cpp
index 8974279..3226e0e 100644
--- a/src/plugins/canbus/socketcan/socketcanbackend.cpp
+++ b/src/plugins/canbus/socketcan/socketcanbackend.cpp
@@ -380,6 +380,7 @@ bool SocketCanBackend::writeFrame(const QCanBusFrame &newData)
     qint64 bytesWritten = 0;
     if (newData.hasFlexibleDataRateFormat()) {
         canfd_frame frame;
+        memset(&frame, 0, sizeof(frame));
         frame.len = newData.payload().size();
         frame.can_id = canId;
         ::memcpy(frame.data, newData.payload().constData(), frame.len);
@@ -387,6 +388,7 @@ bool SocketCanBackend::writeFrame(const QCanBusFrame &newData)
         bytesWritten = ::write(canSocket, &frame, sizeof(frame));
     } else {
         can_frame frame;
+        memset(&frame, 0, sizeof(frame));
         frame.can_dlc = newData.payload().size();
         frame.can_id = canId;
         ::memcpy(frame.data, newData.payload().constData(), frame.can_dlc);