Skip to content

Added Linux support to qc_handle_{accept,bind,connect} functions. #23

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Added Linux support to qc_handle_{accept,bind,connect} functions. #23

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
0839660
Added Linux support to tcp tunneling
MCApollo Aug 6, 2020
59a223a
guest-socket: clean up addrlen's
MCApollo Aug 6, 2020
1377261
guest-socket: reformat code; handle errno's on host side
MCApollo Aug 11, 2020
df08c4c
guest-socket: Improved readability and flow
MCApollo Aug 12, 2020
1f63f61
guest-socket: inherit flags from fd
MCApollo Aug 26, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
126 changes: 112 additions & 14 deletions hw/arm/guest-socket.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,59 @@ static int32_t find_free_socket(void) {
return -1;
}


#ifndef __APPLE__
/* convert_sockaddr_from/to darwin:
See `guest-services/socket.h`:
* Darwin still uses `sin_len` in sockaddr,
causing errors when copying memory from iOS.
*/
static struct sockaddr_in convert_sockaddr_from_darwin(struct darwin_sockaddr_in from) {
return (struct sockaddr_in) {
from.sin_family,
from.sin_port,
from.sin_addr,
{ [0 ... sizeof(from.sin_zero) - 1] = 0 } };
}

static struct darwin_sockaddr_in convert_sockaddr_to_darwin(struct sockaddr_in from) {
return (struct darwin_sockaddr_in) {
sizeof(from),
from.sin_family,
from.sin_port,
from.sin_addr,
{ [0 ... sizeof(from.sin_zero) - 1] = 0 } };
}
#endif

/* darwin_error:
At compile-time, redefine what errors mean to tcp-tunnel before sending them.
*/
static int darwin_error(int err) {
#ifdef __APPLE__
return err;
#else
int result;

switch(err) {
case EAGAIN:
/* case EWOULDBLOCK: */
result = 35;
break;
case EDEADLK:
/* case EDEADLOCK: */
result = 11;
break;
default:
result = err;
break;
}

return result;
#endif
}


int32_t qc_handle_socket(CPUState *cpu, int32_t domain, int32_t type,
int32_t protocol)
{
Expand All @@ -49,17 +102,17 @@ int32_t qc_handle_socket(CPUState *cpu, int32_t domain, int32_t type,
guest_svcs_errno = ENOTSOCK;
} else if ((guest_svcs_fds[retval] = socket(domain, type, protocol)) < 0) {
retval = -1;
guest_svcs_errno = errno;
guest_svcs_errno = darwin_error(errno);
}

return retval;
}

int32_t qc_handle_accept(CPUState *cpu, int32_t sckt, struct sockaddr *g_addr,
int32_t qc_handle_accept(CPUState *cpu, int32_t sckt, struct SOCKADDR *g_addr,
socklen_t *g_addrlen)
{
struct sockaddr_in addr;
socklen_t addrlen;
socklen_t addrlen = sizeof(addr);

VERIFY_FD(sckt);

Expand All @@ -72,8 +125,25 @@ int32_t qc_handle_accept(CPUState *cpu, int32_t sckt, struct sockaddr *g_addr,
(struct sockaddr *) &addr,
&addrlen)) < 0) {
retval = -1;
guest_svcs_errno = errno;
guest_svcs_errno = darwin_error(errno);
} else {
#ifndef __APPLE__
/* XXX:
On Linux, the new socket returned by accept() does not inherit file
status flags such as O_NONBLOCK and O_ASYNC from the listening socket.
*/

if ((fcntl(guest_svcs_fds[retval], F_SETFL, fcntl(guest_svcs_fds[sckt], F_GETFL) | O_NONBLOCK)) < 0) {
retval = -1;
guest_svcs_errno = darwin_error(errno);

return retval;
}

struct darwin_sockaddr_in darwin_addr = convert_sockaddr_to_darwin(addr);
cpu_memory_rw_debug(cpu, (target_ulong) g_addr, (uint8_t*) &darwin_addr,
sizeof(darwin_addr), 1);
#endif
cpu_memory_rw_debug(cpu, (target_ulong) g_addr, (uint8_t*) &addr,
sizeof(addr), 1);
cpu_memory_rw_debug(cpu, (target_ulong) g_addrlen,
Expand All @@ -83,10 +153,13 @@ int32_t qc_handle_accept(CPUState *cpu, int32_t sckt, struct sockaddr *g_addr,
return retval;
}

int32_t qc_handle_bind(CPUState *cpu, int32_t sckt, struct sockaddr *g_addr,
int32_t qc_handle_bind(CPUState *cpu, int32_t sckt, struct SOCKADDR *g_addr,
socklen_t addrlen)
{
struct sockaddr_in addr;
#ifndef __APPLE__
struct darwin_sockaddr_in darwin_addr;
#endif

VERIFY_FD(sckt);

Expand All @@ -95,25 +168,39 @@ int32_t qc_handle_bind(CPUState *cpu, int32_t sckt, struct sockaddr *g_addr,
if (addrlen > sizeof(addr)) {
guest_svcs_errno = ENOMEM;
} else {
#ifndef __APPLE__
cpu_memory_rw_debug(cpu, (target_ulong) g_addr, (uint8_t*) &darwin_addr,
sizeof(darwin_addr), 0);
addr = convert_sockaddr_from_darwin(darwin_addr);
#else
cpu_memory_rw_debug(cpu, (target_ulong) g_addr, (uint8_t*) &addr,
sizeof(addr), 0);

sizeof(addr), 0);
#endif
if ((retval = bind(guest_svcs_fds[sckt], (struct sockaddr *) &addr,
addrlen)) < 0) {
guest_svcs_errno = errno;
guest_svcs_errno = darwin_error(errno);
} else {
#ifndef __APPLE__
darwin_addr = convert_sockaddr_to_darwin(addr);
cpu_memory_rw_debug(cpu, (target_ulong) g_addr, (uint8_t*) &darwin_addr,
sizeof(darwin_addr), 1);
#else
cpu_memory_rw_debug(cpu, (target_ulong) g_addr, (uint8_t*) &addr,
sizeof(addr), 1);
#endif
}
}

return retval;
}

int32_t qc_handle_connect(CPUState *cpu, int32_t sckt, struct sockaddr *g_addr,
int32_t qc_handle_connect(CPUState *cpu, int32_t sckt, struct SOCKADDR *g_addr,
socklen_t addrlen)
{
struct sockaddr_in addr;
#ifndef __APPLE__
struct darwin_sockaddr_in darwin_addr;
#endif

VERIFY_FD(sckt);

Expand All @@ -122,15 +209,26 @@ int32_t qc_handle_connect(CPUState *cpu, int32_t sckt, struct sockaddr *g_addr,
if (addrlen > sizeof(addr)) {
guest_svcs_errno = ENOMEM;
} else {
#ifndef __APPLE__
cpu_memory_rw_debug(cpu, (target_ulong) g_addr, (uint8_t*) &darwin_addr,
sizeof(darwin_addr), 0);
addr = convert_sockaddr_from_darwin(darwin_addr);
#else
cpu_memory_rw_debug(cpu, (target_ulong) g_addr, (uint8_t*) &addr,
sizeof(addr), 0);

#endif
if ((retval = connect(guest_svcs_fds[sckt], (struct sockaddr *) &addr,
addrlen)) < 0) {
guest_svcs_errno = errno;
guest_svcs_errno = darwin_error(errno);
} else {
#ifndef __APPLE__
darwin_addr = convert_sockaddr_to_darwin(addr);
cpu_memory_rw_debug(cpu, (target_ulong) g_addr, (uint8_t*) &darwin_addr,
sizeof(darwin_addr), 1);
#else
cpu_memory_rw_debug(cpu, (target_ulong) g_addr, (uint8_t*) &addr,
sizeof(addr), 1);
#endif
}
}

Expand All @@ -144,7 +242,7 @@ int32_t qc_handle_listen(CPUState *cpu, int32_t sckt, int32_t backlog)
int retval = 0;

if ((retval = listen(guest_svcs_fds[sckt], backlog)) < 0) {
guest_svcs_errno = errno;
guest_svcs_errno = darwin_error(errno);
}

return retval;
Expand All @@ -162,7 +260,7 @@ int32_t qc_handle_recv(CPUState *cpu, int32_t sckt, void *g_buffer,
if (length > MAX_BUF_SIZE) {
guest_svcs_errno = ENOMEM;
} else if ((retval = recv(guest_svcs_fds[sckt], buffer, length, flags)) <= 0) {
guest_svcs_errno = errno;
guest_svcs_errno = darwin_error(errno);
} else {
cpu_memory_rw_debug(cpu, (target_ulong) g_buffer, buffer, retval, 1);
}
Expand All @@ -184,7 +282,7 @@ int32_t qc_handle_send(CPUState *cpu, int32_t sckt, void *g_buffer,
cpu_memory_rw_debug(cpu, (target_ulong) g_buffer, buffer, length, 0);

if ((retval = send(guest_svcs_fds[sckt], buffer, length, flags)) < 0) {
guest_svcs_errno = errno;
guest_svcs_errno = darwin_error(errno);
}
}

Expand Down
41 changes: 36 additions & 5 deletions include/hw/arm/guest-services/socket.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,37 @@
#include "sys/socket.h"
#endif

#ifdef __APPLE__
#define SOCKADDR sockaddr
#define SOCKADDR_IN sockaddr_in
Comment on lines +36 to +37
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for these defines. You can use darwin_sockaddr/darwin_sockaddr_in directly. If you don't want to redefine the structs in Darwin, you can use #define darwin_sockaddr sockaddr and #define darwin_sockaddr_in sockaddr_inwhen __APPLE__ is defined.

#else /* __linux__ */
/* XXX: A lot of this is defined by the standard.
But doesn't hurt to ensure Darwin/BSD lengths.

- in_addr_t should be uint32 on all systems:
typedef unsigned int in_addr_t;
struct in_addr {
in_addr_t s_addr;
};
*/

struct darwin_sockaddr {
unsigned char sa_len;
unsigned char sa_family;
char sa_data[14];
};
struct darwin_sockaddr_in {
unsigned char sin_len;
unsigned char sin_family;
unsigned short sin_port;
struct in_addr sin_addr;
char sin_zero[8];
};

#define SOCKADDR darwin_sockaddr
#define SOCKADDR_IN darwin_sockaddr_in
#endif

#pragma GCC diagnostic push
#pragma GCC diagnostic ignored "-Wredundant-decls"
extern int32_t guest_svcs_errno;
Expand All @@ -47,13 +78,13 @@ typedef struct __attribute__((packed)) {

typedef struct __attribute__((packed)) {
int32_t socket;
struct sockaddr *addr;
struct SOCKADDR *addr;
socklen_t *addrlen;
} qc_accept_args_t;

typedef struct __attribute__((packed)) {
int32_t socket;
struct sockaddr *addr;
struct SOCKADDR *addr;
socklen_t addrlen;
} qc_bind_args_t, qc_connect_args_t;

Expand All @@ -72,11 +103,11 @@ typedef struct __attribute__((packed)) {
#ifndef OUT_OF_TREE_BUILD
int32_t qc_handle_socket(CPUState *cpu, int32_t domain, int32_t type,
int32_t protocol);
int32_t qc_handle_accept(CPUState *cpu, int32_t sckt, struct sockaddr *addr,
int32_t qc_handle_accept(CPUState *cpu, int32_t sckt, struct SOCKADDR *addr,
socklen_t *addrlen);
int32_t qc_handle_bind(CPUState *cpu, int32_t sckt, struct sockaddr *addr,
int32_t qc_handle_bind(CPUState *cpu, int32_t sckt, struct SOCKADDR *addr,
socklen_t addrlen);
int32_t qc_handle_connect(CPUState *cpu, int32_t sckt, struct sockaddr *addr,
int32_t qc_handle_connect(CPUState *cpu, int32_t sckt, struct SOCKADDR *addr,
socklen_t addrlen);
int32_t qc_handle_listen(CPUState *cpu, int32_t sckt, int32_t backlog);
int32_t qc_handle_recv(CPUState *cpu, int32_t sckt, void *buffer,
Expand Down