Hi,

Having a UDP source port of 0 is not actually a mistake. By passing this
value you're asking the kernel to assign a free source port for your
socket. This is a desirable feature imho.

The attached patch should make sport=0 an option for udptunnel drivers.

Christophe
Index: DrvUDPTunnel.cpp
===================================================================
--- DrvUDPTunnel.cpp	(revision 39753)
+++ DrvUDPTunnel.cpp	(working copy)
@@ -508,8 +508,6 @@
     if (RT_FAILURE(rc))
         rc = PDMDRV_SET_ERROR(pDrvIns, rc,
                               N_("DrvUDPTunnel: Configuration error: Converting \"sport\" to integer failed"));
-    if (!pThis->uSrcPort)
-        pThis->uSrcPort = 4444;
 
     rc = CFGMR3QueryStringDef(pCfg, "dport", szVal, sizeof(szVal), "4445");
     if (RT_FAILURE(rc))
_______________________________________________
vbox-dev mailing list
vbox-dev@virtualbox.org
https://www.virtualbox.org/mailman/listinfo/vbox-dev

Reply via email to