Browse Source

[1540] Changes after review:

- test prints out information that failed socket binding is ok.
- iface_mgr buffers are no longer static.
Tomek Mrugalski 13 years ago
parent
commit
be560f4516
2 changed files with 5 additions and 2 deletions
  1. 3 0
      src/bin/dhcp6/tests/dhcp6_test.py
  2. 2 2
      src/lib/dhcp/iface_mgr.cc

+ 3 - 0
src/bin/dhcp6/tests/dhcp6_test.py

@@ -36,6 +36,9 @@ class TestDhcpv6Daemon(unittest.TestCase):
         # to the main program ASAP in each test... this prevents
         # hangs reading from the child process (as the pipe is only
         # open in the child), and also insures nice pretty output
+        print("Please ignore any socket errors. Purpose of this test is to")
+        print("verify that DHCPv6 process could be started, not that socket")
+        print("could be bound. Binding fails when run as non-root user.")
 
     def tearDown(self):
         # clean up our stdout munging

+ 2 - 2
src/lib/dhcp/iface_mgr.cc

@@ -707,7 +707,7 @@ IfaceMgr::receive4() {
 
     // Now we have a socket, let's get some data from it!
     struct sockaddr_in from_addr;
-    static uint8_t buf[RCVBUFSIZE];
+    uint8_t buf[RCVBUFSIZE];
 
     memset(&control_buf_[0], 0, control_buf_len_);
     memset(&from_addr, 0, sizeof(from_addr));
@@ -772,7 +772,7 @@ IfaceMgr::receive4() {
 }
 
 Pkt6Ptr IfaceMgr::receive6() {
-    static uint8_t buf[RCVBUFSIZE];
+    uint8_t buf[RCVBUFSIZE];
 
     memset(&control_buf_[0], 0, control_buf_len_);
     struct sockaddr_in6 from;