X-Git-Url: https://tinc-vpn.org/git/browse?p=tinc;a=blobdiff_plain;f=src%2Fuml_socket%2Fdevice.c;h=9d8bb490da583e6df123a3631c6ff2b39fcc0ad3;hp=08339d80b8ec2cf9d10e21e4d3671df03396a69f;hb=1f00810da336f3b7132df17b7fe4625748ff4b63;hpb=dcec713675b604f5ef82e64d0671727e3f5ea518 diff --git a/src/uml_socket/device.c b/src/uml_socket/device.c index 08339d80..9d8bb490 100644 --- a/src/uml_socket/device.c +++ b/src/uml_socket/device.c @@ -30,8 +30,6 @@ #include "utils.h" #include "route.h" -#include "system.h" - int device_fd = -1; static int listen_fd = -1; static int request_fd = -1; @@ -39,14 +37,14 @@ static int data_fd = -1; static int write_fd = -1; static int state = 0; char *device; -char *iface; +char *iface = NULL; char *device_info; extern char *identname; extern bool running; -int device_total_in = 0; -int device_total_out = 0; +static int device_total_in = 0; +static int device_total_out = 0; enum request_type { REQ_NEW_CONTROL }; @@ -75,8 +73,7 @@ bool setup_device(void) if(!get_config_string(lookup_config(config_tree, "Device"), &device)) asprintf(&device, LOCALSTATEDIR "/run/%s.umlsocket", identname); - if(!get_config_string(lookup_config(config_tree, "Interface"), &iface)) - iface = device; + get_config_string(lookup_config(config_tree, "Interface"), &iface); device_info = _("UML network socket"); @@ -264,16 +261,18 @@ bool write_packet(vpn_packet_t *packet) if(state != 2) { ifdebug(TRAFFIC) logger(LOG_DEBUG, _("Dropping packet of %d bytes to %s: not connected to UML yet"), packet->len, device_info); - return true; + return false; } ifdebug(TRAFFIC) logger(LOG_DEBUG, _("Writing packet of %d bytes to %s"), packet->len, device_info); - if(sendto(write_fd, packet->data, packet->len, 0, &request.sock, sizeof request.sock) < 0) { - logger(LOG_ERR, _("Can't write to %s %s: %s"), device_info, device, - strerror(errno)); - running = false; + if(write(write_fd, packet->data, packet->len) < 0) { + if(errno != EINTR && errno != EAGAIN) { + logger(LOG_ERR, _("Can't write to %s %s: %s"), device_info, device, strerror(errno)); + running = false; + } + return false; }