interesting
/
getdns
mirror of https://github.com/getdnsapi/getdns.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Revert syntactic/style changes 

So actual changes aren't obfuscated
This commit is contained in:
Willem Toorop 2015-12-17 12:37:33 +01:00
parent 0d6d8116bc
commit a2e15a169d
3 changed files with 54 additions and 58 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
configure.ac
View File

@ -429,8 +429,6 @@ case "$enable_stub_only" in
;; ;;
esac esac
# search to set include and library paths right # search to set include and library paths right
# find libidn # find libidn
my_with_libidn=1 my_with_libidn=1

56
src/context.c
View File

@ -1793,7 +1793,7 @@ getdns_context_set_dnssec_allowed_skew(struct getdns_context *context,
*/ */
getdns_return_t getdns_return_t
getdns_context_set_upstream_recursive_servers(struct getdns_context *context, getdns_context_set_upstream_recursive_servers(struct getdns_context *context,
struct getdns_list *upstream_list) struct getdns_list *upstream_list)
{ {
getdns_return_t r; getdns_return_t r;
size_t count = 0; size_t count = 0;
@ -1810,16 +1810,16 @@ struct getdns_list *upstream_list)
return GETDNS_RETURN_CONTEXT_UPDATE_FAIL; return GETDNS_RETURN_CONTEXT_UPDATE_FAIL;
} }
memset(&hints, 0, sizeof(struct addrinfo)); memset(&hints, 0, sizeof(struct addrinfo));
hints.ai_family = AF_UNSPEC; /* Allow IPv4 or IPv6 */ hints.ai_family = AF_UNSPEC; /* Allow IPv4 or IPv6 */
hints.ai_socktype = 0; /* Datagram socket */ hints.ai_socktype = 0; /* Datagram socket */
hints.ai_flags = AI_NUMERICHOST; /* No reverse name lookups */ hints.ai_flags = AI_NUMERICHOST; /* No reverse name lookups */
hints.ai_protocol = 0; /* Any protocol */ hints.ai_protocol = 0; /* Any protocol */
hints.ai_canonname = NULL; hints.ai_canonname = NULL;
hints.ai_addr = NULL; hints.ai_addr = NULL;
hints.ai_next = NULL; hints.ai_next = NULL;
upstreams = upstreams_create( upstreams = upstreams_create(
context, count * GETDNS_UPSTREAM_TRANSPORTS); context, count * GETDNS_UPSTREAM_TRANSPORTS);
for (i = 0; i < count; i++) { for (i = 0; i < count; i++) {
getdns_dict *dict; getdns_dict *dict;
getdns_bindata *address_type; getdns_bindata *address_type;
@ -1834,7 +1834,7 @@ struct getdns_list *upstream_list)
goto error; goto error;
if ((r = getdns_dict_get_bindata( if ((r = getdns_dict_get_bindata(
dict, "address_type", &address_type))) dict, "address_type",&address_type)))
goto error; goto error;
if (address_type->size < 4) if (address_type->size < 4)
goto invalid_parameter; goto invalid_parameter;
@ -1845,24 +1845,24 @@ struct getdns_list *upstream_list)
else goto invalid_parameter; else goto invalid_parameter;
if ((r = getdns_dict_get_bindata( if ((r = getdns_dict_get_bindata(
dict, "address_data", &address_data))) dict, "address_data", &address_data)))
goto error; goto error;
if ((addr.ss_family == AF_INET && if ((addr.ss_family == AF_INET &&
address_data->size != 4) || address_data->size != 4) ||
(addr.ss_family == AF_INET6 && (addr.ss_family == AF_INET6 &&
address_data->size != 16)) address_data->size != 16))
goto invalid_parameter; goto invalid_parameter;
if (inet_ntop(addr.ss_family, address_data->data, if (inet_ntop(addr.ss_family, address_data->data,
addrstr, 1024) == NULL) addrstr, 1024) == NULL)
goto invalid_parameter; goto invalid_parameter;
if (getdns_dict_get_bindata(dict, "scope_id", &scope_id) == if (getdns_dict_get_bindata(dict, "scope_id", &scope_id) ==
GETDNS_RETURN_GOOD) { GETDNS_RETURN_GOOD) {
if (strlen(addrstr) + scope_id->size > 1022) if (strlen(addrstr) + scope_id->size > 1022)
goto invalid_parameter; goto invalid_parameter;
eos = &addrstr[strlen(addrstr)]; eos = &addrstr[strlen(addrstr)];
*eos++ = '%'; *eos++ = '%';
(void)memcpy(eos, scope_id->data, scope_id->size); (void) memcpy(eos, scope_id->data, scope_id->size);
eos[scope_id->size] = 0; eos[scope_id->size] = 0;
} }
@ -1875,10 +1875,10 @@ struct getdns_list *upstream_list)
continue; continue;
if (getdns_upstream_transports[j] != GETDNS_TRANSPORT_TLS) if (getdns_upstream_transports[j] != GETDNS_TRANSPORT_TLS)
(void)getdns_dict_get_int(dict, "port", &port); (void) getdns_dict_get_int(dict, "port", &port);
else else
(void)getdns_dict_get_int(dict, "tls_port", &port); (void) getdns_dict_get_int(dict, "tls_port", &port);
(void)snprintf(portstr, 1024, "%d", (int)port); (void) snprintf(portstr, 1024, "%d", (int)port);
if (getaddrinfo(addrstr, portstr, &hints, &ai)) if (getaddrinfo(addrstr, portstr, &hints, &ai))
goto invalid_parameter; goto invalid_parameter;
@ -1894,12 +1894,12 @@ struct getdns_list *upstream_list)
upstream_init(upstream, upstreams, ai); upstream_init(upstream, upstreams, ai);
upstream->transport = getdns_upstream_transports[j]; upstream->transport = getdns_upstream_transports[j];
if (getdns_upstream_transports[j] == GETDNS_TRANSPORT_TLS || if (getdns_upstream_transports[j] == GETDNS_TRANSPORT_TLS ||
getdns_upstream_transports[j] == GETDNS_TRANSPORT_STARTTLS) { getdns_upstream_transports[j] == GETDNS_TRANSPORT_STARTTLS) {
if ((r = getdns_dict_get_bindata( if ((r = getdns_dict_get_bindata(
dict, "tls_auth_name", &tls_auth_name)) == GETDNS_RETURN_GOOD) { dict, "tls_auth_name", &tls_auth_name)) == GETDNS_RETURN_GOOD) {
/*TODO: VALIDATE THIS STRING!*/ /*TODO: VALIDATE THIS STRING!*/
memcpy(upstream->tls_auth_name, memcpy(upstream->tls_auth_name,
(char *)tls_auth_name->data, (char *)tls_auth_name->data,
tls_auth_name->size); tls_auth_name->size);
upstream->tls_auth_name[tls_auth_name->size] = '\0'; upstream->tls_auth_name[tls_auth_name->size] = '\0';
} }
@ -1920,7 +1920,8 @@ invalid_parameter:
error: error:
_getdns_upstreams_dereference(upstreams); _getdns_upstreams_dereference(upstreams);
return GETDNS_RETURN_CONTEXT_UPDATE_FAIL; return GETDNS_RETURN_CONTEXT_UPDATE_FAIL;
} } /* getdns_context_set_upstream_recursive_servers */
static void static void
set_ub_edns_maximum_udp_payload_size(struct getdns_context* context, set_ub_edns_maximum_udp_payload_size(struct getdns_context* context,
@ -2355,7 +2356,7 @@ _getdns_context_prepare_for_resolution(struct getdns_context *context,
#ifdef HAVE_TLS_v1_2 #ifdef HAVE_TLS_v1_2
/* Create client context, use TLS v1.2 only for now */ /* Create client context, use TLS v1.2 only for now */
context->tls_ctx = SSL_CTX_new(TLSv1_2_client_method()); context->tls_ctx = SSL_CTX_new(TLSv1_2_client_method());
if (context->tls_ctx == NULL) if(context->tls_ctx == NULL)
#ifndef USE_WINSOCK #ifndef USE_WINSOCK
return GETDNS_RETURN_BAD_CONTEXT; return GETDNS_RETURN_BAD_CONTEXT;
#else #else
@ -2364,16 +2365,13 @@ _getdns_context_prepare_for_resolution(struct getdns_context *context,
/* Be strict and only use the cipher suites recommended in RFC7525 /* Be strict and only use the cipher suites recommended in RFC7525
Unless we later fallback to opportunistic. */ Unless we later fallback to opportunistic. */
const char* const PREFERRED_CIPHERS = "EECDH+aRSA+AESGCM:EECDH+aECDSA+AESGCM:EDH+aRSA+AESGCM"; const char* const PREFERRED_CIPHERS = "EECDH+aRSA+AESGCM:EECDH+aECDSA+AESGCM:EDH+aRSA+AESGCM";
if (!SSL_CTX_set_cipher_list(context->tls_ctx, PREFERRED_CIPHERS)) { if (!SSL_CTX_set_cipher_list(context->tls_ctx, PREFERRED_CIPHERS))
return GETDNS_RETURN_BAD_CONTEXT; return GETDNS_RETURN_BAD_CONTEXT;
} if (!SSL_CTX_set_default_verify_paths(context->tls_ctx))
if (!SSL_CTX_set_default_verify_paths(context->tls_ctx)) {
return GETDNS_RETURN_BAD_CONTEXT; return GETDNS_RETURN_BAD_CONTEXT;
}
#else #else
if (tls_only_is_in_transports_list(context) == 1) { if (tls_only_is_in_transports_list(context) == 1)
return GETDNS_RETURN_BAD_CONTEXT; return GETDNS_RETURN_BAD_CONTEXT;
}
/* A null tls_ctx will make TLS fail and fallback to the other /* A null tls_ctx will make TLS fail and fallback to the other
transports will kick-in.*/ transports will kick-in.*/
#endif #endif

8
src/stub.c
View File

@ -773,8 +773,8 @@ stub_tcp_write(int fd, getdns_tcp_state *tcp, getdns_network_req *netreq)
written = write(fd, netreq->query - 2, pkt_len + 2); written = write(fd, netreq->query - 2, pkt_len + 2);
#endif #endif
if ((written == -1 && (errno == EAGAIN || if ((written == -1 && (errno == EAGAIN ||
errno == EWOULDBLOCK)) || errno == EWOULDBLOCK)) ||
written < pkt_len + 2) { written < pkt_len + 2) {
#endif #endif
/* We couldn't write the whole packet. /* We couldn't write the whole packet.
* We have to return with STUB_TCP_AGAIN. * We have to return with STUB_TCP_AGAIN.
@ -1374,7 +1374,7 @@ stub_tcp_read_cb(void *userarg)
int q; int q;
switch ((q = stub_tcp_read(netreq->fd, &netreq->tcp, switch ((q = stub_tcp_read(netreq->fd, &netreq->tcp,
&dnsreq->context->mf, &netreq->event))) { &dnsreq->context->mf))) {
case STUB_TCP_AGAIN: case STUB_TCP_AGAIN:
return; return;
@ -1452,7 +1452,7 @@ upstream_read_cb(void *userarg)
&upstream->upstreams->mf); &upstream->upstreams->mf);
else else
q = stub_tcp_read(upstream->fd, &upstream->tcp, q = stub_tcp_read(upstream->fd, &upstream->tcp,
&upstream->upstreams->mf, &netreq->event); &upstream->upstreams->mf);
switch (q) { switch (q) {
case STUB_TCP_AGAIN: case STUB_TCP_AGAIN: