[2298_7] Merge branch 'trac2298_7'

[2298_1] Merge branch 'trac2298_1'

[trac2305] various minor documentation changes

add some docbook formatting tags

minor formatting changes.

bindctl -a defaults to 127.0.0.1 not localhost

add a space between "BIND 10"

some rewording

a few punctuation changes

add a sentence to describe an example

[2372] use specific constatnt values for Options instead of using the << trick

the previous code doesn't seem to be understandable as I thought, so I
simplfy revert to the more straightforward way.
also added doc for the enum

[2372] catch a buggy case of bogus ID in State::getInstance

[2372] made sure entire test sstring is built before starting test case.

The previous code should actually be safe based on the iostream/streambuf
APIs, but it's even much safer to completely give the ownership of the
passed stream to the input source.

[master] ChangeLog for trac2342

[2372] hide specific state cleasse inside an unnamed namespace.

they don't have to be (actually they already aren't) visible outside this
translation unit.

[2372] a minor documentation wording correction.

[2372] minor editorial cleanup in a comment

[2342] Merge branch 'master' into trac2342

[2342] Miscellaneous changes as a result of review

Make some constants unsigned explicitly

This should fix compilation, where it complains:

rrttl_unittest.cc:79: warning: this decimal constant is unsigned only in ISO C90

Changelog for #2384

Merge #2384

[2342] Merge branch 'master' into trac2342

Conflicts:
doc/devel/mainpage.dox
src/lib/dhcp/Makefile.am
src/lib/dhcp/lease_mgr.cc
src/lib/dhcp/lease_mgr.h
src/lib/dhcp/tests/Makefile.am
src/lib/dhcp/tests/lease_mgr_unittest.cc

A number of other files were modified to resolve incompatibilities
in the way lease managers were created, and to promote the dummy
"memfile" lease manager to a fully-fledged lease manager with
unit tests.

[2384] Little bit more range checking

There was a gap when one thing could overflow to negative numbers.

[2471] make sure rdata iterator is valid in DNAME proc for sufficiently long

otherwise it could result in using a freed object if the RRset comes from
the built in in-memory data source.  This change makes the test in the
previous commit pass.

[2471] added a test cast for DNAME query proc using real in-memory datasrc.

this reproduces the problem of this ticket.

[2384] style fix: folded a long line

[master] ChangeLog for trac2414

[2414] Merge branch 'master' into trac2414

[2342] Added comments as result of review

[2414] Changes after review

[2342] Minor: typos fixed.

[2342] Temporarily moved duid.{h,cc} from libdhcp++ to libdhcp_srv.

Having duid.{h, cc} in libdhcp++ causes dependency between two libraries.
Due to this dependency multi-threaded build fails because libs are created
by different threads simultaneously and race condition occurs. Libraries
should be moved to separate folders and have separate makefiles. This
complies with BIND10 structure and will make this issue goe away.

[2376] Documentation update

[2376] Disallow missing last unit

Also, simplify the code little bit and make it run faster in the case of
TTL without any units.

[2376] Disallow case with the last unit missing

[2464] fix call to last_had_signature()

[2464] Add lettuce test for TSIG

Should fail at this time

[2353] Test verbose case of BoB.start_simple()

[2353] runnable doesn't have to be set here

[2353] Directly use bob.components instead of bob.get_processes()

[2353] Just check the output of get_processes() is empty (as a boolean)

[2353] Check that kill_started_components() forcefully kills components

[2353] runnable doesn't have to be set here

[2353] runnable doesn't have to be set here

[2353] Ensure that get_processes() returns a sorted list

[2353] Check remove_socket_srv() with different states of BoB._srv_socket

[2353] Ensure that the server listens on the socket

[2353] Check that getsockname() on the socket returns what we expect

[2353] Test that the socket and tmpdir are deleted

[master] Merge branch 'trac2447'

[2414] Merge branch 'master' into trac2414

[2414] Remove "using namespace boost"

For some reason, on Solaris this causes types "uintXX_t" to become
undefined.  The "using" statement was removed and an explicit
namespace prepended to declarations of shared_ptr and dynamic_pointer_cast.

[2414] Get rid of memory leak

The allocation engine is now pointed to by a shared pointer.  Previously
it was pointed toi by a raw pointer and not deleted in the destructor.
Also, use an automatic variable to hold the DHCP6 server in the
unit tests.

[2384] Remove obsolete comment

[2384] Test leading zeroes in TTL

[2384] Simplify range check

It doesn't need to check the number of digits. It seems the tests pass
now without problems as well. Also, it might be problematic with large
number of leading zeroes.

Also, simplify comment with description.

[2447] corrected a log name

[2463] Minor: typo fixed.

[2463] Minor changes in comments.

[master] split up test

From http://bind10.isc.org/ticket/2468#comment:2 to help troubleshoot later.

Okayed via jabber.

[2342] More changes as a result of review

a) Corrected some method headers
b) Added additional diagnostic output should the test fail to
   open the database.

[2463] Added new constructor to OptionDescriptor structure.

[2463] Reset configuration after each unit test.

[2342] Encapsulate mysql_stmt_free_result in a temporary object

... to ensure that it is always called when a method terminates.

[2342] Documentation changes as result of review

[2447] fix small typo in docstring

[2447] dump a log message if connecting to cc-channel fails.

and suggest checking output from msgq using -v.

[2447] improve intitial failure handling for msgq, gracefully and with logs

[2447] show the reason of the error when opening IPC lock file fails.

[2447] use helper __kill_children method to kill all initial processes

previously it could result in exception (by the attempt of killing sockcreator
as a chuser'ed process) during initial setup failure, leading to unclean
shutdown.  the revised code prevents it from happening, and also makes the
code concise by unifying the common logic.

[2447] catch failure on initial log, which is most likely a permission issue.

so, it now just prints some hints to stderr and exits.

[2447] improved detailed description of CONFIG_LOAD_FAIL ref'ing sock failure.

it even suggests restarting as a root.

[2447] improved exception message on sockcreator failure to help admin.

it now includes textual representation of errno, and, if it's EACCES
("permission denied") the messsage suggesting restarting as a super user.

[master] check -lnsl, -lsocket for some network API libs.

this is necessary for Solaris. okayed on jabber.

[2414] Compilation fix for Solaris 11

[2414] Compilation fix for FreeBSD

[master] Merge branch 'trac2449'

[master] Merge branch 'trac2371'

[2414] Removed hardcoded DNS_SERVERS option

DNS_SERVERS option is now sent only if requested by clients
 (as it should be)

[2414] solicitBasic1 test fixed and renamed to advertiseOptions

[2421] Handle exceptions when loading a zone

[2421] Add a testcase for empty zone too (which should throw EmptyZone)

[2421] Change test data to throw ZoneDataUpdater::AddError

... instead of MasterLoadError.

[2421] Add lettuce test with broken zone

[2384] editorial in exception message: add a space for readability

[2463] Preserve global configuration values.

Rather tham clear global configuration values we replace them (override)
in subsequent configuration commits. This guarantees that global
values are always available when subnet configuration is performed.

[2414] Merge branch 'master' into trac2414

Conflicts:
src/bin/dhcp6/dhcp6_messages.mes
src/bin/dhcp6/dhcp6_srv.cc
src/bin/dhcp6/dhcp6_srv.h
src/bin/dhcp6/tests/dhcp6_srv_unittest.cc
src/lib/dhcp/addr_utilities.cc
src/lib/dhcp/tests/cfgmgr_unittest.cc

Files automerged successfully:
    src/lib/dhcp/alloc_engine.cc
    src/lib/dhcp/subnet.h
    src/lib/dhcp/tests/alloc_engine_unittest.cc

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

[2305] address review comments

updated documentation of bindctl chapter
fixed a stray space in module help output in bindctl

[2384] Document the use of units in TTL

[2384] Test border cases

[2384] Rearrange tests

There were some non-fromText related tests in the fromText test case.
Moving to separate test case.

[2384] Better TTL range checking

Also check partial values and wrap-arouds for numbers.

[master] Merge branch 'trac2417'

[2384] Test case of reused unit

[2384] Remove unnecessary cast

[2384] Show the broken TTL in exception messages

[2384] Disallow empty TTLs and units-only TTLs

These are questionable and bind9 does not support them. Even the units
are bind extension of some kind probably.

Merge branch 'master' into trac2369

[2369] Use prefix syntax for increment/decrement operators

[2459] do not throw when getZoneWriter result in "not cached".

instead, explicitly log the event at the debug level.  behavior-wise there's
no difference, but the corresponding log message can be misleading.

[2459] protect the call to getCachedZoneWriter() by mutex. it can cause race.

[2353] Test bind10_src.fatal_signal()

[2353] Test bind10_src.get_signame()

[2353] Test BoB.init_socket_srv() and BoB.remove_socket_srv()

[2353] Test more component startup methods