Merge #1357

[2218] Combine blocks

[2218] Add recursive findNSEC3() tests

[2218] Remove duplicate copies of NSEC3 data

[2218] Add trace when walking the tree

[1357] Test nothing is committed on error

[2218] Move NSEC3 hash calculation to NSEC3Hash class

[2218] Test findNSEC3() by walking the tree

[2218] Change DomainTreeNodeChain::getLevelCount()'s return type to size_t

... so that it's unsigned. Also rename node_count_ to level_count_
to avoid confusion with DomainTree::node_count_.

[2218] Change DomainTree::getNodeCount()'s return type to size_t

... so that it's unsigned.

[2218] Fix comment

[2218] Check that the NSEC3 tree has nodes before performing findNSEC3()

[2218] Assert that ZoneNodes in NSEC3 tree contain NSEC3 RRType

[2218] Don't upcase the first label when inserting into NSEC3 tree

This is no longer required as tree.find() works without it.

[2218] Simplify code, also shortcutting RRset construction

[2218] Rename set variable to rdataset

[2218] Rewrite Name generation to be more efficient

[2218] Move some statements out of loop

[2218] Move expression to simplify code

[2219] workaround for use-after-destroy cases in python client-list tests.

by making sure cache client/finder are destroyed at the end of each test.
we may want to improve the reliability, but for now that should be sufficient.

[2219] move memory leak check from in-memory client destructor to client list.

now the memory segment is shared for multiple in-memory zones, we cannot check
it in each client.

[2219] editorial: move DataSourceInfo::getCacheClient closer to other methods

[2219] workaround for letting auth test refer to in-memory data source client

this is a hack, and we should eventually make it more cleanly.

[2219] updated ZoneFinderContextTest so it will use the new in-mem datasrc.

[2219] re-added findAll() version of finder context constructor.

since we don't yet support in-memory optimization for addtional section
processing, we need to have the other version at the moment.

[2219] regression fix: support glue_ok option at the zone cut in the main find

in the old implementation we did it for the specialized finder context, but
we are currently using the generic context, so we need to handle the case
in the main method.  (and, technically, this case should be handled in the
main method even if we had a specialized context as it's an API contract.)

[2219] style fix: add () after return

[2219] bug fix: store real object of rrclass in memory zone finder, not a ref.

having it as a reference can easily cause a dangling reference (and it actually
happens in some tests).
we could probably have it as a reference without causing disruption if we
carefully, but that doesn't seem to be reliable.
copying RRclass should be cheap, so it should be okay in this case.

[2219] a bug fix to zone finder: make RRSIG visible only when DNSSEC is sought.

[2219] added unit tests TreeNodeRRset::getRRsig().

not directly related to this branch, but it should have been done when the
method was implemented.
Also made small optimization: just return NULL if it's not supposed to be
signed.

[2219] updated client list test so it works with new in-memory implementation.

the points of the test should be preserved, but some internal behaviors
have to be adjusted.

[master] cleanup: removed dead code.

I suspect it was a leftover of an intermediate attempt that ended up with
    found = RdataSet::find(node->getData(), type);
at commi 32f6a873.

Whatever the intent was, it's clear the code does nothing, so cleaning it
up shouldn't harm anyway.

okayed on jabber.

[master] Merge branch 'trac2267'

[2267] cleanup: removed a redundant 'return'

[2269] Initial subnet6 definitions for DHCPv6

[2272] Replaced malloc with new in command options helper.

[1357] Create only one Diff

It failed to commit the previous changes, as it created new Diff and the
old was just forgotten.

[2272] Fixed typo in HAVE_OPTRESET in configure.ac.

[2272] Check for HAVE_OPTRESET in the perfdhcp command options.

[1357] Commit only after verifying the last TSIG

We can't commit during the stream of changes, they might be not signed
and that'd be a security risk (if someone stole the TCP connection and
pushed a change that'd put bogus data inside before getting to the last
message which would be rejected then).

Fixed the check: it now happens on the last message, not on each message
except the last. Returning true there does not mean to continue, but
that the current RR has been handled fully.

[2272] Check for optreset declaration in unistd.h.

[2222] removed unnecessary tests from TestXfroutSession

`None` is not set to a counter handler as a default. These tests are
no longer effective. This change is due to a merge of #2158.

[2222] used find() instead of `in` operator

 - used find() instead of `in` operator for exact pattern matching of string
 - a style fix in the list

[2222] added 'ixfr_running' and 'axfr_running' descriptions into the manpage of b10-xfrout, and untabfy

[2222] added checking for the number of xfr_running and axfr_running

 - added checking for the number of xfr_running and axfr_running

 - added changes related to the previous changes of bind10_control.py
   (the step method name and the 'between A and B' style)

[2222] updated lettuce step methods for statisitcs

 - revised statistics tests to use 'between A and B' style

 - revised the name of the step method for checking the statistics value

 - updated copyright

[2222] added "axfr_running" and "ixfr_running" definitions for statistics into xfrout.spec

[2222] added tests for counting the number of AXFR/IXFR running

 - added tests which check properly counting the number of AXFR/IXFR
   running

 - added changes related to the changes of XfroutCounter

 - added misc refactoring for testing

 - used validate_statistics() for checking statistics data format to be
   returned to the stats module

 - updated copyright

[2222] updated classes: XfroutCounter, XfroutServer and XfroutSession

 - added incrementer/decrementer for the number of IXFR/AXFR running into
   the
   XfroutCounter class

 - added getter methods of counter handlers for the XfroutSession/NotifyOut
   class into the XfroutCounter class

 - inserted incrementer/decrementer handler for the number of AXFR/IXFR
   running into XfroutSession

 - removed the unnecessary thread lock from the get_statistics()

 - removed a commented-out code

 - updated copyright

[stats_lettuce_fix] added configuration of stats

Changed the default time (60sec) of poll-interval to one second.  This is
because it's so infrequent for lettuce test of stats.

[stats_lettuce_fix] fix for the lettuce error: Not found statistics counter notifyoutv4

http://git.bind10.isc.org/~tester/builder//BIND10-lettuce/20120918121701-DebianLinux-x86_64-GCC/logs/lettuce.out
  AssertionError: Not found statistics counter notifyoutv4 for zone example.org.

Added waiting until the last requesting is finished instead of one-second
sleep. It should be done before checking the result of "Stats show" command via
bindctl.

[2218] editorial fixes: folded long lines, constify

Revert "[stats_lettuce_fix] fix for the lettuce error: Not found statistics counter notifyoutv4"

This reverts commit 14727f8945873937f353915eac0e918cdb58721f.

[stats_lettuce_fix] fix for the lettuce error: AssertionError: Got 3, expected less than 3 as counter xfrrej

http://git.bind10.isc.org/~tester/builder/BIND10-lettuce/20120918210000-MacOS/logs/lettuce.out
  AssertionError: Got 3, expected less than 3 as counter xfrrej for zone _SERVER_

Removed the upper limits of the counter xfrrej. Because a xfer request might be
rejected greater than 3 times. This number of times might depend on something
like the environment or the platform or the rasing condition, where this
lettuce test is running.

[stats_lettuce_fix] fix for the lettuce error: Not found statistics counter notifyoutv4

http://git.bind10.isc.org/~tester/builder//BIND10-lettuce/20120918121701-DebianLinux-x86_64-GCC/logs/lettuce.out
  AssertionError: Not found statistics counter notifyoutv4 for zone example.org.

It has to wait at least for one second since the query to
b10-stats. b10-stats doesn't request to other module, e.g. b10-xfrout,
for less than one second, even if show command is invoked via
bindctl. This is for avoiding too frequent requests to b10-stats.

[2219] use the new version in-memory data source client in client_list.

it compiles, but some tests currently fail.

[2219] Merge branch 'trac2218' into trac2219

[2219] Merge branch 'trac2267' into trac2219

[master] be sure to wait for getstats command in xfrin notify lettuce test.

as proposed by naokikambe.  confirmed by myself.  okayed on jabber.

[2267] unify the call to addRdataSet.

as suggested in review.

[2267] comment updates

[2238] Circular dependency (asiolink->log->util->asiolink) fixed.

[master] Update various .gitignore files

[master] Track origin of uninitalized memory on all Valgrind targets

[master] Move GTEST_LDADD to the bottom of the list

[master] Cleanup a Makefile.am (which didn't link to libb10-dhcp++)

Also move GTEST_LDADD to the bottom of the list of LDADDs.

[2190] do normal shutdown on msgq in statshttpd test

[2218] Add test for DomainTreeNode::getLargestInSubTree()

[2218] Add test for DomainTreeNode::getSubTreeRoot()

[2218] Work inside the subtree under the origin's node only

[2218] Add DomainTreeNode::getLargestInSubTree() method

This is used to find the largest (covering) node under the origin's
subtree in the NSEC3 tree.

[2218] Remove DomainTree::getLargestNode()

... which was broken anyway.

[2218] Add DomainTreeNode::getSubTreeRoot()

[2218] Make DomainTreeNode::predecessor() and ::successor() public

[2218] Make DomainNode::isSubTreeRoot() a public function

This will be required for finding whether a hash is less than,
or greater than all nodes in the origin's subtree in an NSEC3 tree.

[2218] Insert using the NSEC3 record's entire name into the NSEC3Data tree

[2267] one more comment cleanup: updated the comment about setSigned().

(the old comment doesn't make sense for the new interface).

[2218] Check for NULL covering_node

[2218] Use the class RdataEncoder

[2267] (unrelated) comment (and some style) cleanups.

some comments were obsolete in the newer implementation.  they were updated
or simply removed.

[2267] cleanup: make retval of impl::add() void.

in the new implementation it always returns SUCCESS anyway, so it doesn't make
sense to return it and check the value.

[2267] cleanup: pass zone name and data at the loader construction.

[2267] catch and reject the bogus case of passing empty RRSIG

[2267] some comments

[2267] catch and reject RRSIGs that don't have a covered RRset.

the loadRRSIGFollowsNothing test now passes and is re-enabled.
the 'name unmatched' and 'type unmatched' tests are essentially the same
so are removed (so are their test zone files).

[2267] cover the case separated RRSIGs of the same type are given.

[2267] enabled the commented RRSIG test data, with which it now passes.

[2267] cleanup: removed last_rrset_. we don't need this beast any more.

[2267] cleanup: removed now-unused methods

and renamed addRdataSet2 to addRdataSet now that we don't have to distingush
these two.

[2267] temporarily disabled some tests, which should expect a different result.

[2267] add RRsets to zone via the new loader class, ensuring the pairing.

this fixes the main issue of this ticket, but made some other tests fail
(which will be addressed in later commits)

[2267] refactoring: introduce a helper Loader class and move addFromLoad there.

[2267] have add() take both RRset and RRSIG at the same time.

as a helper another version of addRdataSet() is introduced too.
these don't need the last_rrset_ magic.
InMemoryClient::add now uses this version.

[2267] added a test that checks if load() with separate_rr iterator fails.

to catch any later regression.

[2267] unrelated cleanup: use MemorySegmentTest for instrumental mem segment

instead of a local copy of it inside memory_client_unittest.

[2267] added RRSIGs to test iterator for memory client tests

with a commented-out one that would reveal a bug of the implementation.
also did some trivial style fixes.

[2218] Implement TreeNodeRRset::getRRsig()

[2218] Add facility to setup a fake NSEC3Calculate function for use in tests

[master]Merge branch 'master' of ssh://git.bind10.isc.org/var/bind10/git/bind10

[master] fix spelling typo in output

I didn't get this reviewed, but make check still worked for me.