View Issue Details

IDProjectCategoryView StatusLast Update
0002480GNUnethello librarypublic2012-11-05 17:34
ReporterMatthias Wachs Assigned ToMatthias Wachs Due Date
PriorityimmediateSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product VersionGit master 
Target Version0.9.4 
Summary0002480: GNUNET_HELLO_address_cmp does not compare correctly
Descriptiongnunet-service-transport_validation:

validation_entry_match (void *cls, const struct GNUNET_HashCode * key, void *value)
{
  struct ValidationEntryMatchContext *vemc = cls;
  struct ValidationEntry *ve = value;

  if (0 == GNUNET_HELLO_address_cmp (ve->address, vemc->address))
  {
    char * ve_addr = strdup (GST_plugins_a2s(ve->address));
    GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
                "Addresses are equal: `%s' address `%s' == `%s'\n",
                GNUNET_i2s (&ve->address->peer),
                ve_addr,
                GST_plugins_a2s(vemc->address));
    GNUNET_free (ve_addr);
    vemc->ve = ve;
    return GNUNET_NO;
  }
  return GNUNET_YES;
}

Result:
Jul 04 11:26:03-192278 transport-5155 ERROR Addresses are equal: `9P6V' address `13:22:33:44:A9:67' == `4E:4D:F4:F2:9D:12'
TagsNo tags attached.

Relationships

related to 0002472 closedMatthias Wachs WLAN testcases fail: peers got not connected 

Activities

Matthias Wachs

2012-07-04 09:37

reporter   ~0006206

If address length for both addresses were equal, it did that:

return memcmp (a1->address, a1->address, a1->address_length);

obvious...

fixed in 22472.

Issue History

Date ModifiedUsernameFieldChange
2012-07-04 09:28Matthias WachsNew Issue
2012-07-04 09:28Matthias WachsStatusnew => assigned
2012-07-04 09:28Matthias WachsAssigned To => Matthias Wachs
2012-07-04 09:31Matthias WachsRelationship addedrelated to 0002472
2012-07-04 09:37Matthias WachsNote Added: 0006206
2012-07-04 09:38Matthias WachsStatusassigned => resolved
2012-07-04 09:38Matthias WachsResolutionopen => fixed
2012-07-04 09:38Matthias WachsTarget Version => 0.9.4
2012-11-05 17:34Christian GrothoffStatusresolved => closed