Changeset 16726 in vbox for trunk/src/VBox/Devices

Timestamp:

Feb 13, 2009 5:16:48 AM (16 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

42762

Message:

NAT: avoiding "false alarms"

File:

• trunk/src/VBox/Devices/Network/DrvNAT.cpp (modified) (1 diff)

trunk/src/VBox/Devices/Network/DrvNAT.cpp

@@ -400 +400 @@
                 char ch[1];
                 size_t cbRead;
-                RTFileRead(pThis->PipeRead, &ch, 1, &cbRead);
+                /* 
+                 * drvNATSend decoupled so we don't know how many times 
+                 * device's thread sends before we've entered multiplex,
+                 * so to avoid false alarm drain pipe here to the very end
+                 *
+                 * @todo: Probably we should counter drvNATSend to count how 
+                 * deep pipe has been filed before drain
+                 */
+                while(RTFileRead(pThis->PipeRead, &ch, 1, &cbRead) > 0);
             }
             /* process _all_ outstanding requests but don't wait */