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

Timestamp:

Nov 20, 2018 11:59:31 AM (6 years ago)

Author:

vboxsync

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

126792

Message:

Rolled back r126789.

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 (deleted)
• VBoxNetDhcpdHardened.cpp (deleted)

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

@@ -35 +35 @@
     ClientId()
       : m_mac(), m_id() {}
-    ClientId(const RTMAC &macParam, const OptClientId &idParam)
-      : m_mac(macParam), m_id(idParam) {}
+    ClientId(const RTMAC &mac, const OptClientId &id)
+      : m_mac(mac), m_id(id) {}
 
     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 (optmap_t::const_iterator it = m_GlobalOptions.begin(); it != m_GlobalOptions.end(); ++it) {
-        std::shared_ptr<DhcpOption> opt(it->second);
+    for (auto it: m_GlobalOptions) {
+        std::shared_ptr<DhcpOption> opt(it.second);
 
         octets_t data;
@@ -596 +596 @@
 
         bool space = false;
-        for (octets_t::const_iterator itData = data.begin(); itData != data.end(); ++itData) {
-            uint8_t c = *itData;
+        for (auto c: data) {
             if (space)
                 std::cout << " ";
@@ -855 +854 @@
     optmap << new OptSubnetMask(m_IPv4Netmask);
 
-    const OptParameterRequest::value_t& reqValue = reqOpts.value();
-    for (octets_t::const_iterator itOptReq = reqValue.begin(); itOptReq != reqValue.end(); ++itOptReq)
-    {
-        uint8_t optreq = *itOptReq;
+    for (auto optreq: reqOpts.value())
+    {
         std::cout << ">>> requested option " << (int)optreq << std::endl;
 
@@ -893 +890 @@
     if (vmopts != NULL)
     {
-        for (optmap_t::const_iterator it = vmopts->begin(); it != vmopts->end(); ++it) {
-            std::shared_ptr<DhcpOption> opt(it->second);
+        for (auto it: *vmopts) {
+            std::shared_ptr<DhcpOption> opt(it.second);
             if (optmap.count(opt->optcode()) == 0 && opt->optcode() > 127)
             {
@@ -903 +900 @@
     }
 
-    for (optmap_t::const_iterator it = m_GlobalOptions.begin(); it != m_GlobalOptions.end(); ++it) {
-        std::shared_ptr<DhcpOption> opt(it->second);
+    for (auto it: m_GlobalOptions) {
+        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 addrParam)
-      : m_addr(addrParam), m_state(FREE),
+    explicit Binding(RTNETADDRIPV4 addr)
+      : m_addr(addr), m_state(FREE),
         m_issued(), m_secLease() {}
 
-    Binding(RTNETADDRIPV4 addrParam, const ClientId &idParam)
-      : m_addr(addrParam), m_state(FREE), m_id(idParam),
+    Binding(RTNETADDRIPV4 addr, const ClientId &id)
+      : m_addr(addr), m_state(FREE), m_id(id),
         m_issued(), m_secLease() {}
 
@@ -70 +70 @@
     TimeStamp issued() const { return m_issued; }
 
-    Binding &setState(State stateParam)
+    Binding &setState(State state)
     {
-        m_state = stateParam;
+        m_state = state;
         return *this;
     }
@@ -85 +85 @@
     }
 
-    Binding &giveTo(const ClientId &idParam)
+    Binding &giveTo(const ClientId &id)
     {
-        m_id = idParam;
+        m_id = id;
         m_state = FREE;
         return *this;

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

@@ -24 +24 @@
 
 #include <map>
+#include <memory>
 #include <vector>
-
-#if __cplusplus >= 199711
-#include <memory>
-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;
@@ -43 +32 @@
 
 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 messageTypeParam, RTNETADDRIPV4 serverAddr)
+                                     uint8_t messageType, RTNETADDRIPV4 serverAddr)
   : DhcpMessage(),
     m_optServerId(serverAddr)
@@ -329 +329 @@
     m_dst.u = 0xffffffff;       /* broadcast */
 
-    m_optMessageType = OptMessageType(messageTypeParam);
+    m_optMessageType = OptMessageType(messageType);
 
     /* 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)
+    if (option == NULL)
         return optmap;
 

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

@@ -342 +342 @@
     static OptValue *parse(const char *pcszValue)
     {
-        typename OptValueBase<T>::value_t v;
+        value_t v;
         int rc = DhcpOption::parse1(v, pcszValue);
         if (RT_FAILURE(rc))
@@ -544 +544 @@
     static OptList *parse(const char *pcszValue)
     {
-        typename OptListBase<T>::value_t v;
+        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
-  DLLS += VBoxNetDhcpd
+  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
  else
   PROGRAMS += VBoxNetDhcpd
  endif
 
- 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_TEMPLATE := $(if-expr defined(VBOX_WITH_HARDENING),VBOXMAINDLL,VBOXMAINCLIENTEXE)
+ VBoxNetDhcpd_NAME     := VBoxNetDhcpd
  # VBoxNetDhcpd_DEFS      = IPv6
- # VBoxNetDhcpd_DEFS.linux = WITH_VALGRIND
- #VBoxNetDhcpd_DEFS.win  = VBOX_COM_OUTOFPROC_MODULE _WIN32_WINNT=0x501 # Windows XP
+ # VBoxNetLwipNAT_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_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))
+ 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_LIBS = $(LIB_RUNTIME)
+ VBoxNetLwipNAT_LIBS = $(LIB_RUNTIME)
 
- VBoxNetDhcpd_LIBS.solaris += socket nsl
- VBoxNetDhcpd_LDFLAGS.win = /SUBSYSTEM:windows
+ VBoxNetLwipNAT_LIBS.solaris += socket nsl
+ VBoxNetLwipNAT_LDFLAGS.win = /SUBSYSTEM:windows
 
  ifeq ($(KBUILD_TARGET),win)

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

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