Merge remote-tracking branch 'origin/5.9' into 5.10v5.10.0-beta4

 Conflicts:
	.qmake.conf

Change-Id: Icfbad81955ad8a7c4ddc49ac4ee952bcba6317ec

1 files changed, 13 insertions, 1 deletions

diff --git a/src/plugins/canbus/peakcan/peakcanbackend.cpp b/src/plugins/canbus/peakcan/peakcanbackend.cpp
index 8f47343..8f97f5d 100644
--- a/src/plugins/canbus/peakcan/peakcanbackend.cpp
+++ b/src/plugins/canbus/peakcan/peakcanbackend.cpp
@@ -272,7 +272,6 @@ bool PeakCanBackendPrivate::open()
             return false;
         }
     }
-#endif
 
     const TPCANStatus err = ::CAN_SetValue(channelIndex, PCAN_RECEIVE_EVENT, &readHandle, sizeof(readHandle));
     if (Q_UNLIKELY(err != PCAN_ERROR_OK)) {
@@ -280,6 +279,14 @@ bool PeakCanBackendPrivate::open()
         return false;
     }
 
+#else
+    const TPCANStatus err = ::CAN_GetValue(channelIndex, PCAN_RECEIVE_EVENT, &readHandle, sizeof(readHandle));
+    if (Q_UNLIKELY(err != PCAN_ERROR_OK)) {
+        q->setError(systemErrorString(err), QCanBusDevice::ConnectionError);
+        return false;
+    }
+#endif
+
     writeNotifier = new WriteNotifier(this, q);
     writeNotifier->setInterval(0);
 
@@ -413,6 +420,11 @@ void PeakCanBackendPrivate::startRead()
             break;
         }
 
+        // Filter out PCAN status frames, to avoid turning them
+        // into QCanBusFrame::DataFrames with random canId
+        if (Q_UNLIKELY(message.MSGTYPE & PCAN_MESSAGE_STATUS))
+            continue;
+
         QCanBusFrame frame(message.ID, QByteArray(reinterpret_cast<const char *>(message.DATA), int(message.LEN)));
         const quint64 millis = timestamp.millis + Q_UINT64_C(0xFFFFFFFF) * timestamp.millis_overflow;
         const quint64 micros = Q_UINT64_C(1000) * millis + timestamp.micros;