Fix send UDP socket

This commit fixes UDP packet sending for both connected and unconnected sockets.

b\383980995

Change-Id: Iddff989ccce8261395bb1ea36a23716a742c6ea2

net/socket/udp_socket_starboard.cc

@@ -621,34 +621,37 @@ int UDPSocketStarboard::InternalReadMultiplePackets(
 int UDPSocketStarboard::InternalSendTo(IOBuffer* buf,
                                        int buf_len,
                                        const IPEndPoint* address) {
-  SbSocketAddress sb_address;
-  if (!address && !g_socket_extension) {
-    // Platforms without the socket extension require a destination address.
-    address = remote_address_.get();
-    if (!address) {
-      int result = ERR_FAILED;
-      LogWrite(result, NULL, NULL);
-      return result;
+  // If remote_address_ is set, it indicates that the socket is connected.
+  // This is because remote_address_ is assigned in InternalConnect.
+  // In this case, we treat the socket as connected and ignore the 'address' parameter,
+  // sending the data to the pre-established remote address.
+  // Otherwise, if remote_address_ is not set, we use the 'address' parameter
+  // to specify the destination for the data.
+  if (remote_address_) {
+    int result = SbSocketSendTo(socket_, buf->data(), buf_len, nullptr);
+    if (result < 0) {
+      result = MapLastSocketError(socket_);
     }
+    LogWrite(result, buf->data(), nullptr);
+    return result;
   }
+
+  SbSocketAddress sb_address;
+
   if (address && !address->ToSbSocketAddress(&sb_address)) {
     int result = ERR_FAILED;
-    LogWrite(result, NULL, NULL);
+    LogWrite(result, nullptr, nullptr);
     return result;
   }
 
   int result = SbSocketSendTo(socket_, buf->data(), buf_len,
                               address ? &sb_address : nullptr);
-
-  if (result < 0)
+  if (result < 0) {
     result = MapLastSocketError(socket_);
+  }
 
   if (result != ERR_IO_PENDING) {
-    if (!address) {
-      LogWrite(result, buf->data(), nullptr);
-    } else {
-      LogWrite(result, buf->data(), address);
-    }
+    LogWrite(result, buf->data(), address);
   }
 
   return result;

starboard/shared/posix/socket_send_to.cc

@@ -57,9 +57,11 @@ int SbSocketSendTo(SbSocket socket,
     socket->error = sbposix::TranslateSocketErrno(errno);
     return -1;
   } else if (socket->protocol == kSbSocketProtocolUdp) {
+    ssize_t bytes_written = 0;
     sbposix::SockAddr sock_addr;
-    const sockaddr* sockaddr = NULL;
+    const sockaddr* sockaddr = nullptr;
     socklen_t sockaddr_length = 0;
+
     if (destination) {
       if (!sock_addr.FromSbSocketAddress(destination)) {
         SB_LOG(FATAL) << "Invalid destination passed to UDP send.";
@@ -70,8 +72,9 @@ int SbSocketSendTo(SbSocket socket,
       sockaddr_length = sock_addr.length;
     }
 
-    ssize_t bytes_written = sendto(socket->socket_fd, data, data_size,
-                                   kSendFlags, sockaddr, sockaddr_length);
+    bytes_written = sendto(socket->socket_fd, data, data_size, kSendFlags,
+                           sockaddr, sockaddr_length);
+
     if (bytes_written >= 0) {
       socket->error = kSbSocketOk;
       return static_cast<int>(bytes_written);