Changeset 75614 in vbox for trunk/src/VBox/NetworkServices/Dhcpd

Timestamp:

Nov 20, 2018 11:41:35 AM (6 years ago)

Author:

vboxsync

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

126789

Message:

Main/DHCPD: bugref:9288 Use new implementation of DHCP server (VCC 10 and GCC 4.4.4 support).

Location:

trunk/src/VBox/NetworkServices/Dhcpd

Files:

• ClientId.h (modified) (1 diff)
• Config.cpp (modified) (5 diffs)
• Db.h (modified) (3 diffs)
• Defs.h (modified) (2 diffs)
• DhcpMessage.cpp (modified) (2 diffs)
• DhcpOptions.cpp (modified) (1 diff)
• DhcpOptions.h (modified) (2 diffs)
• Makefile.kmk (modified) (1 diff)
• VBoxNetDhcpd.cpp (modified) (2 diffs)
• VBoxNetDhcpd.rc (added)
• VBoxNetDhcpdHardened.cpp (added)

trunk/src/VBox/NetworkServices/Dhcpd/ClientId.h

@@ -35 +35 @@
     ClientId()
       : m_mac(), m_id() {}
-    ClientId(const RTMAC &mac, const OptClientId &id)
-      : m_mac(mac), m_id(id) {}
+    ClientId(const RTMAC &macParam, const OptClientId &idParam)
+      : m_mac(macParam), m_id(idParam) {}
 
     const RTMAC &mac() const { return m_mac; }

trunk/src/VBox/NetworkServices/Dhcpd/Config.cpp

@@ -589 +589 @@
     /** @todo r=bird: Visual C++ 2010 does not grok this use of 'auto'. */
     // XXX: debug
-    for (auto it: m_GlobalOptions) {
-        std::shared_ptr<DhcpOption> opt(it.second);
+    for (optmap_t::const_iterator it = m_GlobalOptions.begin(); it != m_GlobalOptions.end(); ++it) {
+        std::shared_ptr<DhcpOption> opt(it->second);
 
         octets_t data;
@@ -596 +596 @@
 
         bool space = false;
-        for (auto c: data) {
+        for (octets_t::const_iterator itData = data.begin(); itData != data.end(); ++itData) {
+            uint8_t c = *itData;
             if (space)
                 std::cout << " ";
@@ -854 +855 @@
     optmap << new OptSubnetMask(m_IPv4Netmask);
 
-    for (auto optreq: reqOpts.value())
-    {
+    const OptParameterRequest::value_t& reqValue = reqOpts.value();
+    for (octets_t::const_iterator itOptReq = reqValue.begin(); itOptReq != reqValue.end(); ++itOptReq)
+    {
+        uint8_t optreq = *itOptReq;
         std::cout << ">>> requested option " << (int)optreq << std::endl;
 
@@ -890 +893 @@
     if (vmopts != NULL)
     {
-        for (auto it: *vmopts) {
-            std::shared_ptr<DhcpOption> opt(it.second);
+        for (optmap_t::const_iterator it = vmopts->begin(); it != vmopts->end(); ++it) {
+            std::shared_ptr<DhcpOption> opt(it->second);
             if (optmap.count(opt->optcode()) == 0 && opt->optcode() > 127)
             {
@@ -900 +903 @@
     }
 
-    for (auto it: m_GlobalOptions) {
-        std::shared_ptr<DhcpOption> opt(it.second);
+    for (optmap_t::const_iterator it = m_GlobalOptions.begin(); it != m_GlobalOptions.end(); ++it) {
+        std::shared_ptr<DhcpOption> opt(it->second);
         if (optmap.count(opt->optcode()) == 0 && opt->optcode() > 127)
         {

trunk/src/VBox/NetworkServices/Dhcpd/Db.h

@@ -51 +51 @@
     Binding(const Binding &);
 
-    explicit Binding(RTNETADDRIPV4 addr)
-      : m_addr(addr), m_state(FREE),
+    explicit Binding(RTNETADDRIPV4 addrParam)
+      : m_addr(addrParam), m_state(FREE),
         m_issued(), m_secLease() {}
 
-    Binding(RTNETADDRIPV4 addr, const ClientId &id)
-      : m_addr(addr), m_state(FREE), m_id(id),
+    Binding(RTNETADDRIPV4 addrParam, const ClientId &idParam)
+      : m_addr(addrParam), m_state(FREE), m_id(idParam),
         m_issued(), m_secLease() {}
 
@@ -70 +70 @@
     TimeStamp issued() const { return m_issued; }
 
-    Binding &setState(State state)
+    Binding &setState(State stateParam)
     {
-        m_state = state;
+        m_state = stateParam;
         return *this;
     }
@@ -85 +85 @@
     }
 
-    Binding &giveTo(const ClientId &id)
+    Binding &giveTo(const ClientId &idParam)
     {
-        m_id = id;
+        m_id = idParam;
         m_state = FREE;
         return *this;

trunk/src/VBox/NetworkServices/Dhcpd/Defs.h

@@ -24 +24 @@
 
 #include <map>
+#include <vector>
+
+#if __cplusplus >= 199711
 #include <memory>
-#include <vector>
+using std::shared_ptr;
+#else
+#include <tr1/memory>
+using std::tr1::shared_ptr;
+#endif
+
+#ifdef _MSC_VER
+# define __func__ __FUNCTION__
+#endif
 
 typedef std::vector<uint8_t> octets_t;
@@ -32 +43 @@
 
 class DhcpOption;
-typedef std::map<uint8_t, std::shared_ptr<DhcpOption>> optmap_t;
+typedef std::map<uint8_t, std::shared_ptr<DhcpOption> > optmap_t;
 
 inline bool operator==(const RTMAC &l, const RTMAC &r)

trunk/src/VBox/NetworkServices/Dhcpd/DhcpMessage.cpp

@@ -323 +323 @@
 
 DhcpServerMessage::DhcpServerMessage(const DhcpClientMessage &req,
-                                     uint8_t messageType, RTNETADDRIPV4 serverAddr)
+                                     uint8_t messageTypeParam, RTNETADDRIPV4 serverAddr)
   : DhcpMessage(),
     m_optServerId(serverAddr)
@@ -329 +329 @@
     m_dst.u = 0xffffffff;       /* broadcast */
 
-    m_optMessageType = OptMessageType(messageType);
+    m_optMessageType = OptMessageType(messageTypeParam);
 
     /* copy values from the request (cf. RFC2131 Table 3) */

trunk/src/VBox/NetworkServices/Dhcpd/DhcpOptions.cpp

@@ -36 +36 @@
 optmap_t &operator<<(optmap_t &optmap, const std::shared_ptr<DhcpOption> &option)
 {
-    if (option == NULL)
+    if (!option)
         return optmap;
 

trunk/src/VBox/NetworkServices/Dhcpd/DhcpOptions.h

@@ -342 +342 @@
     static OptValue *parse(const char *pcszValue)
     {
-        value_t v;
+        typename OptValueBase<T>::value_t v;
         int rc = DhcpOption::parse1(v, pcszValue);
         if (RT_FAILURE(rc))
@@ -544 +544 @@
     static OptList *parse(const char *pcszValue)
     {
-        value_t v;
+        typename OptListBase<T>::value_t v;
         int rc = DhcpOption::parseList(v, pcszValue);
         if (RT_FAILURE(rc) || v.empty())

trunk/src/VBox/NetworkServices/Dhcpd/Makefile.kmk

@@ -29 +29 @@
  ifdef VBOX_WITH_HARDENING
   PROGRAMS += VBoxNetDhcpdHardened
-  VBoxNetDhcpdHardened_TEMPLATE = VBOXR3HARDENEDEXE
-  VBoxNetDhcpdHardened_NAME     = VBoxNetDhcpd
-  VBoxNetDhcpdHardened_DEFS     = SERVICE_NAME=\"VBoxNetDhcpd\"
-  VBoxNetDhcpdHardened_SOURCES  = VBoxNetDhcpdHardened.cpp
-  VBoxNetDhcpdHardened_SOURCES.win = $(VBoxNetLwipNAT_0_OUTDIR)/VBoxNetDhcpd-icon.rc
-  VBoxNetDhcpdHardened_LDFLAGS.win = /SUBSYSTEM:windows
+  DLLS += VBoxNetDhcpd
  else
   PROGRAMS += VBoxNetDhcpd
  endif
 
- VBoxNetDhcpd_TEMPLATE := $(if-expr defined(VBOX_WITH_HARDENING),VBOXMAINDLL,VBOXMAINCLIENTEXE)
- VBoxNetDhcpd_NAME     := VBoxNetDhcpd
+ VBoxNetDhcpdHardened_TEMPLATE = VBOXR3HARDENEDEXE
+ VBoxNetDhcpdHardened_NAME     = VBoxNetDHCP
+ VBoxNetDhcpdHardened_DEFS     = SERVICE_NAME=\"VBoxNetDhcpd\"
+ VBoxNetDhcpdHardened_SOURCES  = VBoxNetDhcpdHardened.cpp
+ VBoxNetDhcpdHardened_SOURCES.win = $(VBoxNetDhcpd_0_OUTDIR)/VBoxNetDhcpd-icon.rc
+ VBoxNetDhcpdHardened_LDFLAGS.win = /SUBSYSTEM:windows
+
+ VBoxNetDhcpd_TEMPLATE := $(if-expr defined(VBOX_WITH_HARDENING),VBoxR3Dll,VBOXR3EXE)
+ VBoxNetDhcpd_NAME      = VBoxNetDHCP
  # VBoxNetDhcpd_DEFS      = IPv6
- # VBoxNetLwipNAT_DEFS.linux = WITH_VALGRIND
- VBoxNetDhcpd_DEFS.win  = VBOX_COM_OUTOFPROC_MODULE _WIN32_WINNT=0x501 # Windows XP
+ # VBoxNetDhcpd_DEFS.linux = WITH_VALGRIND
+ #VBoxNetDhcpd_DEFS.win  = VBOX_COM_OUTOFPROC_MODULE _WIN32_WINNT=0x501 # Windows XP
 
  # (current dir is for for lwipopts.h)
  VBoxNetDhcpd_INCS += . $(addprefix ../../Devices/Network/lwip-new/,$(LWIP_INCS))
 
- VBoxNetDhcpd_SOURCES  =
- VBoxNetDhcpd_SOURCES += ClientId.cpp
- VBoxNetDhcpd_SOURCES += Config.cpp
- VBoxNetDhcpd_SOURCES += DHCPD.cpp
- VBoxNetDhcpd_SOURCES += Db.cpp
- VBoxNetDhcpd_SOURCES += DhcpMessage.cpp
- VBoxNetDhcpd_SOURCES += DhcpOptions.cpp
- VBoxNetDhcpd_SOURCES += IPv4Pool.cpp
- VBoxNetDhcpd_SOURCES += TimeStamp.cpp
- VBoxNetDhcpd_SOURCES += VBoxNetDhcpd.cpp
- VBoxNetDhcpd_SOURCES += $(addprefix ../../Devices/Network/lwip-new/,$(LWIP_SOURCES))
+ VBoxNetDhcpd_DEFS    = KBUILD_TYPE=\"$(KBUILD_TYPE)\"
+ ifneq ($(KBUILD_TARGET),win)
+  VBoxNetDhcpd_DEFS     += VBOX_WITH_XPCOM
+  VBoxNetDhcpd_INCS     += $(VBOX_XPCOM_INCS)
+  # We want -std=c++11 for 4.7 and newer compilers, and -std=c++0x for older ones.
+  VBoxNetDhcpd_CXXFLAGS += -std=$(if $(VBOX_GCC_VERSION_CXX),$(if $(VBOX_GCC_VERSION_CXX) < 40700,c++0x,c++11),c++0x)
+ endif
+ VBoxNetDhcpd_SOURCES = ../../Main/glue/VBoxLogRelCreate.cpp \
+                        ../../Main/glue/GetVBoxUserHomeDirectory.cpp \
+                        ClientId.cpp \
+                        Config.cpp \
+                        DHCPD.cpp \
+                        Db.cpp \
+                        DhcpMessage.cpp \
+                        DhcpOptions.cpp \
+                        IPv4Pool.cpp \
+                        TimeStamp.cpp \
+                        VBoxNetDhcpd.cpp \
+                        $(addprefix ../../Devices/Network/lwip-new/,$(LWIP_SOURCES))
 
- VBoxNetLwipNAT_LIBS = $(LIB_RUNTIME)
+ VBoxNetDhcpd_LIBS = $(LIB_RUNTIME)
 
- VBoxNetLwipNAT_LIBS.solaris += socket nsl
- VBoxNetLwipNAT_LDFLAGS.win = /SUBSYSTEM:windows
+ VBoxNetDhcpd_LIBS.solaris += socket nsl
+ VBoxNetDhcpd_LDFLAGS.win = /SUBSYSTEM:windows
 
  ifeq ($(KBUILD_TARGET),win)

trunk/src/VBox/NetworkServices/Dhcpd/VBoxNetDhcpd.cpp

@@ -448 +448 @@
                                                      i, cSegs,
                                                      &cbSegFrame);
-                ifInput(pvSegFrame, cbFrame);
+                ifInput(pvSegFrame, (uint32_t)cbFrame);
             }
         }
@@ -738 +738 @@
 
     unique_ptr_pbuf q ( pbuf_alloc(PBUF_RAW, (u16_t)data.size(), PBUF_RAM) );
-    if (q == NULL)
-        return;
-
-    error = pbuf_take(q.get(), &data.front(), data.size());
+    if (!q)
+        return;
+
+    error = pbuf_take(q.get(), &data.front(), (u16_t)data.size());
     if (error != ERR_OK)
         return;