Forráskód Böngészése

IPv4: More hacking on bringing up TCP support.

This was a bit more complicated than I expected, but it's moving forward.
Andreas Kling 6 éve
szülő
commit
032d9d7065
7 módosított fájl, 193 hozzáadás és 33 törlés
  1. 116 18
      Kernel/IPv4Socket.cpp
  2. 25 0
      Kernel/IPv4Socket.h
  3. 1 1
      Kernel/NetworkOrdered.h
  4. 14 1
      Kernel/NetworkTask.cpp
  5. 11 6
      Kernel/Process.cpp
  6. 21 4
      Kernel/TCP.h
  7. 5 3
      Userland/tc.cpp

+ 116 - 18
Kernel/IPv4Socket.cpp

@@ -4,6 +4,7 @@
 #include <Kernel/NetworkAdapter.h>
 #include <Kernel/NetworkAdapter.h>
 #include <Kernel/IPv4.h>
 #include <Kernel/IPv4.h>
 #include <Kernel/ICMP.h>
 #include <Kernel/ICMP.h>
+#include <Kernel/TCP.h>
 #include <Kernel/UDP.h>
 #include <Kernel/UDP.h>
 #include <Kernel/ARP.h>
 #include <Kernel/ARP.h>
 #include <LibC/errno_numbers.h>
 #include <LibC/errno_numbers.h>
@@ -97,6 +98,16 @@ KResult IPv4Socket::connect(const sockaddr* address, socklen_t address_size)
     m_destination_address = IPv4Address((const byte*)&ia.sin_addr.s_addr);
     m_destination_address = IPv4Address((const byte*)&ia.sin_addr.s_addr);
     m_destination_port = ntohs(ia.sin_port);
     m_destination_port = ntohs(ia.sin_port);
 
 
+    if (type() == SOCK_STREAM) {
+        // FIXME: Figure out the adapter somehow differently.
+        auto* adapter = NetworkAdapter::from_ipv4_address(IPv4Address(192, 168, 5, 2));
+        if (!adapter)
+            ASSERT_NOT_REACHED();
+        send_tcp_packet(*adapter, TCPFlags::SYN);
+        m_tcp_state = TCPState::Connecting1;
+        return KSuccess;
+    }
+
     return KSuccess;
     return KSuccess;
 }
 }
 
 
@@ -164,10 +175,77 @@ void IPv4Socket::allocate_source_port_if_needed()
     }
     }
 }
 }
 
 
+struct [[gnu::packed]] TCPPseudoHeader {
+    IPv4Address source;
+    IPv4Address destination;
+    byte zero;
+    byte protocol;
+    NetworkOrdered<word> payload_size;
+};
+
+NetworkOrdered<word> IPv4Socket::compute_tcp_checksum(const IPv4Address& source, const IPv4Address& destination, const TCPPacket& packet, word payload_size)
+{
+    TCPPseudoHeader pseudo_header { source, destination, 0, (byte)IPv4Protocol::TCP, sizeof(TCPPacket) + payload_size };
+
+    dword checksum = 0;
+    auto* w = (const NetworkOrdered<word>*)&pseudo_header;
+    for (size_t i = 0; i < sizeof(pseudo_header) / sizeof(word); ++i) {
+        checksum += w[i];
+        if (checksum > 0xffff)
+            checksum = (checksum >> 16) + (checksum & 0xffff);
+    }
+    w = (const NetworkOrdered<word>*)&packet;
+    for (size_t i = 0; i < sizeof(packet) / sizeof(word); ++i) {
+        checksum += w[i];
+        if (checksum > 0xffff)
+            checksum = (checksum >> 16) + (checksum & 0xffff);
+    }
+    ASSERT(packet.data_offset() * 4 == sizeof(TCPPacket));
+    w = (const NetworkOrdered<word>*)packet.payload();
+    for (size_t i = 0; i < payload_size / sizeof(word); ++i) {
+        checksum += w[i];
+        if (checksum > 0xffff)
+            checksum = (checksum >> 16) + (checksum & 0xffff);
+    }
+    if (payload_size & 1)
+        ASSERT_NOT_REACHED();
+    return ~(checksum & 0xffff);
+}
+
+void IPv4Socket::send_tcp_packet(NetworkAdapter& adapter, word flags, const void* payload, size_t payload_size)
+{
+    auto buffer = ByteBuffer::create_zeroed(sizeof(TCPPacket) + payload_size);
+    auto& tcp_packet = *(TCPPacket*)(buffer.pointer());
+    tcp_packet.set_source_port(m_source_port);
+    tcp_packet.set_destination_port(m_destination_port);
+    tcp_packet.set_window_size(1024);
+    tcp_packet.set_sequence_number(m_tcp_sequence_number);
+    tcp_packet.set_data_offset(5);
+    tcp_packet.set_flags(flags);
+
+    if (flags & TCPFlags::ACK)
+        tcp_packet.set_ack_number(m_tcp_ack_number);
+
+    if (flags == TCPFlags::SYN) {
+        ++m_tcp_sequence_number;
+    } else {
+        m_tcp_sequence_number += payload_size;
+    }
+
+    memcpy(tcp_packet.payload(), payload, payload_size);
+    tcp_packet.set_checksum(compute_tcp_checksum(adapter.ipv4_address(), m_destination_address, tcp_packet, payload_size));
+    kprintf("sending tcp packet from %s:%u to %s:%u!\n",
+        adapter.ipv4_address().to_string().characters(),
+        source_port(),
+        m_destination_address.to_string().characters(),
+        m_destination_port);
+    adapter.send_ipv4(MACAddress(), m_destination_address, IPv4Protocol::TCP, move(buffer));
+}
+
 ssize_t IPv4Socket::sendto(const void* data, size_t data_length, int flags, const sockaddr* addr, socklen_t addr_length)
 ssize_t IPv4Socket::sendto(const void* data, size_t data_length, int flags, const sockaddr* addr, socklen_t addr_length)
 {
 {
     (void)flags;
     (void)flags;
-    if (addr_length != sizeof(sockaddr_in))
+    if (addr && addr_length != sizeof(sockaddr_in))
         return -EINVAL;
         return -EINVAL;
     // FIXME: Find the adapter some better way!
     // FIXME: Find the adapter some better way!
     auto* adapter = NetworkAdapter::from_ipv4_address(IPv4Address(192, 168, 5, 2));
     auto* adapter = NetworkAdapter::from_ipv4_address(IPv4Address(192, 168, 5, 2));
@@ -176,12 +254,12 @@ ssize_t IPv4Socket::sendto(const void* data, size_t data_length, int flags, cons
         ASSERT_NOT_REACHED();
         ASSERT_NOT_REACHED();
     }
     }
 
 
-    if (addr->sa_family != AF_INET) {
-        kprintf("sendto: Bad address family: %u is not AF_INET!\n", addr->sa_family);
-        return -EAFNOSUPPORT;
-    }
-
     if (addr) {
     if (addr) {
+        if (addr->sa_family != AF_INET) {
+            kprintf("sendto: Bad address family: %u is not AF_INET!\n", addr->sa_family);
+            return -EAFNOSUPPORT;
+        }
+
         auto& ia = *(const sockaddr_in*)addr;
         auto& ia = *(const sockaddr_in*)addr;
         m_destination_address = IPv4Address((const byte*)&ia.sin_addr.s_addr);
         m_destination_address = IPv4Address((const byte*)&ia.sin_addr.s_addr);
         m_destination_port = ntohs(ia.sin_port);
         m_destination_port = ntohs(ia.sin_port);
@@ -191,12 +269,8 @@ ssize_t IPv4Socket::sendto(const void* data, size_t data_length, int flags, cons
 
 
     kprintf("sendto: destination=%s:%u\n", m_destination_address.to_string().characters(), m_destination_port);
     kprintf("sendto: destination=%s:%u\n", m_destination_address.to_string().characters(), m_destination_port);
 
 
-    // FIXME: If we can't find the right MAC address, block until it's available?
-    //        I feel like this should happen in a layer below this code.
-    MACAddress mac_address;
-
     if (type() == SOCK_RAW) {
     if (type() == SOCK_RAW) {
-        adapter->send_ipv4(mac_address, m_destination_address, (IPv4Protocol)protocol(), ByteBuffer::copy((const byte*)data, data_length));
+        adapter->send_ipv4(MACAddress(), m_destination_address, (IPv4Protocol)protocol(), ByteBuffer::copy((const byte*)data, data_length));
         return data_length;
         return data_length;
     }
     }
 
 
@@ -212,7 +286,12 @@ ssize_t IPv4Socket::sendto(const void* data, size_t data_length, int flags, cons
             source_port(),
             source_port(),
             m_destination_address.to_string().characters(),
             m_destination_address.to_string().characters(),
             m_destination_port);
             m_destination_port);
-        adapter->send_ipv4(mac_address, m_destination_address, IPv4Protocol::UDP, move(buffer));
+        adapter->send_ipv4(MACAddress(), m_destination_address, IPv4Protocol::UDP, move(buffer));
+        return data_length;
+    }
+
+    if (type() == SOCK_STREAM) {
+        send_tcp_packet(*adapter, 0, data, data_length);
         return data_length;
         return data_length;
     }
     }
 
 
@@ -222,7 +301,7 @@ ssize_t IPv4Socket::sendto(const void* data, size_t data_length, int flags, cons
 ssize_t IPv4Socket::recvfrom(void* buffer, size_t buffer_length, int flags, sockaddr* addr, socklen_t* addr_length)
 ssize_t IPv4Socket::recvfrom(void* buffer, size_t buffer_length, int flags, sockaddr* addr, socklen_t* addr_length)
 {
 {
     (void)flags;
     (void)flags;
-    if (*addr_length < sizeof(sockaddr_in))
+    if (addr_length && *addr_length < sizeof(sockaddr_in))
         return -EINVAL;
         return -EINVAL;
 
 
 #ifdef IPV4_SOCKET_DEBUG
 #ifdef IPV4_SOCKET_DEBUG
@@ -256,10 +335,13 @@ ssize_t IPv4Socket::recvfrom(void* buffer, size_t buffer_length, int flags, sock
     ASSERT(!packet_buffer.is_null());
     ASSERT(!packet_buffer.is_null());
     auto& ipv4_packet = *(const IPv4Packet*)(packet_buffer.pointer());
     auto& ipv4_packet = *(const IPv4Packet*)(packet_buffer.pointer());
 
 
-    auto& ia = *(sockaddr_in*)addr;
-    memcpy(&ia.sin_addr, &m_destination_address, sizeof(IPv4Address));
-    ia.sin_family = AF_INET;
-    *addr_length = sizeof(sockaddr_in);
+    if (addr) {
+        auto& ia = *(sockaddr_in*)addr;
+        memcpy(&ia.sin_addr, &m_destination_address, sizeof(IPv4Address));
+        ia.sin_family = AF_INET;
+        ASSERT(addr_length);
+        *addr_length = sizeof(sockaddr_in);
+    }
 
 
     if (type() == SOCK_RAW) {
     if (type() == SOCK_RAW) {
         ASSERT(buffer_length >= ipv4_packet.payload_size());
         ASSERT(buffer_length >= ipv4_packet.payload_size());
@@ -271,11 +353,26 @@ ssize_t IPv4Socket::recvfrom(void* buffer, size_t buffer_length, int flags, sock
         auto& udp_packet = *static_cast<const UDPPacket*>(ipv4_packet.payload());
         auto& udp_packet = *static_cast<const UDPPacket*>(ipv4_packet.payload());
         ASSERT(udp_packet.length() >= sizeof(UDPPacket)); // FIXME: This should be rejected earlier.
         ASSERT(udp_packet.length() >= sizeof(UDPPacket)); // FIXME: This should be rejected earlier.
         ASSERT(buffer_length >= (udp_packet.length() - sizeof(UDPPacket)));
         ASSERT(buffer_length >= (udp_packet.length() - sizeof(UDPPacket)));
-        ia.sin_port = htons(udp_packet.destination_port());
+        if (addr) {
+            auto& ia = *(sockaddr_in*)addr;
+            ia.sin_port = htons(udp_packet.destination_port());
+        }
         memcpy(buffer, udp_packet.payload(), udp_packet.length() - sizeof(UDPPacket));
         memcpy(buffer, udp_packet.payload(), udp_packet.length() - sizeof(UDPPacket));
         return udp_packet.length() - sizeof(UDPPacket);
         return udp_packet.length() - sizeof(UDPPacket);
     }
     }
 
 
+    if (type() == SOCK_STREAM) {
+        auto& tcp_packet = *static_cast<const TCPPacket*>(ipv4_packet.payload());
+        size_t payload_size = packet_buffer.size() - sizeof(TCPPacket);
+        ASSERT(buffer_length >= payload_size);
+        if (addr) {
+            auto& ia = *(sockaddr_in*)addr;
+            ia.sin_port = htons(tcp_packet.destination_port());
+        }
+        memcpy(buffer, tcp_packet.payload(), payload_size);
+        return payload_size;
+    }
+
     ASSERT_NOT_REACHED();
     ASSERT_NOT_REACHED();
 }
 }
 
 
@@ -287,4 +384,5 @@ void IPv4Socket::did_receive(ByteBuffer&& packet)
 #ifdef IPV4_SOCKET_DEBUG
 #ifdef IPV4_SOCKET_DEBUG
     kprintf("IPv4Socket(%p): did_receive %d bytes, packets in queue: %d\n", this, packet.size(), m_receive_queue.size_slow());
     kprintf("IPv4Socket(%p): did_receive %d bytes, packets in queue: %d\n", this, packet.size(), m_receive_queue.size_slow());
 #endif
 #endif
+
 }
 }

+ 25 - 0
Kernel/IPv4Socket.h

@@ -7,6 +7,18 @@
 #include <AK/Lock.h>
 #include <AK/Lock.h>
 #include <AK/SinglyLinkedList.h>
 #include <AK/SinglyLinkedList.h>
 
 
+class NetworkAdapter;
+class TCPPacket;
+
+enum TCPState {
+    Disconnected,
+    Connecting1,
+    Connecting2,
+    Connected,
+    Disconnecting1,
+    Disconnecting2,
+};
+
 class IPv4Socket final : public Socket {
 class IPv4Socket final : public Socket {
 public:
 public:
     static Retained<IPv4Socket> create(int type, int protocol);
     static Retained<IPv4Socket> create(int type, int protocol);
@@ -35,11 +47,20 @@ public:
     word source_port() const { return m_source_port; }
     word source_port() const { return m_source_port; }
     word destination_port() const { return m_destination_port; }
     word destination_port() const { return m_destination_port; }
 
 
+    void send_tcp_packet(NetworkAdapter&, word flags, const void* payload = nullptr, size_t = 0);
+    void set_tcp_state(TCPState state) { m_tcp_state = state; }
+    TCPState tcp_state() const { return m_tcp_state; }
+    void set_tcp_ack_number(dword n) { m_tcp_ack_number = n; }
+    void set_tcp_sequence_number(dword n) { m_tcp_sequence_number = n; }
+    dword tcp_ack_number() const { return m_tcp_ack_number; }
+    dword tcp_sequence_number() const { return m_tcp_sequence_number; }
+
 private:
 private:
     IPv4Socket(int type, int protocol);
     IPv4Socket(int type, int protocol);
     virtual bool is_ipv4() const override { return true; }
     virtual bool is_ipv4() const override { return true; }
 
 
     void allocate_source_port_if_needed();
     void allocate_source_port_if_needed();
+    NetworkOrdered<word> compute_tcp_checksum(const IPv4Address& source, const IPv4Address& destination, const TCPPacket&, word payload_size);
 
 
     bool m_bound { false };
     bool m_bound { false };
     int m_attached_fds { 0 };
     int m_attached_fds { 0 };
@@ -53,6 +74,10 @@ private:
     word m_source_port { 0 };
     word m_source_port { 0 };
     word m_destination_port { 0 };
     word m_destination_port { 0 };
 
 
+    dword m_tcp_sequence_number { 0 };
+    dword m_tcp_ack_number { 0 };
+    TCPState m_tcp_state { Disconnected };
+
     Lock m_lock;
     Lock m_lock;
     bool m_can_read { false };
     bool m_can_read { false };
 };
 };

+ 1 - 1
Kernel/NetworkOrdered.h

@@ -44,5 +44,5 @@ public:
     operator T() const { return convert_between_host_and_network(m_network_value); }
     operator T() const { return convert_between_host_and_network(m_network_value); }
 
 
 private:
 private:
-    T m_network_value;
+    T m_network_value { 0 };
 };
 };

+ 14 - 1
Kernel/NetworkTask.cpp

@@ -263,7 +263,7 @@ void handle_tcp(const EthernetFrameHeader& eth, int frame_size)
 
 
     auto& tcp_packet = *static_cast<const TCPPacket*>(ipv4_packet.payload());
     auto& tcp_packet = *static_cast<const TCPPacket*>(ipv4_packet.payload());
 #ifdef TCP_DEBUG
 #ifdef TCP_DEBUG
-    kprintf("handle_tcp: source=%s:%u, destination=%s:%u seq=%u, ack=%u, flags=%w, window_size=%u\n",
+    kprintf("handle_tcp: source=%s:%u, destination=%s:%u seq_no=%u, ack_no=%u, flags=%w (%s %s), window_size=%u\n",
         ipv4_packet.source().to_string().characters(),
         ipv4_packet.source().to_string().characters(),
         tcp_packet.source_port(),
         tcp_packet.source_port(),
         ipv4_packet.destination().to_string().characters(),
         ipv4_packet.destination().to_string().characters(),
@@ -271,6 +271,8 @@ void handle_tcp(const EthernetFrameHeader& eth, int frame_size)
         tcp_packet.sequence_number(),
         tcp_packet.sequence_number(),
         tcp_packet.ack_number(),
         tcp_packet.ack_number(),
         tcp_packet.flags(),
         tcp_packet.flags(),
+        tcp_packet.has_syn() ? "SYN" : "",
+        tcp_packet.has_ack() ? "ACK" : "",
         tcp_packet.window_size()
         tcp_packet.window_size()
     );
     );
 #endif
 #endif
@@ -288,5 +290,16 @@ void handle_tcp(const EthernetFrameHeader& eth, int frame_size)
     LOCKER(socket->lock());
     LOCKER(socket->lock());
     ASSERT(socket->type() == SOCK_STREAM);
     ASSERT(socket->type() == SOCK_STREAM);
     ASSERT(socket->source_port() == tcp_packet.destination_port());
     ASSERT(socket->source_port() == tcp_packet.destination_port());
+
+    size_t payload_size = ipv4_packet.payload_size() - sizeof(TCPPacket);
+
+    if (tcp_packet.has_syn() && tcp_packet.has_ack()) {
+        socket->set_tcp_ack_number(socket->tcp_sequence_number() + payload_size + 1);
+        socket->send_tcp_packet(*adapter, TCPFlags::ACK);
+        kprintf("Connected!\n");
+        socket->set_tcp_state(Connected);
+        return;
+    }
+
     socket->did_receive(ByteBuffer::copy((const byte*)&ipv4_packet, sizeof(IPv4Packet) + ipv4_packet.payload_size()));
     socket->did_receive(ByteBuffer::copy((const byte*)&ipv4_packet, sizeof(IPv4Packet) + ipv4_packet.payload_size()));
 }
 }

+ 11 - 6
Kernel/Process.cpp

@@ -1073,6 +1073,7 @@ void Process::crash()
 {
 {
     ASSERT_INTERRUPTS_DISABLED();
     ASSERT_INTERRUPTS_DISABLED();
     ASSERT(state() != Dead);
     ASSERT(state() != Dead);
+
     m_termination_signal = SIGSEGV;
     m_termination_signal = SIGSEGV;
     dump_regions();
     dump_regions();
     ASSERT(is_ring3());
     ASSERT(is_ring3());
@@ -2533,7 +2534,7 @@ ssize_t Process::sys$sendto(const Syscall::SC_sendto_params* params)
 
 
     if (!validate_read(data, data_length))
     if (!validate_read(data, data_length))
         return -EFAULT;
         return -EFAULT;
-    if (!validate_read(addr, addr_length))
+    if (addr && !validate_read(addr, addr_length))
         return -EFAULT;
         return -EFAULT;
     auto* descriptor = file_descriptor(sockfd);
     auto* descriptor = file_descriptor(sockfd);
     if (!descriptor)
     if (!descriptor)
@@ -2559,17 +2560,21 @@ ssize_t Process::sys$recvfrom(const Syscall::SC_recvfrom_params* params)
 
 
     if (!validate_write(buffer, buffer_length))
     if (!validate_write(buffer, buffer_length))
         return -EFAULT;
         return -EFAULT;
-    if (!validate_read_typed(addr_length))
-        return -EFAULT;
-    if (!validate_read(addr, *addr_length))
-        return -EFAULT;
+    if (addr_length) {
+        if (!validate_read_typed(addr_length))
+            return -EFAULT;
+        if (!validate_read(addr, *addr_length))
+            return -EFAULT;
+    } else if (addr) {
+       return -EINVAL;
+    }
     auto* descriptor = file_descriptor(sockfd);
     auto* descriptor = file_descriptor(sockfd);
     if (!descriptor)
     if (!descriptor)
         return -EBADF;
         return -EBADF;
     if (!descriptor->is_socket())
     if (!descriptor->is_socket())
         return -ENOTSOCK;
         return -ENOTSOCK;
     auto& socket = *descriptor->socket();
     auto& socket = *descriptor->socket();
-    kprintf("recvfrom %p (%u), flags=%u, addr: %p (%u)\n", buffer, buffer_length, flags, addr, *addr_length);
+    kprintf("recvfrom %p (%u), flags=%u, addr: %p (%p)\n", buffer, buffer_length, flags, addr, addr_length);
     return socket.recvfrom(buffer, buffer_length, flags, addr, addr_length);
     return socket.recvfrom(buffer, buffer_length, flags, addr, addr_length);
 }
 }
 
 

+ 21 - 4
Kernel/TCP.h

@@ -2,6 +2,17 @@
 
 
 #include <Kernel/IPv4.h>
 #include <Kernel/IPv4.h>
 
 
+struct TCPFlags {
+enum : word {
+    FIN  = 0x01,
+    SYN  = 0x02,
+    RST  = 0x04,
+    PUSH = 0x08,
+    ACK  = 0x10,
+    URG  = 0x20
+};
+};
+
 class [[gnu::packed]] TCPPacket {
 class [[gnu::packed]] TCPPacket {
 public:
 public:
     TCPPacket() { }
     TCPPacket() { }
@@ -19,8 +30,14 @@ public:
     dword ack_number() const { return m_ack_number; }
     dword ack_number() const { return m_ack_number; }
     void set_ack_number(dword number) { m_ack_number = number; }
     void set_ack_number(dword number) { m_ack_number = number; }
 
 
-    word flags() const { return m_flags; }
-    void set_flags(word flags) { m_flags = flags; }
+    word flags() const { return m_flags_and_data_offset & 0x1ff; }
+    void set_flags(word flags) { m_flags_and_data_offset = (m_flags_and_data_offset & ~0x1ff) | (flags & 0x1ff); }
+
+    bool has_syn() const { return flags() & TCPFlags::SYN; }
+    bool has_ack() const { return flags() & TCPFlags::ACK; }
+
+    byte data_offset() const { return (m_flags_and_data_offset & 0xf000) >> 12; }
+    void set_data_offset(word data_offset) { m_flags_and_data_offset = (m_flags_and_data_offset & ~0xf000) | data_offset << 12; }
 
 
     word window_size() const { return m_window_size; }
     word window_size() const { return m_window_size; }
     void set_window_size(word window_size) { m_window_size = window_size; }
     void set_window_size(word window_size) { m_window_size = window_size; }
@@ -40,10 +57,10 @@ private:
     NetworkOrdered<dword> m_sequence_number;
     NetworkOrdered<dword> m_sequence_number;
     NetworkOrdered<dword> m_ack_number;
     NetworkOrdered<dword> m_ack_number;
 
 
-    NetworkOrdered<word> m_flags;
+    NetworkOrdered<word> m_flags_and_data_offset;
     NetworkOrdered<word> m_window_size;
     NetworkOrdered<word> m_window_size;
     NetworkOrdered<word> m_checksum;
     NetworkOrdered<word> m_checksum;
     NetworkOrdered<word> m_urgent;
     NetworkOrdered<word> m_urgent;
 };
 };
 
 
-static_assert(sizeof(UDPPacket) == 8);
+static_assert(sizeof(TCPPacket) == 20);

+ 5 - 3
Userland/tc.cpp

@@ -51,11 +51,13 @@ int main(int argc, char** argv)
     char buffer[BUFSIZ];
     char buffer[BUFSIZ];
     const char* msg = "Test message";
     const char* msg = "Test message";
 
 
-    send(fd, (const char*)msg, strlen(msg), 0);
+    rc = send(fd, (const char*)msg, strlen(msg), 0);
+    if (rc < 0) {
+        perror("send");
+        return 1;
+    }
     printf("Message sent.\n");
     printf("Message sent.\n");
 
 
-    struct sockaddr_in src_addr;
-    socklen_t src_addr_len = sizeof(src_addr);
     ssize_t nrecv = recv(fd, buffer, sizeof(buffer), 0);
     ssize_t nrecv = recv(fd, buffer, sizeof(buffer), 0);
     if (nrecv < 0) {
     if (nrecv < 0) {
         perror("recvfrom");
         perror("recvfrom");