Adding alternate dests to private address book fails on blank hostsdb.blockfile

[Zlatin Balevsky]

Hi,

Adding alternate locations to the private address book doesn't work on blank address books. It works after a restart.

Steps to reproduce:

1. Delete hostsdb.blockfile
2. Start router
3. Add an entry in private address book
4. Try to add alternate destinations for same host

Reason:
When initializing a new block file, BlockFileNamingService doesn't add the "privatehosts.txt" SkipList to the "_lists" variable.

[zzz]

hmm. In initNew() line 245 we do set the list variable to HostsTxtNamingService?.DEFAULT_HOSTS_FILE which is "privatehosts.txt,userhosts.txt,hosts.txt" and then add each to the _lists List at the bottom of the loop. Also, the string is put in the info property, which is put in the header skiplist.

One quirk of the underlying blockfile/skiplist implementation is that empty skiplists are deleted. That's why in e.g. addEntry() line 528 it must be checked and created if absent.

I don't yet see where the problem is exactly, but I haven't tried to follow your steps to reproduce yet. For further research.

[Zlatin Balevsky]

On a blank router without a privatehosts.txt file, the checks on line 253 are going to short-circuit the addition to "_lists" which happens on line 291

[zzz]

Thanks, in 57cdeb15d186bfaf6d8683f11c653bc82b907abf to be 0.9.32-4