View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0001658GNUnetDHT servicepublic2011-02-04 23:182011-09-19 10:45
Reportertoelke Assigned Tomrwiggles  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Summary0001658: Block retrieved from DHT do not have correct size
Description
When putting a block of size 616:

Feb 04 23:14:21 dns-13773 DEBUG Putting with key 798e8cf8, size = 616

A block of size 620 is retrieved later:
Feb 04 23:14:59 dht-13771 DEBUG DNS-Block is invalid: reply_block_size=620 != 616

Regards,
Philipp T
TagsNo tags attached.

Activities

mrwiggles

2011-02-04 23:29

reporter   ~0004246

Is there a test case I can use to reproduce this behavior?

toelke

2011-02-04 23:37

reporter   ~0004247

Last edited: 2011-02-04 23:37

 Not yet.

In the meantime, please take a look at the function "publish_name" in vpn/gnunet-service-dns.c, perhaps I am doing wrong something obvious.

toelke

2011-02-05 14:14

reporter   ~0004248

Works now, thanks!

mrwiggles

2011-02-05 14:25

reporter   ~0004249

Changed internal storage of data (thanks to path tracking requirements) and needed to either 1) alter the block libraries or 2) provide the original, unaltered data to the block plugins. Chose 2, though this doesn't leave an option for validating paths in the block library, which may be desired in the future.

toelke

2011-02-05 14:37

reporter   ~0004250

I now do not get an error, but also no data. The test-case fails with DNS set to GNUNET_YES:

PASS: test_dht_api
TEST: Generating hostkey...PASS
TEST: Starting ARM...PASS
TEST: Testing put...PASS
TEST: Testing get...PASS
PASS: test_dht_tools.sh
Feb 05 14:35:14 test-dht-twopeer-27783 WARNING Found file size 3481426 for hostkeys, expect hostkeys to be size 914
Feb 05 14:35:14 test-dht-twopeer-27783 WARNING Will read 3809 hostkeys from file
PASS: test_dht_twopeer
Feb 05 14:35:19 test-dht-twopeer-put-get-27873 WARNING Found file size 3481426 for hostkeys, expect hostkeys to be size 914
Feb 05 14:35:19 test-dht-twopeer-put-get-27873 WARNING Will read 3809 hostkeys from file
FAIL: test_dht_twopeer_put_get
Feb 05 14:35:34 test-dht-twopeer-put-get-28081 WARNING Found file size 3481426 for hostkeys, expect hostkeys to be size 914
Feb 05 14:35:34 test-dht-twopeer-put-get-28081 WARNING Will read 3809 hostkeys from file
PASS: test_dht_twopeer_path_tracking
Feb 05 14:35:49 test-dht-multipeer-28287 WARNING Found file size 3481426 for hostkeys, expect hostkeys to be size 914
Feb 05 14:35:49 test-dht-multipeer-28287 WARNING Will read 3809 hostkeys from file
Feb 05 14:35:49 test-dht-multipeer-28287 WARNING Read 5 total peers in topology
PUT Path: FC74
GET Path: KJI3->FC74
PASS: test_dht_multipeer
PASS: test_dhtlog
===================================
1 of 7 tests failed
Please report to bug-gnunet@gnu.org
===================================

mrwiggles

2011-02-05 14:57

reporter   ~0004251

Should be fixed in revision 14355. I was getting data, but comparing the key/data pair to the non-DNS version of the testcase.

toelke

2011-02-05 15:06

reporter   ~0004252

Now it really works (test-case and my code)

mrwiggles

2011-02-05 15:07

reporter   ~0004253

Test case also fixed (:

Issue History

Date Modified Username Field Change
2011-02-04 23:18 toelke New Issue
2011-02-04 23:18 toelke Status new => assigned
2011-02-04 23:18 toelke Assigned To => mrwiggles
2011-02-04 23:29 mrwiggles Note Added: 0004246
2011-02-04 23:37 toelke Note Added: 0004247
2011-02-04 23:37 toelke Note Edited: 0004247
2011-02-05 14:14 toelke Note Added: 0004248
2011-02-05 14:25 mrwiggles Note Added: 0004249
2011-02-05 14:25 mrwiggles Status assigned => closed
2011-02-05 14:25 mrwiggles Resolution open => fixed
2011-02-05 14:25 mrwiggles Fixed in Version => Git master
2011-02-05 14:37 toelke Note Added: 0004250
2011-02-05 14:37 toelke Status closed => feedback
2011-02-05 14:37 toelke Resolution fixed => reopened
2011-02-05 14:57 mrwiggles Note Added: 0004251
2011-02-05 15:06 toelke Note Added: 0004252
2011-02-05 15:07 mrwiggles Note Added: 0004253
2011-02-05 15:07 mrwiggles Status feedback => closed
2011-02-05 15:07 mrwiggles Resolution reopened => fixed
2011-09-19 10:45 Christian Grothoff Fixed in Version Git master => 0.9.0pre3
2011-09-19 10:45 Christian Grothoff Target Version => 0.9.0pre3