Jan/WLED_MM_Infinity
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

free udp receive buffers before parsePacket

Browse Source 
robustness improvement - parsepacket() will fail if the udp handler still has an active receive buffer.
This commit is contained in:
Frank
2023-06-21 00:29:16 +02:00
parent 8aef4347a1
commit 708cd8e73d
2 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
wled00/ntp.cpp
View File

@@ -226,6 +226,7 @@ void sendNTPPacket()
bool checkNTPResponse() bool checkNTPResponse()
{ {
ntpUdp.flush();
int cb = ntpUdp.parsePacket(); int cb = ntpUdp.parsePacket();
if (!cb) return false; if (!cb) return false;

3
wled00/udp.cpp
View File

@@ -240,8 +240,10 @@ void handleNotifications()
if (!udpConnected) return; if (!udpConnected) return;
bool isSupp = false; bool isSupp = false;
notifierUdp.flush();
int packetSize = notifierUdp.parsePacket(); // WLEDMM function returns int, not size_t int packetSize = notifierUdp.parsePacket(); // WLEDMM function returns int, not size_t
if ((packetSize < 1) && udp2Connected) { if ((packetSize < 1) && udp2Connected) {
notifier2Udp.flush();
packetSize = notifier2Udp.parsePacket(); packetSize = notifier2Udp.parsePacket();
isSupp = true; isSupp = true;
} }
@@ -249,6 +251,7 @@ void handleNotifications()
//hyperion / raw RGB //hyperion / raw RGB
if (!packetSize && udpRgbConnected) { if (!packetSize && udpRgbConnected) {
rgbUdp.flush();
packetSize = rgbUdp.parsePacket(); packetSize = rgbUdp.parsePacket();
if (packetSize) { if (packetSize) {
if (!receiveDirect) {rgbUdp.flush(); notifierUdp.flush(); notifier2Udp.flush(); return;} if (!receiveDirect) {rgbUdp.flush(); notifierUdp.flush(); notifier2Udp.flush(); return;}