/*
This file is part of GNUnet.
- (C) 2010-2013 Christian Grothoff (and other contributing authors)
+ Copyright (C) 2010-2015 GNUnet e.V.
- GNUnet is free software; you can redistribute it and/or modify
- it under the terms of the GNU General Public License as published
- by the Free Software Foundation; either version 3, or (at your
- option) any later version.
+ GNUnet is free software: you can redistribute it and/or modify it
+ under the terms of the GNU Affero General Public License as published
+ by the Free Software Foundation, either version 3 of the License,
+ or (at your option) any later version.
GNUnet is distributed in the hope that it will be useful, but
WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- General Public License for more details.
-
- You should have received a copy of the GNU General Public License
- along with GNUnet; see the file COPYING. If not, write to the
- Free Software Foundation, Inc., 59 Temple Place - Suite 330,
- Boston, MA 02111-1307, USA.
+ Affero General Public License for more details.
+
+ You should have received a copy of the GNU Affero General Public License
+ along with this program. If not, see <http://www.gnu.org/licenses/>.
*/
/**
* @author Christian Grothoff
*/
#include "platform.h"
-#include "gnunet-service-transport_validation.h"
-#include "gnunet-service-transport_plugins.h"
+#include "gnunet-service-transport_ats.h"
#include "gnunet-service-transport_hello.h"
-#include "gnunet-service-transport_blacklist.h"
#include "gnunet-service-transport_neighbours.h"
+#include "gnunet-service-transport_plugins.h"
+#include "gnunet-service-transport_validation.h"
#include "gnunet-service-transport.h"
#include "gnunet_hello_lib.h"
#include "gnunet_ats_service.h"
#include "gnunet_peerinfo_service.h"
#include "gnunet_signatures.h"
+/**
+ * Current state of a validation process.
+ *
+ * FIXME: what state is used to indicate that a validation
+ * was successful? If that is clarified/determined, "UGH" in
+ * ~gnunetpeerinfogtk.c:1103 should be resolved.
+ */
+enum GNUNET_TRANSPORT_ValidationState
+{
+ /**
+ * Undefined state
+ *
+ * Used for final callback indicating operation done
+ */
+ GNUNET_TRANSPORT_VS_NONE,
+
+ /**
+ * Fresh validation entry
+ *
+ * Entry was just created, no validation process was executed
+ */
+ GNUNET_TRANSPORT_VS_NEW,
+
+ /**
+ * Updated validation entry
+ *
+ * This is an update for an existing validation entry
+ */
+ GNUNET_TRANSPORT_VS_UPDATE,
+
+ /**
+ * Timeout for validation entry
+ *
+ * A timeout occured during the validation process
+ */
+ GNUNET_TRANSPORT_VS_TIMEOUT,
+
+ /**
+ * Validation entry is removed
+ *
+ * The validation entry is getting removed due to a failed validation
+ */
+ GNUNET_TRANSPORT_VS_REMOVE
+};
+
+
+
/**
* How long is a PONG signature valid? We'll recycle a signature until
{
/**
- * Type will be GNUNET_MESSAGE_TYPE_TRANSPORT_PING
+ * Type will be #GNUNET_MESSAGE_TYPE_TRANSPORT_PING
*/
struct GNUNET_MessageHeader header;
{
/**
- * Type will be GNUNET_MESSAGE_TYPE_TRANSPORT_PONG
+ * Type will be #GNUNET_MESSAGE_TYPE_TRANSPORT_PONG
*/
struct GNUNET_MessageHeader header;
struct GNUNET_CRYPTO_EddsaSignature signature;
/**
- * GNUNET_SIGNATURE_PURPOSE_TRANSPORT_PONG_OWN to confirm that this is a
+ * #GNUNET_SIGNATURE_PURPOSE_TRANSPORT_PONG_OWN to confirm that this is a
* plausible address for the signing peer.
*/
struct GNUNET_CRYPTO_EccSignaturePurpose purpose;
*/
struct GST_BlacklistCheck *bc;
- /**
- * Public key of the peer.
- */
- struct GNUNET_CRYPTO_EddsaPublicKey public_key;
-
- /**
- * The identity of the peer. FIXME: duplicated (also in 'address')
- */
- struct GNUNET_PeerIdentity pid;
-
/**
* Cached PONG signature
*/
/**
* ID of task that will clean up this entry if nothing happens.
*/
- GNUNET_SCHEDULER_TaskIdentifier timeout_task;
+ struct GNUNET_SCHEDULER_Task *timeout_task;
/**
* ID of task that will trigger address revalidation.
*/
- GNUNET_SCHEDULER_TaskIdentifier revalidation_task;
+ struct GNUNET_SCHEDULER_Task *revalidation_task;
/**
* At what time did we send the latest validation request (PING)?
* Current state of this validation entry
*/
enum GNUNET_TRANSPORT_ValidationState state;
+
/**
* Challenge number we used.
*/
uint32_t challenge;
/**
- * When passing the address in 'add_valid_peer_address', did we
+ * When passing the address in #add_valid_peer_address(), did we
* copy the address to the HELLO yet?
*/
int copied;
*/
int expecting_pong;
- enum GNUNET_ATS_Network_Type network;
-};
-
-
-/**
- * Context of currently active requests to peerinfo
- * for validation of HELLOs.
- */
-struct CheckHelloValidatedContext
-{
-
- /**
- * This is a doubly-linked list.
- */
- struct CheckHelloValidatedContext *next;
-
/**
- * This is a doubly-linked list.
+ * Is this address known to ATS as valid right now?
*/
- struct CheckHelloValidatedContext *prev;
+ int known_to_ats;
/**
- * Hello that we are validating.
+ * Which network type does our address belong to?
*/
- const struct GNUNET_HELLO_Message *hello;
-
+ enum GNUNET_ATS_Network_Type network;
};
-/**
- * Head of linked list of HELLOs awaiting validation.
- */
-static struct CheckHelloValidatedContext *chvc_head;
-
-/**
- * Tail of linked list of HELLOs awaiting validation
- */
-static struct CheckHelloValidatedContext *chvc_tail;
-
/**
* Map of PeerIdentities to 'struct ValidationEntry*'s (addresses
* of the given peer that we are currently validating, have validated
*/
static struct GNUNET_PEERINFO_NotifyContext *pnc;
-
/**
* Minimum delay between to validations
*/
static struct GNUNET_TIME_Relative validation_delay;
/**
- * Number of validations running
+ * Number of validations running; any PING that was not yet
+ * matched by a PONG and for which we have not yet hit the
+ * timeout is considered a running 'validation'.
*/
static unsigned int validations_running;
*/
static struct GNUNET_TIME_Absolute validation_next;
-static GST_ValidationChangedCallback validation_entry_changed_cb;
-static void *validation_entry_changed_cb_cls;
/**
* Context for the validation entry match function.
};
+/**
+ * Provide an update on the `validation_map` map size to statistics.
+ * This function should be called whenever the `validation_map`
+ * is changed.
+ */
+static void
+publish_ve_stat_update ()
+{
+ GNUNET_STATISTICS_set (GST_stats,
+ gettext_noop ("# Addresses in validation map"),
+ GNUNET_CONTAINER_multipeermap_size (validation_map),
+ GNUNET_NO);
+}
+
+
/**
* Iterate over validation entries until a matching one is found.
*
- * @param cls the 'struct ValidationEntryMatchContext'
+ * @param cls the `struct ValidationEntryMatchContext *`
* @param key peer identity (unused)
- * @param value a 'struct ValidationEntry' to match
- * @return GNUNET_YES if the entry does not match,
- * GNUNET_NO if the entry does match
+ * @param value a `struct ValidationEntry *` to match
+ * @return #GNUNET_YES if the entry does not match,
+ * #GNUNET_NO if the entry does match
*/
static int
-validation_entry_match (void *cls, const struct GNUNET_PeerIdentity * key, void *value)
+validation_entry_match (void *cls,
+ const struct GNUNET_PeerIdentity *key,
+ void *value)
{
struct ValidationEntryMatchContext *vemc = cls;
struct ValidationEntry *ve = value;
- if (0 == GNUNET_HELLO_address_cmp (ve->address, vemc->address))
+ if (0 == GNUNET_HELLO_address_cmp (ve->address,
+ vemc->address))
{
vemc->ve = ve;
return GNUNET_NO;
return GNUNET_YES;
}
+
+/**
+ * A validation entry changed. Update the state and notify
+ * monitors.
+ *
+ * @param ve validation entry that changed
+ * @param state new state
+ */
static void
-validation_entry_changed (struct ValidationEntry *ve, enum GNUNET_TRANSPORT_ValidationState state)
+validation_entry_changed (struct ValidationEntry *ve,
+ enum GNUNET_TRANSPORT_ValidationState state)
{
- char *t_sent = GNUNET_strdup(GNUNET_STRINGS_absolute_time_to_string(ve->send_time));
- char *t_valid = GNUNET_strdup(GNUNET_STRINGS_absolute_time_to_string(ve->valid_until));
- char *t_next = GNUNET_strdup(GNUNET_STRINGS_absolute_time_to_string(ve->next_validation));
- GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Validation entry changed for peer `%s' address `%s':\n\tSent: %s\n\tValid: %s\n\tNext: %s\n",
- GNUNET_i2s(&ve->pid), GST_plugins_a2s(ve->address),
- t_sent, t_valid, t_next);
ve->state = state;
-
- GNUNET_free (t_sent);
- GNUNET_free (t_valid);
- GNUNET_free (t_next);
- validation_entry_changed_cb (validation_entry_changed_cb_cls, &ve->pid,
- ve->address, ve->send_time, ve->valid_until, ve->next_validation, state);
}
*
* @param cls (unused)
* @param key peer identity (unused)
- * @param value a 'struct ValidationEntry' to clean up
- * @return GNUNET_YES (continue to iterate)
+ * @param value a `struct ValidationEntry *` to clean up
+ * @return #GNUNET_YES (continue to iterate)
*/
static int
-cleanup_validation_entry (void *cls, const struct GNUNET_PeerIdentity * key, void *value)
+cleanup_validation_entry (void *cls,
+ const struct GNUNET_PeerIdentity *key,
+ void *value)
{
struct ValidationEntry *ve = value;
- ve->next_validation = GNUNET_TIME_absolute_get_zero_();
+ ve->next_validation = GNUNET_TIME_UNIT_ZERO_ABS;
ve->valid_until = GNUNET_TIME_UNIT_ZERO_ABS;
/* Notify about deleted entry */
- validation_entry_changed (ve, GNUNET_TRANSPORT_VS_REMOVE);
+ validation_entry_changed (ve,
+ GNUNET_TRANSPORT_VS_REMOVE);
if (NULL != ve->bc)
{
}
GNUNET_break (GNUNET_OK ==
GNUNET_CONTAINER_multipeermap_remove (validation_map,
- &ve->pid, ve));
+ &ve->address->peer,
+ ve));
+ publish_ve_stat_update ();
+ if (GNUNET_YES == ve->known_to_ats)
+ {
+ GST_ats_expire_address (ve->address);
+ GNUNET_assert (GNUNET_NO ==
+ GST_ats_is_known_no_session (ve->address));
+ ve->known_to_ats = GNUNET_NO;
+ }
GNUNET_HELLO_address_free (ve->address);
- if (GNUNET_SCHEDULER_NO_TASK != ve->timeout_task)
+ if (NULL != ve->timeout_task)
{
GNUNET_SCHEDULER_cancel (ve->timeout_task);
- ve->timeout_task = GNUNET_SCHEDULER_NO_TASK;
+ ve->timeout_task = NULL;
}
- if (GNUNET_SCHEDULER_NO_TASK != ve->revalidation_task)
+ if (NULL != ve->revalidation_task)
{
GNUNET_SCHEDULER_cancel (ve->revalidation_task);
- ve->revalidation_task = GNUNET_SCHEDULER_NO_TASK;
+ ve->revalidation_task = NULL;
}
- if ((GNUNET_YES == ve->expecting_pong) &&
- (validations_running > 0))
+ if ( (GNUNET_YES == ve->expecting_pong) &&
+ (validations_running > 0) )
{
- validations_running --;
- GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
- "Validation finished, %u validation processes running\n",
- validations_running);
+ validations_running--;
+ GNUNET_STATISTICS_set (GST_stats,
+ gettext_noop ("# validations running"),
+ validations_running,
+ GNUNET_NO);
}
GNUNET_free (ve);
return GNUNET_OK;
* Address validation cleanup task. Assesses if the record is no
* longer valid and then possibly triggers its removal.
*
- * @param cls the 'struct ValidationEntry'
- * @param tc scheduler context (unused)
+ * @param cls the `struct ValidationEntry`
*/
static void
-timeout_hello_validation (void *cls,
- const struct GNUNET_SCHEDULER_TaskContext *tc)
+timeout_hello_validation (void *cls)
{
struct ValidationEntry *ve = cls;
struct GNUNET_TIME_Absolute max;
struct GNUNET_TIME_Relative left;
- ve->timeout_task = GNUNET_SCHEDULER_NO_TASK;
- max = GNUNET_TIME_absolute_max (ve->valid_until, ve->revalidation_block);
+ ve->timeout_task = NULL;
+ /* For valid addresses, we want to wait until the expire;
+ for addresses under PING validation, we want to wait
+ until we give up on the PING */
+ max = GNUNET_TIME_absolute_max (ve->valid_until,
+ ve->revalidation_block);
left = GNUNET_TIME_absolute_get_remaining (max);
if (left.rel_value_us > 0)
{
- /* should wait a bit longer */
+ /* We should wait a bit longer. This happens when
+ address lifetimes are extended due to successful
+ validations. */
ve->timeout_task =
- GNUNET_SCHEDULER_add_delayed (left, &timeout_hello_validation, ve);
+ GNUNET_SCHEDULER_add_delayed (left,
+ &timeout_hello_validation,
+ ve);
return;
}
GNUNET_STATISTICS_update (GST_stats,
- gettext_noop ("# address records discarded"), 1,
+ gettext_noop ("# address records discarded (timeout)"),
+ 1,
GNUNET_NO);
- cleanup_validation_entry (NULL, &ve->pid, ve);
+ cleanup_validation_entry (NULL,
+ &ve->address->peer,
+ ve);
}
* Function called with the result from blacklisting.
* Send a PING to the other peer if a communication is allowed.
*
- * @param cls our 'struct ValidationEntry'
+ * @param cls our `struct ValidationEntry`
* @param pid identity of the other peer
- * @param result #GNUNET_OK if the connection is allowed, #GNUNET_NO if not
+ * @param address_null address associated with the request, always NULL
+ * @param session_null session associated with the request, always NULL
+ * @param result #GNUNET_OK if the connection is allowed,
+ * #GNUNET_NO if not,
+ * #GNUNET_SYSERR if operation was aborted
*/
static void
transmit_ping_if_allowed (void *cls,
const struct GNUNET_PeerIdentity *pid,
+ const struct GNUNET_HELLO_Address *address_null,
+ struct GNUNET_ATS_Session *session_null,
int result)
{
struct ValidationEntry *ve = cls;
struct GNUNET_TRANSPORT_PluginFunctions *papi;
struct GNUNET_TIME_Absolute next;
const struct GNUNET_MessageHeader *hello;
- enum GNUNET_ATS_Network_Type network;
ssize_t ret;
size_t tsize;
size_t slen;
uint16_t hsize;
+ struct GNUNET_ATS_Session *session;
ve->bc = NULL;
- if (GNUNET_NO == result)
+ if (GNUNET_OK != result)
{
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
- "Blacklist denies to send PING to `%s' `%s' `%s'\n",
+ "Blacklist denies sending PING to `%s' `%s' `%s'\n",
GNUNET_i2s (pid),
GST_plugins_a2s (ve->address),
ve->address->transport_name);
+ GNUNET_STATISTICS_update (GST_stats,
+ gettext_noop ("# address records discarded (blacklist)"),
+ 1,
+ GNUNET_NO);
+ cleanup_validation_entry (NULL,
+ pid,
+ ve);
return;
}
-
- GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
- "Transmitting plain PING to `%s' `%s' `%s'\n",
- GNUNET_i2s (pid),
- GST_plugins_a2s (ve->address),
- ve->address->transport_name);
-
- next = GNUNET_TIME_absolute_add (GNUNET_TIME_absolute_get(),
- validation_delay);
- if (next.abs_value_us > validation_next.abs_value_us)
- validation_next = next; /* We're going to send a PING so delay next validation */
-
- slen = strlen (ve->address->transport_name) + 1;
hello = GST_hello_get ();
+ GNUNET_assert (NULL != hello);
+ slen = strlen (ve->address->transport_name) + 1;
hsize = ntohs (hello->size);
- tsize =
- sizeof (struct TransportPingMessage) + ve->address->address_length +
- slen + hsize;
+ tsize = sizeof (struct TransportPingMessage) +
+ ve->address->address_length + slen + hsize;
ping.header.size =
htons (sizeof (struct TransportPingMessage) +
ping.challenge = htonl (ve->challenge);
ping.target = *pid;
- if (tsize >= GNUNET_SERVER_MAX_MESSAGE_SIZE)
+ if (tsize >= GNUNET_MAX_MESSAGE_SIZE)
{
- GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
- _
- ("Not transmitting `%s' with `%s', message too big (%u bytes!). This should not happen.\n"),
- "HELLO", "PING", (unsigned int) tsize);
- /* message too big (!?), get rid of HELLO */
+ GNUNET_break (0);
hsize = 0;
tsize =
sizeof (struct TransportPingMessage) + ve->address->address_length +
slen + hsize;
}
{
- char message_buf[tsize];
-
- /* build message with structure:
- * [HELLO][TransportPingMessage][Transport name][Address] */
- memcpy (message_buf, hello, hsize);
- memcpy (&message_buf[hsize], &ping, sizeof (struct TransportPingMessage));
- memcpy (&message_buf[sizeof (struct TransportPingMessage) + hsize],
- ve->address->transport_name, slen);
- memcpy (&message_buf[sizeof (struct TransportPingMessage) + slen + hsize],
- ve->address->address, ve->address->address_length);
+ char message_buf[tsize] GNUNET_ALIGN;
+
+ GNUNET_memcpy (message_buf,
+ hello,
+ hsize);
+ GNUNET_memcpy (&message_buf[hsize],
+ &ping,
+ sizeof (struct TransportPingMessage));
+ GNUNET_memcpy (&message_buf[sizeof (struct TransportPingMessage) + hsize],
+ ve->address->transport_name,
+ slen);
+ GNUNET_memcpy (&message_buf[sizeof (struct TransportPingMessage) + slen + hsize],
+ ve->address->address,
+ ve->address->address_length);
papi = GST_plugins_find (ve->address->transport_name);
- if (papi == NULL)
- ret = -1;
- else
+ GNUNET_assert (NULL != papi);
+ session = papi->get_session (papi->cls,
+ ve->address);
+ if (NULL == session)
{
- GNUNET_assert (papi->send != NULL);
- GNUNET_assert (papi->get_session != NULL);
- struct Session * session = papi->get_session(papi->cls, ve->address);
-
- if (session != NULL)
- {
- ret = papi->send (papi->cls, session,
- message_buf, tsize,
- PING_PRIORITY, ACCEPTABLE_PING_DELAY,
- NULL, NULL);
- network = papi->get_network (ve->address, session);
- if (GNUNET_ATS_NET_UNSPECIFIED == network)
- {
- GNUNET_log(GNUNET_ERROR_TYPE_ERROR,
- "Could not obtain a valid network for `%s' `%s'\n",
- GNUNET_i2s (pid), GST_plugins_a2s (ve->address));
- GNUNET_break(0);
- }
- GST_neighbours_notify_data_sent (pid, ve->address, session, tsize);
- }
- else
- {
- /* Could not get a valid session */
- GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Could not get a valid session for `%s' `%s'\n",
- GNUNET_i2s (pid), GST_plugins_a2s (ve->address));
- ret = -1;
- }
+ /* Could not get a valid session */
+ GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+ "Failed to get session to send PING to `%s' at `%s'\n",
+ GNUNET_i2s (pid),
+ GST_plugins_a2s (ve->address));
+ return;
}
- }
- if (-1 != ret)
- {
+
+ ret = papi->send (papi->cls, session,
+ message_buf, tsize,
+ PING_PRIORITY,
+ ACCEPTABLE_PING_DELAY,
+ NULL, NULL);
+ if (-1 == ret)
+ {
+ GNUNET_log (GNUNET_ERROR_TYPE_INFO,
+ "Failed to send PING to `%s' at `%s'\n",
+ GNUNET_i2s (pid),
+ GST_plugins_a2s (ve->address));
+ return;
+ }
+ GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+ "Transmitted plain PING to `%s' `%s' `%s'\n",
+ GNUNET_i2s (pid),
+ GST_plugins_a2s (ve->address),
+ ve->address->transport_name);
+ ve->network = papi->get_network (papi->cls,
+ session);
+ GNUNET_break (GNUNET_ATS_NET_UNSPECIFIED != ve->network);
+ GST_neighbours_notify_data_sent (ve->address,
+ session,
+ tsize);
+ next = GNUNET_TIME_relative_to_absolute (validation_delay);
+ validation_next = GNUNET_TIME_absolute_max (next,
+ validation_next);
ve->send_time = GNUNET_TIME_absolute_get ();
GNUNET_STATISTICS_update (GST_stats,
- gettext_noop
- ("# PING without HELLO messages sent"), 1,
+ gettext_noop ("# PINGs for address validation sent"),
+ 1,
GNUNET_NO);
-
- ve->network = network;
ve->expecting_pong = GNUNET_YES;
- validations_running ++;
- GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
- "Validation started, %u validation processes running\n",
- validations_running);
+ validations_running++;
+ GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+ "Validation started, %u validation processes running\n",
+ validations_running);
+ GNUNET_STATISTICS_set (GST_stats,
+ gettext_noop ("# validations running"),
+ validations_running,
+ GNUNET_NO);
/* Notify about PING sent */
- validation_entry_changed (ve, GNUNET_TRANSPORT_VS_UPDATE);
+ validation_entry_changed (ve,
+ GNUNET_TRANSPORT_VS_UPDATE);
+
}
}
/**
* Do address validation again to keep address valid.
*
- * @param cls the 'struct ValidationEntry'
- * @param tc scheduler context (unused)
+ * @param cls the `struct ValidationEntry`
*/
static void
-revalidate_address (void *cls,
- const struct GNUNET_SCHEDULER_TaskContext *tc)
+revalidate_address (void *cls)
{
struct ValidationEntry *ve = cls;
struct GNUNET_TIME_Relative canonical_delay;
struct GST_BlacklistCheck *bc;
uint32_t rdelay;
- ve->revalidation_task = GNUNET_SCHEDULER_NO_TASK;
+ ve->revalidation_task = NULL;
delay = GNUNET_TIME_absolute_get_remaining (ve->revalidation_block);
- /* How long until we can possibly permit the next PING? */
- canonical_delay =
- (ve->in_use ==
- GNUNET_YES) ? CONNECTED_PING_FREQUENCY
- : ((GNUNET_TIME_absolute_get_remaining (ve->valid_until).rel_value_us >
- 0) ? VALIDATED_PING_FREQUENCY : UNVALIDATED_PING_KEEPALIVE);
- if (delay.rel_value_us > canonical_delay.rel_value_us * 2)
- {
- /* situation changed, recalculate delay */
- delay = canonical_delay;
- ve->revalidation_block = GNUNET_TIME_relative_to_absolute (delay);
- }
+ /* Considering current connectivity situation, what is the maximum
+ block period permitted? */
+ if (GNUNET_YES == ve->in_use)
+ canonical_delay = CONNECTED_PING_FREQUENCY;
+ else if (GNUNET_TIME_absolute_get_remaining (ve->valid_until).rel_value_us > 0)
+ canonical_delay = VALIDATED_PING_FREQUENCY;
+ else
+ canonical_delay = UNVALIDATED_PING_KEEPALIVE;
+ /* Use delay that is MIN of original delay and possibly adjusted
+ new maximum delay (which may be lower); the real delay
+ is originally randomized between "canonical_delay" and "2 * canonical_delay",
+ so continue to permit that window for the operation. */
+ delay = GNUNET_TIME_relative_min (delay,
+ GNUNET_TIME_relative_multiply (canonical_delay,
+ 2));
+ ve->revalidation_block = GNUNET_TIME_relative_to_absolute (delay);
if (delay.rel_value_us > 0)
{
/* should wait a bit longer */
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
- "Waiting for %s longer before validating address `%s'\n",
+ "Waiting for %s longer before (re)validating address `%s'\n",
GNUNET_STRINGS_relative_time_to_string (delay,
GNUNET_YES),
GST_plugins_a2s (ve->address));
ve->revalidation_task =
- GNUNET_SCHEDULER_add_delayed (delay, &revalidate_address, ve);
- ve->next_validation = GNUNET_TIME_absolute_add(GNUNET_TIME_absolute_get(), delay);
+ GNUNET_SCHEDULER_add_delayed (delay,
+ &revalidate_address, ve);
+ ve->next_validation = GNUNET_TIME_relative_to_absolute (delay);
return;
}
- blocked_for = GNUNET_TIME_absolute_get_remaining(validation_next);
- if ((validations_running > validations_fast_start_threshold) &&
- (blocked_for.rel_value_us > 0))
+ /* check if globally we have too many active validations at a
+ too high rate, if so, delay ours */
+ blocked_for = GNUNET_TIME_absolute_get_remaining (validation_next);
+ if ( (validations_running > validations_fast_start_threshold) &&
+ (blocked_for.rel_value_us > 0) )
{
/* Validations are blocked, have to wait for blocked_for time */
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
GNUNET_STRINGS_relative_time_to_string (blocked_for,
GNUNET_YES),
GST_plugins_a2s (ve->address));
+ GNUNET_STATISTICS_update (GST_stats,
+ gettext_noop ("# validations delayed by global throttle"),
+ 1,
+ GNUNET_NO);
ve->revalidation_task =
- GNUNET_SCHEDULER_add_delayed (blocked_for, &revalidate_address, ve);
- ve->next_validation = GNUNET_TIME_absolute_add(GNUNET_TIME_absolute_get(), blocked_for);
+ GNUNET_SCHEDULER_add_delayed (blocked_for,
+ &revalidate_address,
+ ve);
+ ve->next_validation = GNUNET_TIME_relative_to_absolute (blocked_for);
return;
}
- ve->revalidation_block = GNUNET_TIME_relative_to_absolute (canonical_delay);
+ /* We are good to go; remember to not go again for `canonical_delay` time;
+ add up to `canonical_delay` to randomize start time */
+ ve->revalidation_block = GNUNET_TIME_relative_to_absolute (canonical_delay);
/* schedule next PINGing with some extra random delay to avoid synchronous re-validations */
rdelay =
GNUNET_CRYPTO_random_u32 (GNUNET_CRYPTO_QUALITY_WEAK,
canonical_delay.rel_value_us);
- /* Debug code for mantis 0002726 */
- if (GNUNET_TIME_UNIT_FOREVER_REL.rel_value_us ==
- GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_MICROSECONDS, rdelay).rel_value_us)
- {
- GNUNET_break (0);
- delay = canonical_delay;
- }
- else
- {
- delay = GNUNET_TIME_relative_add (canonical_delay,
- GNUNET_TIME_relative_multiply
- (GNUNET_TIME_UNIT_MICROSECONDS, rdelay));
- }
- /* End debug code for mantis 0002726*/
+ delay = GNUNET_TIME_relative_add (canonical_delay,
+ GNUNET_TIME_relative_multiply
+ (GNUNET_TIME_UNIT_MICROSECONDS,
+ rdelay));
+
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
"Validating now, next scheduled for %s, now validating address `%s'\n",
GNUNET_STRINGS_relative_time_to_string (blocked_for,
GNUNET_YES),
GST_plugins_a2s (ve->address));
ve->revalidation_task =
- GNUNET_SCHEDULER_add_delayed (delay, &revalidate_address, ve);
- ve->next_validation = GNUNET_TIME_absolute_add(GNUNET_TIME_absolute_get(), delay);
+ GNUNET_SCHEDULER_add_delayed (delay,
+ &revalidate_address,
+ ve);
+ ve->next_validation = GNUNET_TIME_relative_to_absolute (delay);
/* start PINGing by checking blacklist */
GNUNET_STATISTICS_update (GST_stats,
gettext_noop ("# address revalidations started"), 1,
GNUNET_NO);
- bc = GST_blacklist_test_allowed (&ve->pid, ve->address->transport_name,
- &transmit_ping_if_allowed, ve);
+ if (NULL != ve->bc)
+ {
+ GST_blacklist_test_cancel (ve->bc);
+ ve->bc = NULL;
+ }
+ bc = GST_blacklist_test_allowed (&ve->address->peer,
+ ve->address->transport_name,
+ &transmit_ping_if_allowed,
+ ve,
+ NULL,
+ NULL);
if (NULL != bc)
- ve->bc = bc; /* only set 'bc' if 'transmit_ping_if_allowed' was not already
- * called... */
+ {
+ /* If transmit_ping_if_allowed was already called it may have freed ve,
+ * so only set ve->bc if it has not been called.
+ */
+ ve->bc = bc;
+ }
}
* the given address and transport. If none exists, create one (but
* without starting any validation).
*
- * @param public_key public key of the peer, NULL for unknown
* @param address address to find
* @return validation entry matching the given specifications, NULL
* if we don't have an existing entry and no public key was given
*/
static struct ValidationEntry *
-find_validation_entry (const struct GNUNET_CRYPTO_EddsaPublicKey *public_key,
- const struct GNUNET_HELLO_Address *address)
+find_validation_entry (const struct GNUNET_HELLO_Address *address)
{
struct ValidationEntryMatchContext vemc;
struct ValidationEntry *ve;
&validation_entry_match, &vemc);
if (NULL != (ve = vemc.ve))
return ve;
- if (public_key == NULL)
- return NULL;
+ GNUNET_assert (GNUNET_NO ==
+ GST_ats_is_known_no_session (address));
ve = GNUNET_new (struct ValidationEntry);
ve->in_use = GNUNET_SYSERR; /* not defined */
ve->address = GNUNET_HELLO_address_copy (address);
- ve->public_key = *public_key;
- ve->pid = address->peer;
- ve->pong_sig_valid_until = GNUNET_TIME_absolute_get_zero_();
- memset (&ve->pong_sig_cache, '\0', sizeof (struct GNUNET_CRYPTO_EddsaSignature));
+ ve->pong_sig_valid_until = GNUNET_TIME_UNIT_ZERO_ABS;
+ memset (&ve->pong_sig_cache,
+ '\0',
+ sizeof (struct GNUNET_CRYPTO_EddsaSignature));
ve->latency = GNUNET_TIME_UNIT_FOREVER_REL;
ve->challenge =
GNUNET_CRYPTO_random_u32 (GNUNET_CRYPTO_QUALITY_NONCE, UINT32_MAX);
ve->timeout_task =
GNUNET_SCHEDULER_add_delayed (UNVALIDATED_PING_KEEPALIVE,
- &timeout_hello_validation, ve);
- GNUNET_CONTAINER_multipeermap_put (validation_map, &address->peer,
+ &timeout_hello_validation,
+ ve);
+ GNUNET_CONTAINER_multipeermap_put (validation_map,
+ &address->peer,
ve,
GNUNET_CONTAINER_MULTIHASHMAPOPTION_MULTIPLE);
- validation_entry_changed (ve, GNUNET_TRANSPORT_VS_NEW);
- ve->expecting_pong = GNUNET_NO;
+ publish_ve_stat_update ();
+ validation_entry_changed (ve,
+ GNUNET_TRANSPORT_VS_NEW);
return ve;
}
* @param cls original HELLO message
* @param address the address
* @param expiration expiration time
- * @return #GNUNET_OK (keep the address)
+ * @return #GNUNET_OK (keep the address), could return
+ * #GNUNET_NO (delete address, but this is ignored);
+ * #GNUNET_SYSERR would abort iteration (but we always iterate all)
*/
static int
add_valid_address (void *cls,
const struct GNUNET_HELLO_Message *hello = cls;
struct ValidationEntry *ve;
struct GNUNET_PeerIdentity pid;
- struct GNUNET_ATS_Information ats;
- struct GNUNET_CRYPTO_EddsaPublicKey public_key;
+ struct GNUNET_ATS_Properties prop;
+ struct GNUNET_TRANSPORT_PluginFunctions *papi;
if (0 == GNUNET_TIME_absolute_get_remaining (expiration).rel_value_us)
return GNUNET_OK; /* expired */
- if ((GNUNET_OK != GNUNET_HELLO_get_id (hello, &pid)) ||
- (GNUNET_OK != GNUNET_HELLO_get_key (hello, &public_key)))
+ if (GNUNET_OK != GNUNET_HELLO_get_id (hello, &pid))
{
GNUNET_break (0);
return GNUNET_OK; /* invalid HELLO !? */
}
- if (0 == memcmp (&GST_my_identity,
- &pid,
- sizeof (struct GNUNET_PeerIdentity)))
+ if (NULL == (papi = GST_plugins_find (address->transport_name)))
{
- /* Peerinfo returned own identity, skip validation */
+ /* might have been valid in the past, but we don't have that
+ plugin loaded right now */
+ return GNUNET_OK;
+ }
+ if (NULL ==
+ papi->address_to_string (papi->cls,
+ address->address,
+ address->address_length))
+ {
+ /* Why do we try to add an ill-formed address? */
+ GNUNET_break (0);
+ GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+ "Address with %u bytes for plugin %s and peer %s is malformed\n",
+ (unsigned int) address->address_length,
+ address->transport_name,
+ GNUNET_i2s (&pid));
return GNUNET_OK;
}
- ve = find_validation_entry (&public_key, address);
- ve->valid_until = GNUNET_TIME_absolute_max (ve->valid_until, expiration);
-
- if (GNUNET_SCHEDULER_NO_TASK == ve->revalidation_task)
+ ve = find_validation_entry (address);
+ ve->network = papi->get_network_for_address (papi->cls,
+ address);
+ GNUNET_break (GNUNET_ATS_NET_UNSPECIFIED != ve->network);
+ ve->valid_until = GNUNET_TIME_absolute_max (ve->valid_until,
+ expiration);
+ if (NULL == ve->revalidation_task)
{
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
"Starting revalidations for valid address `%s'\n",
- GST_plugins_a2s (ve->address));
+ GST_plugins_a2s (ve->address));
ve->next_validation = GNUNET_TIME_absolute_get();
ve->revalidation_task = GNUNET_SCHEDULER_add_now (&revalidate_address, ve);
}
- validation_entry_changed (ve, GNUNET_TRANSPORT_VS_UPDATE);
-
- ats.type = htonl (GNUNET_ATS_NETWORK_TYPE);
- ats.value = htonl (ve->network);
- GNUNET_ATS_address_add (GST_ats, address, NULL, &ats, 1);
-
+ validation_entry_changed (ve,
+ GNUNET_TRANSPORT_VS_UPDATE);
+ memset (&prop, 0, sizeof (prop));
+ prop.scope = ve->network;
+ prop.delay = GNUNET_TIME_relative_divide (ve->latency, 2);
+ if (GNUNET_YES != ve->known_to_ats)
+ {
+ ve->known_to_ats = GNUNET_YES;
+ GST_ats_add_address (address, &prop);
+ GNUNET_assert (GNUNET_YES ==
+ GST_ats_is_known_no_session (ve->address));
+ }
return GNUNET_OK;
}
/**
* Function called for any HELLO known to PEERINFO.
*
- * @param cls unused
- * @param peer id of the peer, NULL for last call
+ * @param cls unused (NULL)
+ * @param peer id of the peer, NULL for last call (during iteration,
+ * as we are monitoring, this should never happen)
* @param hello hello message for the peer (can be NULL)
* @param err_msg error message
*/
static void
-process_peerinfo_hello (void *cls, const struct GNUNET_PeerIdentity *peer,
+process_peerinfo_hello (void *cls,
+ const struct GNUNET_PeerIdentity *peer,
const struct GNUNET_HELLO_Message *hello,
const char *err_msg)
{
GNUNET_assert (NULL != peer);
if (NULL == hello)
return;
+ if (0 == memcmp (&GST_my_identity,
+ peer,
+ sizeof (struct GNUNET_PeerIdentity)))
+ {
+ /* Peerinfo returned own identity, skip validation */
+ return;
+ }
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
"Handling HELLO for peer `%s'\n",
GNUNET_i2s (peer));
/**
* Start the validation subsystem.
*
- * @param cb callback to call with changes to valdidation entries
- * @param cb_cls cls for the callback
* @param max_fds maximum number of fds to use
*/
void
-GST_validation_start (GST_ValidationChangedCallback cb, void *cb_cls, unsigned int max_fds)
+GST_validation_start (unsigned int max_fds)
{
/**
* Initialization for validation throttling
validation_delay.rel_value_us = (GNUNET_CONSTANTS_IDLE_CONNECTION_TIMEOUT.rel_value_us) / (max_fds / 2);
validations_fast_start_threshold = (max_fds / 2);
validations_running = 0;
- validation_entry_changed_cb = cb;
- validation_entry_changed_cb_cls = cb_cls;
- GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Validation uses a fast start threshold of %u connections and a delay between of %s\n ",
+ GNUNET_STATISTICS_set (GST_stats,
+ gettext_noop ("# validations running"),
+ validations_running,
+ GNUNET_NO);
+ GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+ "Validation uses a fast start threshold of %u connections and a delay of %s\n",
validations_fast_start_threshold,
GNUNET_STRINGS_relative_time_to_string (validation_delay,
GNUNET_YES));
void
GST_validation_stop ()
{
- struct CheckHelloValidatedContext *chvc;
-
GNUNET_CONTAINER_multipeermap_iterate (validation_map,
- &cleanup_validation_entry, NULL);
+ &cleanup_validation_entry,
+ NULL);
GNUNET_CONTAINER_multipeermap_destroy (validation_map);
validation_map = NULL;
- while (NULL != (chvc = chvc_head))
- {
- GNUNET_CONTAINER_DLL_remove (chvc_head, chvc_tail, chvc);
- GNUNET_free (chvc);
- }
GNUNET_PEERINFO_notify_cancel (pnc);
}
* Send the given PONG to the given address.
*
* @param cls the PONG message
- * @param public_key public key for the peer, never NULL
* @param valid_until is ZERO if we never validated the address,
* otherwise a time up to when we consider it (or was) valid
* @param validation_block is FOREVER if the address is for an unsupported plugin (from PEERINFO)
*/
static void
multicast_pong (void *cls,
- const struct GNUNET_CRYPTO_EddsaPublicKey *public_key,
struct GNUNET_TIME_Absolute valid_until,
struct GNUNET_TIME_Absolute validation_block,
const struct GNUNET_HELLO_Address *address)
{
struct TransportPongMessage *pong = cls;
struct GNUNET_TRANSPORT_PluginFunctions *papi;
+ struct GNUNET_ATS_Session *session;
papi = GST_plugins_find (address->transport_name);
- if (papi == NULL)
+ if (NULL == papi)
+ {
+ GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+ "Plugin %s not supported, cannot send PONG\n",
+ address->transport_name);
return;
-
- GNUNET_assert (papi->send != NULL);
- GNUNET_assert (papi->get_session != NULL);
-
- struct Session * session = papi->get_session(papi->cls, address);
- if (session == NULL)
+ }
+ GNUNET_assert (NULL != papi->send);
+ GNUNET_assert (NULL != papi->get_session);
+ session = papi->get_session(papi->cls, address);
+ if (NULL == session)
{
GNUNET_break (0);
return;
}
-
+ GST_ats_new_session (address, session);
papi->send (papi->cls, session,
- (const char *) pong, ntohs (pong->header.size),
- PONG_PRIORITY, ACCEPTABLE_PING_DELAY,
+ (const char *) pong,
+ ntohs (pong->header.size),
+ PONG_PRIORITY,
+ ACCEPTABLE_PING_DELAY,
NULL, NULL);
- GST_neighbours_notify_data_sent (&address->peer,
- address, session, pong->header.size);
-
+ GST_neighbours_notify_data_sent (address,
+ session,
+ pong->header.size);
}
GST_validation_handle_ping (const struct GNUNET_PeerIdentity *sender,
const struct GNUNET_MessageHeader *hdr,
const struct GNUNET_HELLO_Address *sender_address,
- struct Session *session)
+ struct GNUNET_ATS_Session *session)
{
const struct TransportPingMessage *ping;
struct TransportPongMessage *pong;
const char *addrend;
char *plugin_name;
char *pos;
- size_t alen;
- size_t slen;
+ size_t len_address;
+ size_t len_plugin;
ssize_t ret;
- int buggy = GNUNET_NO;
struct GNUNET_HELLO_Address address;
+ if (0 ==
+ memcmp (&GST_my_identity,
+ sender,
+ sizeof (struct GNUNET_PeerIdentity)))
+ return GNUNET_OK; /* our own, ignore! */
if (ntohs (hdr->size) < sizeof (struct TransportPingMessage))
{
GNUNET_break_op (0);
gettext_noop ("# PING messages received"), 1,
GNUNET_NO);
addr = (const char *) &ping[1];
- alen = ntohs (hdr->size) - sizeof (struct TransportPingMessage);
+ len_address = ntohs (hdr->size) - sizeof (struct TransportPingMessage);
/* peer wants to confirm that this is one of our addresses, this is what is
* used for address validation */
sig_cache = NULL;
sig_cache_exp = NULL;
papi = NULL;
- if (alen > 0)
+ if (len_address > 0)
{
- addrend = memchr (addr, '\0', alen);
+ addrend = memchr (addr, '\0', len_address);
if (NULL == addrend)
{
GNUNET_break_op (0);
return GNUNET_SYSERR;
}
addrend++;
- slen = strlen (addr) + 1;
- alen -= slen;
+ len_plugin = strlen (addr) + 1;
+ len_address -= len_plugin;
address.local_info = GNUNET_HELLO_ADDRESS_INFO_NONE;
address.address = addrend;
- address.address_length = alen;
+ address.address_length = len_address;
address.transport_name = addr;
address.peer = GST_my_identity;
if (NULL == (papi = GST_plugins_find (plugin_name)))
{
/* we don't have the plugin for this address */
- GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+ GNUNET_log (GNUNET_ERROR_TYPE_INFO,
_("Plugin `%s' not available, cannot confirm having this address\n"),
plugin_name);
GNUNET_free (plugin_name);
return GNUNET_SYSERR;
}
GNUNET_free (plugin_name);
- if (GNUNET_OK != papi->check_address (papi->cls, addrend, alen))
+ if (GNUNET_OK !=
+ papi->check_address (papi->cls,
+ addrend,
+ len_address))
{
GNUNET_STATISTICS_update (GST_stats,
gettext_noop
("# failed address checks during validation"), 1,
GNUNET_NO);
- GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+ GNUNET_log (GNUNET_ERROR_TYPE_INFO,
_("Address `%s' is not one of my addresses, not confirming PING\n"),
GST_plugins_a2s (&address));
return GNUNET_SYSERR;
gettext_noop
("# successful address checks during validation"), 1,
GNUNET_NO);
- GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+ GNUNET_log (GNUNET_ERROR_TYPE_INFO,
"Address `%s' is one of my addresses, confirming PING\n",
GST_plugins_a2s (&address));
}
- if (GNUNET_YES != GST_hello_test_address (&address, &sig_cache, &sig_cache_exp))
+ if (GNUNET_YES !=
+ GST_hello_test_address (&address,
+ &sig_cache,
+ &sig_cache_exp))
{
- if (GNUNET_NO == buggy)
- {
- GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
- _("Not confirming PING from peer `%s' with address `%s' since I cannot confirm having this address.\n"),
- GNUNET_i2s (sender),
- GST_plugins_a2s (&address));
- return GNUNET_SYSERR;
- }
- else
- {
- GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
- _("Received a PING message with validation bug from `%s'\n"),
- GNUNET_i2s (sender));
- }
+ GNUNET_log (GNUNET_ERROR_TYPE_INFO,
+ _("Not confirming PING from peer `%s' with address `%s' since I cannot confirm having this address.\n"),
+ GNUNET_i2s (sender),
+ GST_plugins_a2s (&address));
+ return GNUNET_SYSERR;
}
}
else
{
addrend = NULL; /* make gcc happy */
- slen = 0;
+ len_plugin = 0;
static struct GNUNET_CRYPTO_EddsaSignature no_address_signature;
static struct GNUNET_TIME_Absolute no_address_signature_expiration;
/* message with structure:
* [TransportPongMessage][Transport name][Address] */
- pong = GNUNET_malloc (sizeof (struct TransportPongMessage) + alen + slen);
+ pong = GNUNET_malloc (sizeof (struct TransportPongMessage) + len_address + len_plugin);
pong->header.size =
- htons (sizeof (struct TransportPongMessage) + alen + slen);
+ htons (sizeof (struct TransportPongMessage) + len_address + len_plugin);
pong->header.type = htons (GNUNET_MESSAGE_TYPE_TRANSPORT_PONG);
pong->purpose.size =
htonl (sizeof (struct GNUNET_CRYPTO_EccSignaturePurpose) +
sizeof (uint32_t) + sizeof (struct GNUNET_TIME_AbsoluteNBO) +
- alen + slen);
+ len_address + len_plugin);
pong->purpose.purpose = htonl (GNUNET_SIGNATURE_PURPOSE_TRANSPORT_PONG_OWN);
- memcpy (&pong->challenge, &ping->challenge, sizeof (ping->challenge));
- pong->addrlen = htonl (alen + slen);
- memcpy (&pong[1], addr, slen); /* Copy transport plugin */
- if (alen > 0)
+ GNUNET_memcpy (&pong->challenge, &ping->challenge, sizeof (ping->challenge));
+ pong->addrlen = htonl (len_address + len_plugin);
+ GNUNET_memcpy (&pong[1], addr, len_plugin); /* Copy transport plugin */
+ if (len_address > 0)
{
GNUNET_assert (NULL != addrend);
- memcpy (&((char *) &pong[1])[slen], addrend, alen);
+ GNUNET_memcpy (&((char *) &pong[1])[len_plugin], addrend, len_address);
}
if (GNUNET_TIME_absolute_get_remaining (*sig_cache_exp).rel_value_us <
PONG_SIGNATURE_LIFETIME.rel_value_us / 4)
}
pong->signature = *sig_cache;
- GNUNET_assert (sender_address != NULL);
+ GNUNET_assert (NULL != sender_address);
/* first see if the session we got this PING from can be used to transmit
* a response reliably */
- if (papi == NULL)
+ if (NULL == papi)
+ {
ret = -1;
+ }
else
{
- GNUNET_assert (papi->send != NULL);
- GNUNET_assert (papi->get_session != NULL);
-
- if (session == NULL)
+ GNUNET_assert (NULL != papi->send);
+ GNUNET_assert (NULL != papi->get_session);
+ if (NULL == session)
{
session = papi->get_session (papi->cls, sender_address);
}
- if (session == NULL)
+ if (NULL == session)
{
GNUNET_break (0);
ret = -1;
else
{
ret = papi->send (papi->cls, session,
- (const char *) pong, ntohs (pong->header.size),
+ (const char *) pong,
+ ntohs (pong->header.size),
PONG_PRIORITY, ACCEPTABLE_PING_DELAY,
NULL, NULL);
if (-1 != ret)
- GST_neighbours_notify_data_sent (sender,
- sender_address, session,
+ GST_neighbours_notify_data_sent (sender_address,
+ session,
pong->header.size);
}
}
- if (ret != -1)
+ if (-1 != ret)
{
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
"Transmitted PONG to `%s' via reliable mechanism\n",
/**
- * Context for the #validate_address_iterator() function
+ * Validate an individual address.
+ *
+ * @param address address we should try to validate
*/
-struct ValidateAddressContext
+void
+GST_validation_handle_address (const struct GNUNET_HELLO_Address *address)
{
- /**
- * Hash of the public key of the peer whose address is being validated.
- */
- struct GNUNET_PeerIdentity pid;
-
- /**
- * Public key of the peer whose address is being validated.
- */
- struct GNUNET_CRYPTO_EddsaPublicKey public_key;
+ struct GNUNET_TRANSPORT_PluginFunctions *papi;
+ struct ValidationEntry *ve;
-};
+ papi = GST_plugins_find (address->transport_name);
+ if (NULL == papi)
+ {
+ /* This plugin is currently unvailable ... ignore */
+ GNUNET_log (GNUNET_ERROR_TYPE_INFO,
+ "No plugin available for %s\n",
+ address->transport_name);
+ return;
+ }
+ ve = find_validation_entry (address);
+ if (NULL == ve->revalidation_task)
+ {
+ GNUNET_log (GNUNET_ERROR_TYPE_INFO,
+ "Validation process started for fresh address `%s' of %s\n",
+ GST_plugins_a2s (ve->address),
+ GNUNET_i2s (&ve->address->peer));
+ ve->revalidation_task = GNUNET_SCHEDULER_add_now (&revalidate_address, ve);
+ }
+ else
+ {
+ GNUNET_log (GNUNET_ERROR_TYPE_INFO,
+ "Validation already running for address `%s' of %s\n",
+ GST_plugins_a2s (ve->address),
+ GNUNET_i2s (&ve->address->peer));
+ }
+}
/**
* Iterator callback to go over all addresses and try to validate them
* (unless blocked or already validated).
*
- * @param cls pointer to a `struct ValidateAddressContext`
+ * @param cls NULL
* @param address the address
* @param expiration expiration time
* @return #GNUNET_OK (keep the address)
const struct GNUNET_HELLO_Address *address,
struct GNUNET_TIME_Absolute expiration)
{
- const struct ValidateAddressContext *vac = cls;
- struct ValidationEntry *ve;
-
if (0 == GNUNET_TIME_absolute_get_remaining (expiration).rel_value_us)
{
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
"Skipping expired address from HELLO\n");
return GNUNET_OK; /* expired */
}
- ve = find_validation_entry (&vac->public_key, address);
- if (GNUNET_SCHEDULER_NO_TASK == ve->revalidation_task)
- {
- GNUNET_log (GNUNET_ERROR_TYPE_INFO,
- "Validation process started for fresh address `%s'\n",
- GST_plugins_a2s (ve->address));
- ve->revalidation_task = GNUNET_SCHEDULER_add_now (&revalidate_address, ve);
- }
+ GST_validation_handle_address (address);
return GNUNET_OK;
}
/**
* Add the validated peer address to the HELLO.
*
- * @param cls the 'struct ValidationEntry' with the validated address
- * @param max space in buf
+ * @param cls the `struct ValidationEntry *` with the validated address
+ * @param max space in @a buf
* @param buf where to add the address
- * @return number of bytes written, 0 to signal the
+ * @return number of bytes written, #GNUNET_SYSERR to signal the
* end of the iteration.
*/
-static size_t
-add_valid_peer_address (void *cls, size_t max, void *buf)
+static ssize_t
+add_valid_peer_address (void *cls,
+ size_t max,
+ void *buf)
{
struct ValidationEntry *ve = cls;
if (GNUNET_YES == ve->copied)
- return 0; /* terminate */
+ return GNUNET_SYSERR; /* Done */
ve->copied = GNUNET_YES;
- return GNUNET_HELLO_add_address (ve->address, ve->valid_until, buf, max);
+ return GNUNET_HELLO_add_address (ve->address,
+ ve->valid_until,
+ buf,
+ max);
}
int sig_res;
int do_verify;
+ if (0 ==
+ memcmp (&GST_my_identity,
+ sender,
+ sizeof (struct GNUNET_PeerIdentity)))
+ return GNUNET_OK; /* our own, ignore! */
+
if (ntohs (hdr->size) < sizeof (struct TransportPongMessage))
{
GNUNET_break_op (0);
addr++;
slen = strlen (tname) + 1;
addrlen = size - slen;
+
+ if (NULL == GST_plugins_find (tname))
+ {
+ /* we got the PONG, but the transport plugin specified in it
+ is not supported by this peer, so this cannot be a good
+ PONG for us. */
+ GNUNET_break_op (0);
+ return GNUNET_OK;
+ }
+
address.peer = *sender;
address.address = addr;
address.address_length = addrlen;
address.transport_name = tname;
address.local_info = GNUNET_HELLO_ADDRESS_INFO_NONE;
- ve = find_validation_entry (NULL, &address);
+ ve = find_validation_entry (&address);
if ((NULL == ve) || (GNUNET_NO == ve->expecting_pong))
{
GNUNET_STATISTICS_update (GST_stats,
return GNUNET_OK;
}
/* now check that PONG is well-formed */
- if (0 != memcmp (&ve->pid, sender, sizeof (struct GNUNET_PeerIdentity)))
+ if (0 != memcmp (&ve->address->peer,
+ sender,
+ sizeof (struct GNUNET_PeerIdentity)))
{
GNUNET_break_op (0);
return GNUNET_SYSERR;
}
- if (GNUNET_TIME_absolute_get_remaining
- (GNUNET_TIME_absolute_ntoh (pong->expiration)).rel_value_us == 0)
+ if (0 ==
+ GNUNET_TIME_absolute_get_remaining
+ (GNUNET_TIME_absolute_ntoh (pong->expiration)).rel_value_us)
{
GNUNET_STATISTICS_update (GST_stats,
gettext_noop
sig_res = GNUNET_SYSERR;
do_verify = GNUNET_YES;
- if (0 != GNUNET_TIME_absolute_get_remaining(ve->pong_sig_valid_until).rel_value_us)
+ if (0 != GNUNET_TIME_absolute_get_remaining (ve->pong_sig_valid_until).rel_value_us)
{
/* We have a cached and valid signature for this peer,
* try to compare instead of verify */
- if (0 == memcmp (&ve->pong_sig_cache, &pong->signature, sizeof (struct GNUNET_CRYPTO_EddsaSignature)))
+ if (0 == memcmp (&ve->pong_sig_cache,
+ &pong->signature,
+ sizeof (struct GNUNET_CRYPTO_EddsaSignature)))
{
/* signatures are identical, we can skip verification */
sig_res = GNUNET_OK;
{
/* Do expensive verification */
sig_res = GNUNET_CRYPTO_eddsa_verify (GNUNET_SIGNATURE_PURPOSE_TRANSPORT_PONG_OWN,
- &pong->purpose, &pong->signature,
- &ve->public_key);
+ &pong->purpose,
+ &pong->signature,
+ &ve->address->peer.public_key);
if (sig_res == GNUNET_SYSERR)
{
GNUNET_break_op (0);
GNUNET_log (GNUNET_ERROR_TYPE_INFO,
"Validation process successful for peer `%s' with plugin `%s' address `%s'\n",
- GNUNET_i2s (sender), tname, GST_plugins_a2s (ve->address));
+ GNUNET_i2s (sender),
+ tname,
+ GST_plugins_a2s (ve->address));
+ GNUNET_STATISTICS_update (GST_stats,
+ gettext_noop ("# validations succeeded"),
+ 1,
+ GNUNET_NO);
/* validity achieved, remember it! */
ve->expecting_pong = GNUNET_NO;
ve->valid_until = GNUNET_TIME_relative_to_absolute (HELLO_ADDRESS_EXPIRATION);
ve->pong_sig_cache = pong->signature;
- ve->pong_sig_valid_until = GNUNET_TIME_absolute_ntoh (pong->expiration);
+ ve->pong_sig_valid_until = GNUNET_TIME_absolute_ntoh (pong->expiration);
ve->latency = GNUNET_TIME_absolute_get_duration (ve->send_time);
{
- struct GNUNET_ATS_Information ats[2];
-
- ats[0].type = htonl (GNUNET_ATS_QUALITY_NET_DELAY);
- ats[0].value = htonl ((uint32_t) ve->latency.rel_value_us);
- ats[1].type = htonl (GNUNET_ATS_NETWORK_TYPE);
- ats[1].value = htonl ((uint32_t) ve->network);
- GNUNET_ATS_address_add (GST_ats, ve->address, NULL, ats, 2);
+ if (GNUNET_YES == ve->known_to_ats)
+ {
+ GNUNET_assert (GNUNET_YES ==
+ GST_ats_is_known_no_session (ve->address));
+ GST_ats_update_delay (ve->address,
+ GNUNET_TIME_relative_divide (ve->latency, 2));
+ }
+ else
+ {
+ struct GNUNET_ATS_Properties prop;
+
+ memset (&prop, 0, sizeof (prop));
+ GNUNET_break (GNUNET_ATS_NET_UNSPECIFIED != ve->network);
+ prop.scope = ve->network;
+ prop.delay = GNUNET_TIME_relative_divide (ve->latency, 2);
+ GNUNET_assert (GNUNET_NO ==
+ GST_ats_is_known_no_session (ve->address));
+ ve->known_to_ats = GNUNET_YES;
+ GST_ats_add_address (ve->address, &prop);
+ GNUNET_assert (GNUNET_YES ==
+ GST_ats_is_known_no_session (ve->address));
+ }
}
if (validations_running > 0)
{
- validations_running --;
+ validations_running--;
+ GNUNET_STATISTICS_set (GST_stats,
+ gettext_noop ("# validations running"),
+ validations_running,
+ GNUNET_NO);
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
"Validation finished, %u validation processes running\n",
validations_running);
}
/* Notify about new validity */
- validation_entry_changed (ve, GNUNET_TRANSPORT_VS_UPDATE);
+ validation_entry_changed (ve,
+ GNUNET_TRANSPORT_VS_UPDATE);
/* build HELLO to store in PEERINFO */
+ GNUNET_STATISTICS_update (GST_stats,
+ gettext_noop ("# HELLOs given to peerinfo"),
+ 1,
+ GNUNET_NO);
ve->copied = GNUNET_NO;
- hello = GNUNET_HELLO_create (&ve->public_key,
- &add_valid_peer_address, ve,
+ hello = GNUNET_HELLO_create (&ve->address->peer.public_key,
+ &add_valid_peer_address,
+ ve,
GNUNET_NO);
- GNUNET_PEERINFO_add_peer (GST_peerinfo, hello, NULL, NULL);
+ GNUNET_break (NULL !=
+ GNUNET_PEERINFO_add_peer (GST_peerinfo,
+ hello,
+ NULL,
+ NULL));
GNUNET_free (hello);
return GNUNET_OK;
}
{
const struct GNUNET_HELLO_Message *hm =
(const struct GNUNET_HELLO_Message *) hello;
- struct ValidateAddressContext vac;
- struct GNUNET_HELLO_Message *h;
+ struct GNUNET_PeerIdentity pid;
int friend;
friend = GNUNET_HELLO_is_friend_only (hm);
if ( ( (GNUNET_YES != friend) &&
(GNUNET_NO != friend) ) ||
- (GNUNET_OK != GNUNET_HELLO_get_id (hm, &vac.pid)) ||
- (GNUNET_OK != GNUNET_HELLO_get_key (hm, &vac.public_key)))
+ (GNUNET_OK != GNUNET_HELLO_get_id (hm, &pid)) )
{
/* malformed HELLO */
GNUNET_break_op (0);
return GNUNET_SYSERR;
}
if (0 ==
- memcmp (&GST_my_identity, &vac.pid, sizeof (struct GNUNET_PeerIdentity)))
+ memcmp (&GST_my_identity,
+ &pid,
+ sizeof (struct GNUNET_PeerIdentity)))
+ {
+ /* got our own HELLO, how boring */
+ GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+ "Validation received our own HELLO (%s), ignoring\n",
+ GNUNET_i2s (&pid));
return GNUNET_OK;
- /* Add peer identity without addresses to peerinfo service */
- h = GNUNET_HELLO_create (&vac.public_key, NULL, NULL, friend);
- GNUNET_log (GNUNET_ERROR_TYPE_INFO,
- _("Validation received new %s message for peer `%s' with size %u\n"),
- "HELLO",
- GNUNET_i2s (&vac.pid),
- ntohs (hello->size));
- GNUNET_PEERINFO_add_peer (GST_peerinfo, h, NULL, NULL);
-
+ }
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
- _("Adding `%s' without addresses for peer `%s'\n"), "HELLO",
- GNUNET_i2s (&vac.pid));
-
- GNUNET_free (h);
+ "Validation received HELLO message for peer `%s' with size %u, checking for new addresses\n",
+ GNUNET_i2s (&pid),
+ ntohs (hello->size));
GNUNET_assert (NULL ==
- GNUNET_HELLO_iterate_addresses (hm, GNUNET_NO,
+ GNUNET_HELLO_iterate_addresses (hm,
+ GNUNET_NO,
&validate_address_iterator,
- &vac));
+ NULL));
return GNUNET_OK;
}
* @return #GNUNET_OK (continue to iterate)
*/
static int
-iterate_addresses (void *cls, const struct GNUNET_PeerIdentity *key, void *value)
+iterate_addresses (void *cls,
+ const struct GNUNET_PeerIdentity *key,
+ void *value)
{
struct IteratorContext *ic = cls;
struct ValidationEntry *ve = value;
- ic->cb (ic->cb_cls, &ve->public_key, ve->valid_until, ve->revalidation_block,
+ ic->cb (ic->cb_cls,
+ ve->valid_until,
+ ve->revalidation_block,
ve->address);
return GNUNET_OK;
}
*
* @param target peer information is requested for
* @param cb function to call; will not be called after this function returns
- * @param cb_cls closure for 'cb'
+ * @param cb_cls closure for @a cb
*/
void
GST_validation_get_addresses (const struct GNUNET_PeerIdentity *target,
- GST_ValidationAddressCallback cb, void *cb_cls)
+ GST_ValidationAddressCallback cb,
+ void *cb_cls)
{
struct IteratorContext ic;
* Based on this, the validation module will measure latency for the
* address more or less often.
*
- * @param address the address
- * @param session the session
+ * @param address the address that we are now using (or not)
* @param in_use #GNUNET_YES if we are now using the address for a connection,
* #GNUNET_NO if we are no longer using the address for a connection
*/
void
GST_validation_set_address_use (const struct GNUNET_HELLO_Address *address,
- struct Session *session,
int in_use)
{
struct ValidationEntry *ve;
- if (NULL != address)
- ve = find_validation_entry (NULL, address);
- else
- ve = NULL; /* FIXME: lookup based on session... */
- if (NULL == ve)
+ if (GNUNET_HELLO_address_check_option (address,
+ GNUNET_HELLO_ADDRESS_INFO_INBOUND))
+ return; /* ignore inbound for validation */
+ if (NULL == GST_plugins_find (address->transport_name))
{
- /* this can happen for inbound connections (sender_address_len == 0); */
+ /* How can we use an address for which we don't have the plugin? */
+ GNUNET_break (0);
return;
}
- if (ve->in_use == in_use)
+ ve = find_validation_entry (address);
+ if (NULL == ve)
{
-
- if (GNUNET_YES == in_use)
- {
- GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
- "Error setting address in use for peer `%s' `%s' to USED\n",
- GNUNET_i2s (&address->peer), GST_plugins_a2s (address));
- }
- if (GNUNET_NO == in_use)
- {
- GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
- "Error setting address in use for peer `%s' `%s' to NOT_USED\n",
- GNUNET_i2s (&address->peer), GST_plugins_a2s (address));
- }
+ GNUNET_break (0);
+ return;
}
-
- GNUNET_break (ve->in_use != in_use); /* should be different... */
+ if (in_use == ve->in_use)
+ return;
ve->in_use = in_use;
- if (in_use == GNUNET_YES)
+ if (GNUNET_YES == in_use)
{
/* from now on, higher frequeny, so reschedule now */
- GNUNET_SCHEDULER_cancel (ve->revalidation_task);
- ve->revalidation_task = GNUNET_SCHEDULER_add_now (&revalidate_address, ve);
- }
-}
-
-
-/**
- * Query validation about the latest observed latency on a given
- * address.
- *
- * @param sender peer
- * @param address the address
- * @param session session
- * @return observed latency of the address, FOREVER if the address was
- * never successfully validated
- */
-struct GNUNET_TIME_Relative
-GST_validation_get_address_latency (const struct GNUNET_PeerIdentity *sender,
- const struct GNUNET_HELLO_Address *address,
- struct Session *session)
-{
- struct ValidationEntry *ve;
-
- if (NULL == address)
- {
- GNUNET_break (0); // FIXME: support having latency only with session...
- return GNUNET_TIME_UNIT_FOREVER_REL;
+ if (NULL != ve->revalidation_task)
+ GNUNET_SCHEDULER_cancel (ve->revalidation_task);
+ ve->revalidation_task = GNUNET_SCHEDULER_add_now (&revalidate_address,
+ ve);
}
- ve = find_validation_entry (NULL, address);
- if (NULL == ve)
- return GNUNET_TIME_UNIT_FOREVER_REL;
- return ve->latency;
-}
-
-/**
- * Closure for the validation_entries_iterate function.
- */
-struct ValidationIteratorContext
-{
- /**
- * Function to call on each validation entry
- */
- GST_ValidationChangedCallback cb;
-
- /**
- * Closure for 'cb'.
- */
- void *cb_cls;
-};
-
-static int
-validation_entries_iterate (void *cls,
- const struct GNUNET_PeerIdentity *key,
- void *value)
-{
- struct ValidationIteratorContext *ic = cls;
- struct ValidationEntry *ve = value;
-
- GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Notifying about validation entry for peer `%s' address `%s' \n",
- GNUNET_i2s (&ve->pid), GST_plugins_a2s (ve->address));
- ic->cb (ic->cb_cls, &ve->pid, ve->address, ve->send_time,
- ve->valid_until, ve->next_validation, ve->state);
-
- return GNUNET_OK;
}
-/**
- * Iterate over all iteration entries
- *
- * @param cb function to call
- * @param cb_cls closure for cb
- */
-void
-GST_validation_iterate (GST_ValidationChangedCallback cb, void *cb_cls)
-{
- struct ValidationIteratorContext ic;
-
- if (NULL == validation_map)
- return; /* can happen during shutdown */
- ic.cb = cb;
- ic.cb_cls = cb_cls;
- GNUNET_CONTAINER_multipeermap_iterate (validation_map, &validation_entries_iterate, &ic);
-}
/* end of file gnunet-service-transport_validation.c */