[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[GNUnet-SVN] r12571 - gnunet/src/transport
From: |
gnunet |
Subject: |
[GNUnet-SVN] r12571 - gnunet/src/transport |
Date: |
Mon, 16 Aug 2010 13:50:44 +0200 |
Author: wachs
Date: 2010-08-16 13:50:43 +0200 (Mon, 16 Aug 2010)
New Revision: 12571
Modified:
gnunet/src/transport/plugin_transport_http.c
gnunet/src/transport/plugin_transport_https.c
Log:
fixing bugs
Modified: gnunet/src/transport/plugin_transport_http.c
===================================================================
--- gnunet/src/transport/plugin_transport_http.c 2010-08-16 11:05:20 UTC
(rev 12570)
+++ gnunet/src/transport/plugin_transport_http.c 2010-08-16 11:50:43 UTC
(rev 12571)
@@ -613,6 +613,7 @@
return GNUNET_OK;
}
t6 = GNUNET_malloc(sizeof(struct IPv6HttpAddress));
+ GNUNET_assert(t6 != NULL);
if (plugin->bind6_address != NULL)
{
if (0 == memcmp(&plugin->bind6_address->sin6_addr, &bnd_cmp6, sizeof
(struct in6_addr)))
@@ -685,8 +686,9 @@
{
struct Session *ps = cls;
+ GNUNET_assert(ps != NULL);
+
struct HTTP_PeerContext *pc = ps->peercontext;
- GNUNET_assert(ps != NULL);
GNUNET_assert(pc != NULL);
#if DEBUG_HTTP
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
@@ -798,7 +800,7 @@
int res = GNUNET_NO;
int send_error_to_client;
void * addr;
- size_t addr_len;
+ size_t addr_len = 0 ;
GNUNET_assert(cls !=NULL);
send_error_to_client = GNUNET_NO;
@@ -1289,7 +1291,7 @@
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,"Connection %X: Message with %u bytes
sent, removing message from queue \n",ps, msg->pos);
#endif
/* Calling transmit continuation */
- if (( NULL != ps->pending_msgs_tail) && (NULL !=
ps->pending_msgs_tail->transmit_cont))
+ if (NULL != ps->pending_msgs_tail->transmit_cont)
msg->transmit_cont
(ps->pending_msgs_tail->transmit_cont_cls,&(ps->peercontext)->identity,GNUNET_OK);
remove_http_message(ps, msg);
}
@@ -2410,26 +2412,26 @@
if ((plugin->use_ipv6==GNUNET_YES) && (GNUNET_CONFIGURATION_have_value
(env->cfg,
"transport-http", "BINDTO6")))
{
- GNUNET_break (GNUNET_OK ==
- GNUNET_CONFIGURATION_get_value_string
(env->cfg,
+ if (GNUNET_OK == GNUNET_CONFIGURATION_get_value_string (env->cfg,
"transport-http",
"BINDTO6",
-
&plugin->bind_hostname));
+
&plugin->bind_hostname))
+ {
+ plugin->bind6_address = GNUNET_malloc(sizeof(struct
sockaddr_in6));
+ plugin->bind6_address->sin6_family = AF_INET6;
+ plugin->bind6_address->sin6_port = htons (port);
- plugin->bind6_address = GNUNET_malloc(sizeof(struct sockaddr_in6));
- plugin->bind6_address->sin6_family = AF_INET6;
- plugin->bind6_address->sin6_port = htons (port);
-
- if (inet_pton(AF_INET6,plugin->bind_hostname,
&plugin->bind6_address->sin6_addr)<=0)
- {
- GNUNET_log_from (GNUNET_ERROR_TYPE_ERROR,
- "http",
- _("Misconfigured address to
bind to in configuration!\n"),
- "transport-http");
- GNUNET_free(plugin->bind6_address);
- GNUNET_free(plugin->bind_hostname);
- plugin->bind_hostname = NULL;
- plugin->bind6_address = NULL;
+ if (inet_pton(AF_INET6,plugin->bind_hostname,
&plugin->bind6_address->sin6_addr)<=0)
+ {
+ GNUNET_log_from (GNUNET_ERROR_TYPE_ERROR,
+ "http",
+ _("Misconfigured
address to bind to in configuration!\n"),
+ "transport-http");
+ GNUNET_free(plugin->bind6_address);
+ GNUNET_free(plugin->bind_hostname);
+ plugin->bind_hostname = NULL;
+ plugin->bind6_address = NULL;
+ }
}
}
@@ -2500,7 +2502,7 @@
}
else
{
- char * tmp;
+ char * tmp = NULL;
if ((plugin->use_ipv6 == GNUNET_YES) && (plugin->use_ipv4 ==
GNUNET_YES))
GNUNET_asprintf(&tmp,"with IPv4 and IPv6 enabled");
if ((plugin->use_ipv6 == GNUNET_NO) && (plugin->use_ipv4 == GNUNET_YES))
Modified: gnunet/src/transport/plugin_transport_https.c
===================================================================
--- gnunet/src/transport/plugin_transport_https.c 2010-08-16 11:05:20 UTC
(rev 12570)
+++ gnunet/src/transport/plugin_transport_https.c 2010-08-16 11:50:43 UTC
(rev 12571)
@@ -622,6 +622,7 @@
return GNUNET_OK;
}
t6 = GNUNET_malloc(sizeof(struct IPv6HttpAddress));
+ GNUNET_assert(t6 != NULL);
if (plugin->bind6_address != NULL)
{
if (0 == memcmp(&plugin->bind6_address->sin6_addr, &bnd_cmp6, sizeof
(struct in6_addr)))
@@ -694,8 +695,9 @@
{
struct Session *ps = cls;
+ GNUNET_assert(ps != NULL);
+
struct HTTP_PeerContext *pc = ps->peercontext;
- GNUNET_assert(ps != NULL);
GNUNET_assert(pc != NULL);
#if DEBUG_HTTPS
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
@@ -807,7 +809,7 @@
int res = GNUNET_NO;
int send_error_to_client;
void * addr;
- size_t addr_len;
+ size_t addr_len = 0;
GNUNET_assert(cls !=NULL);
send_error_to_client = GNUNET_NO;
@@ -1300,7 +1302,7 @@
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,"Connection %X: Message with %u bytes
sent, removing message from queue \n",ps, msg->pos);
#endif
/* Calling transmit continuation */
- if (( NULL != ps->pending_msgs_tail) && (NULL !=
ps->pending_msgs_tail->transmit_cont))
+ if (NULL != ps->pending_msgs_tail->transmit_cont)
msg->transmit_cont
(ps->pending_msgs_tail->transmit_cont_cls,&(ps->peercontext)->identity,GNUNET_OK);
remove_http_message(ps, msg);
}
@@ -2514,30 +2516,30 @@
GNUNET_asprintf(&cert_file,"https.cert");
}
- /* Reading ipv4 addresse to bind to from config file */
+ /* Should plugin use ipv6? */
if ((plugin->use_ipv6==GNUNET_YES) && (GNUNET_CONFIGURATION_have_value
(env->cfg,
"transport-https", "BINDTO6")))
{
- GNUNET_break (GNUNET_OK ==
- GNUNET_CONFIGURATION_get_value_string
(env->cfg,
+ if (GNUNET_OK == GNUNET_CONFIGURATION_get_value_string (env->cfg,
"transport-https",
"BINDTO6",
-
&plugin->bind_hostname));
+
&plugin->bind_hostname))
+ {
+ plugin->bind6_address = GNUNET_malloc(sizeof(struct
sockaddr_in6));
+ plugin->bind6_address->sin6_family = AF_INET6;
+ plugin->bind6_address->sin6_port = htons (port);
- plugin->bind6_address = GNUNET_malloc(sizeof(struct sockaddr_in6));
- plugin->bind6_address->sin6_family = AF_INET6;
- plugin->bind6_address->sin6_port = htons (port);
-
- if (inet_pton(AF_INET6,plugin->bind_hostname,
&plugin->bind6_address->sin6_addr)<=0)
- {
- GNUNET_log_from (GNUNET_ERROR_TYPE_ERROR,
- "http",
- _("Misconfigured address to
bind to in configuration!\n"),
- "transport-https");
- GNUNET_free(plugin->bind6_address);
- GNUNET_free(plugin->bind_hostname);
- plugin->bind_hostname = NULL;
- plugin->bind6_address = NULL;
+ if (inet_pton(AF_INET6,plugin->bind_hostname,
&plugin->bind6_address->sin6_addr)<=0)
+ {
+ GNUNET_log_from (GNUNET_ERROR_TYPE_ERROR,
+ "http",
+ _("Misconfigured
address to bind to in configuration!\n"),
+ "transport-https");
+ GNUNET_free(plugin->bind6_address);
+ GNUNET_free(plugin->bind_hostname);
+ plugin->bind_hostname = NULL;
+ plugin->bind6_address = NULL;
+ }
}
}
@@ -2674,7 +2676,7 @@
}
else
{
- char * tmp;
+ char * tmp = NULL;
if ((plugin->use_ipv6 == GNUNET_YES) && (plugin->use_ipv4 ==
GNUNET_YES))
GNUNET_asprintf(&tmp,"with IPv4 and IPv6 enabled");
if ((plugin->use_ipv6 == GNUNET_NO) && (plugin->use_ipv4 == GNUNET_YES))
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [GNUnet-SVN] r12571 - gnunet/src/transport,
gnunet <=