summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlex Trotsenko <alex1973tr@gmail.com>2017-04-25 19:28:04 +0300
committerAlex Trotsenko <alex1973tr@gmail.com>2017-04-25 17:27:35 +0000
commitd88ded29d968b8ccf41d4dffa73c3f8f030a9f5d (patch)
tree22d785d17d2fe951fa05732004bff581f020d2a3
parent225ef1baeb0e49f157f3ba1271915e623bd27dd9 (diff)
downloadqtserialport-d88ded29d968b8ccf41d4dffa73c3f8f030a9f5d.tar.gz
Fix possible UB in ReadFile()v5.9.0-beta4
This patch replicates commit 225ef1baeb0e49f157f3ba1271915e623bd27dd9 for Windows. Change-Id: Ic4e3444183289e7ce1a487500633fb5ad95590b9 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>
-rw-r--r--src/serialport/qserialport_win.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/serialport/qserialport_win.cpp b/src/serialport/qserialport_win.cpp
index 84bf5b9..0494889 100644
--- a/src/serialport/qserialport_win.cpp
+++ b/src/serialport/qserialport_win.cpp
@@ -545,7 +545,7 @@ bool QSerialPortPrivate::startAsyncRead()
if (readBufferMaxSize && bytesToRead > (readBufferMaxSize - buffer.size())) {
bytesToRead = readBufferMaxSize - buffer.size();
- if (bytesToRead == 0) {
+ if (bytesToRead <= 0) {
// Buffer is full. User must read data from the buffer
// before we can read more from the port.
return false;