struct udp {
struct udpcommon uc;
struct udpsocks socks;
+ bool_t addr_configured;
};
/*
assert(ix>=0 && ix<socks->n_socks);
snprintf(sbuf, sizeof(sbuf), "udp:%s%s-%s",
iaddr_to_string(&socks->socks[ix].addr),
- ca->ix<0 ? "&" : "",
+ ca->ix<0 && socks->n_socks>1 ? "&" : "",
iaddr_to_string(&ca->ia));
return sbuf;
}
}
void udp_sock_experienced(struct log_if *lg, struct udpcommon *uc,
- const char *socksdesc, struct udpsock *us,
- bool_t recvsend, int af,
+ struct udpsocks *socks, struct udpsock *us,
+ const union iaddr *dest, int af,
int r, int errnoval)
{
bool_t success=r>=0;
- if (us->experienced[recvsend][af][success]++)
+ if (us->experienced[!!dest][af][success]++)
return;
lg_perror(lg, uc->cc.cl.description, &uc->cc.loc,
success ? M_INFO : M_WARNING,
success ? 0 : errnoval,
- "%s %s experiencing some %s %s%s%s",
- socksdesc,iaddr_to_string(&us->addr),
+ "%s %s experiencing some %s %s%s%s%s%s%s",
+ socks->desc,iaddr_to_string(&us->addr),
success?"success":"trouble",
- recvsend?"transmitting":"receiving",
- af?" ":"", af?af_name(af):"");
+ dest?"transmitting":"receiving",
+ af?" ":"", af?af_name(af):"",
+ dest?" (to ":"",
+ dest?iaddr_to_string(dest):"",
+ dest?")":"");
}
static void udp_socks_afterpoll(void *state, struct pollfd *fds, int nfds)
/* Check that the packet came from our poxy server;
we shouldn't be contacted directly by anybody else
(since they can trivially forge source addresses) */
- if (!iaddr_equal(&from,&uc->proxy)) {
+ if (!iaddr_equal(&from,&uc->proxy,False)) {
Message(M_INFO,"udp: received packet that's not "
"from the proxy\n");
BUF_FREE(cc->rbuf);
ca.ia=from;
ca.ix=i;
done=comm_notify(&cc->notify, cc->rbuf, &ca);
- if (!done) {
+ if (done) {
+ udp_sock_experienced(0,uc,socks,us,0,
+ from.sa.sa_family,0,0);
+ } else {
uint32_t msgtype;
if (cc->rbuf->size>12 /* prevents traffic amplification */
&& ((msgtype=get_uint32(cc->rbuf->start+8))
BUF_ASSERT_FREE(cc->rbuf);
} else { /* rv<=0 */
if (errno!=EINTR && !iswouldblock(errno))
- udp_sock_experienced(0,uc, "socket",us, 0,0, rv,errno);
+ udp_sock_experienced(0,uc,socks,us, 0,0, rv,errno);
BUF_FREE(cc->rbuf);
}
} while (rv>=0);
memcpy(sa+6,&dest->ia.sin.sin_port,2);
int r=sendto(us->fd,sa,buf->size+8,0,&uc->proxy.sa,
iaddr_socklen(&uc->proxy));
- udp_sock_experienced(0,uc, "proxy",us, 1,0, r,errno);
+ udp_sock_experienced(0,uc,socks,us, &dest->ia,0, r,errno);
buf_unprepend(buf,8);
} else {
int i,r;
continue;
r=sendto(us->fd, buf->start, buf->size, 0,
&dest->ia.sa, iaddr_socklen(&dest->ia));
- udp_sock_experienced(0,uc, "socket",us, 1,af, r,errno);
+ udp_sock_experienced(0,uc,socks,us, &dest->ia,af, r,errno);
if (r>=0) return True;
if (!(errno==EAFNOSUPPORT || errno==ENETUNREACH))
/* who knows what that error means? */
}
}
+#define FAIL_LG 0, cc->cl.description, &cc->loc, failmsgclass
+#define FAIL(...) do{ \
+ lg_perror(FAIL_LG,errno,__VA_ARGS__); \
+ goto failed; \
+ }while(0)
+
+static bool_t record_socket_gotaddr(struct udpcommon *uc, struct udpsock *us,
+ int failmsgclass)
+{
+ struct commcommon *cc=&uc->cc;
+ socklen_t salen=sizeof(us->addr);
+ int r=getsockname(us->fd,&us->addr.sa,&salen);
+ if (r) FAIL("getsockname()");
+ if (salen>sizeof(us->addr)) { errno=0; FAIL("getsockname() length"); }
+ return True;
+
+ failed:
+ return False;
+}
+
+bool_t udp_import_socket(struct udpcommon *uc, struct udpsock *us,
+ int failmsgclass, int fd)
+{
+ FILLZERO(us->experienced);
+ us->fd=fd;
+ return record_socket_gotaddr(uc,us,failmsgclass);
+}
+
bool_t udp_make_socket(struct udpcommon *uc, struct udpsock *us,
int failmsgclass)
{
struct commcommon *cc=&uc->cc;
us->fd=-1;
-#define FAIL_LG 0, cc->cl.description, &cc->loc, failmsgclass
-#define FAIL(...) do{ \
- lg_perror(FAIL_LG,errno,__VA_ARGS__); \
- goto failed; \
- }while(0)
-
FILLZERO(us->experienced);
us->fd=socket(addr->sa.sa_family, SOCK_DGRAM, IPPROTO_UDP);
if (us->fd<0) FAIL("socket");
if (uc->authbind) {
pid_t c;
int status;
+ char desc[200];
+ snprintf(desc,sizeof(desc),"authbind for %s: %s",
+ iaddr_to_string(addr), uc->authbind);
/* XXX this fork() and waitpid() business needs to be hidden
in some system-specific library functions. */
if (WIFEXITED(status) && WEXITSTATUS(status)<127) {
int es=WEXITSTATUS(status);
lg_perror(FAIL_LG,es,
- "authbind exited with error exit status %d;"
- " indicates error",es);
+ "%s exited with error exit status %d;"
+ " indicates error",desc,es);
} else {
- lg_exitstatus(FAIL_LG,status,"authbind");
+ lg_exitstatus(FAIL_LG,status,desc);
}
goto failed;
}
if (bind(us->fd, &addr->sa, iaddr_socklen(addr))!=0)
FAIL("bind (%s)",iaddr_to_string(addr));
}
+
+ bool_t ok=record_socket_gotaddr(uc,us,failmsgclass);
+ if (!ok) goto failed;
+
return True;
failed:
udp_destroy_socket(uc,us);
return False;
+}
#undef FAIL
-}
-void udp_socks_register(struct udpcommon *uc, struct udpsocks *socks)
+void udp_socks_register(struct udpcommon *uc, struct udpsocks *socks,
+ const char *desc)
{
socks->uc=uc;
+ socks->desc=desc;
socks->interest=
register_for_poll(socks,udp_socks_beforepoll,udp_socks_afterpoll,"udp");
}
struct udpsocks *socks=&st->socks;
struct udpcommon *uc=&st->uc;
int i;
- for (i=0; i<socks->n_socks; i++)
- udp_make_socket(uc,&socks->socks[i],M_FATAL);
+ bool_t anydone=0;
+
+ for (i=0; i<socks->n_socks; i++) {
+ bool_t required=st->addr_configured
+ || (!anydone && i==socks->n_socks-1);
+ anydone += udp_make_socket(uc,&socks->socks[i],
+ required ? M_FATAL : M_WARNING);
+ }
- udp_socks_register(uc,socks);
+ udp_socks_register(uc,socks, uc->use_proxy ? "proxy" : "socket");
add_hook(PHASE_CHILDPERSIST,udp_childpersist_hook,st);
}
};
caddrl=dict_lookup(d,"address");
- socks->n_socks=caddrl ? list_length(caddrl) : (int)ARRAY_SIZE(defaultaddrs);
+ st->addr_configured=!!caddrl;
+ socks->n_socks=st->addr_configured ? list_length(caddrl)
+ : (int)ARRAY_SIZE(defaultaddrs);
if (socks->n_socks<=0 || socks->n_socks>UDP_MAX_SOCKETS)
cfgfatal(cc->loc,"udp","`address' must be 1..%d addresses",
UDP_MAX_SOCKETS);
for (i=0; i<socks->n_socks; i++) {
struct udpsock *us=&socks->socks[i];
- if (!list_length(caddrl)) {
+ if (!st->addr_configured) {
us->addr=defaultaddrs[i];
} else {
string_item_to_iaddr(list_elem(caddrl,i),uc->port,&us->addr,"udp");