[master] many misspelling and typo fixes

reviewed by jelte via jabber.

Changes some log message descriptions and some
function arguments and some output.

doc/design/cc-protocol.txt

@@ -261,8 +261,8 @@ Group Subscriptions (type "subscribe")
 --------------------------------------
 
 A subscription requires the "group", "instance", and a flag to
-indicate the subscription type ("sybtype").  If instance is "*" the
-instance name will be ignored when decising to forward a message to
+indicate the subscription type ("subtype").  If instance is "*" the
+instance name will be ignored when deciding to forward a message to
 this client or not.
 
 "subtype" is a DATA element, and contains "normal" for normal channel

doc/devel/01-dns.dox

@@ -8,7 +8,7 @@
  *
  * @section b10-cfgmgr b10-cfgmgr Overview
  *
- * @todo: Descibe b10-cfgmgr here.
+ * @todo: Describe b10-cfgmgr here.
  *
  *
  */

doc/devel/02-dhcp.dox

@@ -21,7 +21,7 @@
  *
  * DHCPv4 server component is now integrated with the BIND10 message queue.
  * The integration is performed by establishSession() and disconnectSession()
- * functions in isc::dhcp::ControlledDhcpv4Srv class. main() method deifined
+ * functions in isc::dhcp::ControlledDhcpv4Srv class. main() method defined
  * in the src/bin/dhcp4/main.cc file instantiates isc::dhcp::ControlledDhcpv4Srv
  * class that establishes connection with msgq and install necessary handlers
  * for receiving commands and configuration updates. It is derived from

examples/README

@@ -31,7 +31,7 @@ sinclude(m4/ax_boost_include.m4)
 sinclude(m4/ax_isc_bind10.m4)
 (and same for other m4 files as they are added under m4/)
 
-On some systems, espeically if you have installed the BIND 10
+On some systems, especially if you have installed the BIND 10
 libraries in an uncommon path, programs linked with the BIND 10
 library may not work at run time due to the "missing" shared library.
 Normally, you should be able to avoid this problem by making sure

src/bin/auth/auth_srv.h

@@ -228,7 +228,7 @@ public:
      * \brief Set and get the addresses we listen on.
      */
     void setListenAddresses(const isc::server_common::portconfig::AddressList&
-                            addreses);
+                            addresses);
     const isc::server_common::portconfig::AddressList& getListenAddresses()
         const;
 

src/bin/auth/gen-statisticsitems.py.pre.in

@@ -289,7 +289,7 @@ def generate_cxx(itemsfile, ccfile, utfile, def_mtime):
         This method recursively builds two lists:
         - msg_counter_types consists of strings for all leaf items, each
           defines one enum element with a comment, e.g.
-          COUNTER_ITEM, ///< item's descriptin
+          COUNTER_ITEM, ///< item's description
         - item_names consists of tuples of three elements, depending on
           whether it's a leaf element (no child from it) or not:
           (leaf)   ( "item_name", NULL, COUNTER_ITEM )

src/bin/auth/query.cc

@@ -373,7 +373,7 @@ Query::process(datasrc::ClientList& client_list,
     // If we have no matching authoritative zone for the query name, return
     // REFUSED.  In short, this is to be compatible with BIND 9, but the
     // background discussion is not that simple.  See the relevant topic
-    // at the BIND 10 developers's ML:
+    // at the BIND 10 developers' ML:
     // https://lists.isc.org/mailman/htdig/bind10-dev/2010-December/001633.html
     if (result.dsrc_client_ == NULL) {
         // If we tried to find a "parent zone" for a DS query and failed,

src/bin/bind10/TODO

@@ -6,7 +6,7 @@
   - Stop a component
   - Force-stop a component
 - Mechanism to wait for child to start before continuing
-- Use .spec file to define comands
+- Use .spec file to define commands
 - Rename "c-channel" stuff to msgq for clarity
 - Reply to shutdown message?
 - Some sort of group creation so termination signals can be sent to

src/bin/bind10/init_messages.mes

@@ -34,7 +34,7 @@ The named component is about to be started by the b10-init process.
 
 % BIND10_COMPONENT_START_EXCEPTION component %1 failed to start: %2
 An exception (mentioned in the message) happened during the startup of the
-named component. The componet is not considered started and further actions
+named component. The component is not considered started and further actions
 will be taken about it.
 
 % BIND10_COMPONENT_STOP component %1 is being stopped

src/bin/bindctl/README

 1. Start bindctl by run command "sh bindctl".
-2. Login to b10-cmdctld with the username and password. The username and password will be saved in file default_user.csv automatcally after logining successfully, so next time bindctl will login with the username and password saved in default_user.csv. For more usage information, please turn to "man bindctl".
\ No newline at end of file
+2. Login to b10-cmdctl with the username and password. The username and password will be saved in file default_user.csv automatically after logining successfully, so next time bindctl will login with the username and password saved in default_user.csv. For more usage information, please turn to "man bindctl".

src/bin/cmdctl/b10-certgen.cc

@@ -62,7 +62,7 @@ usage() {
     std::cout << "Options:" << std::endl;
     std::cout << "-c, --certfile=FILE\t\tfile to read or store the certificate"
               << std::endl;
-    std::cout << "-f, --force\t\t\toverwrite existing certficate even if it"
+    std::cout << "-f, --force\t\t\toverwrite existing certificate even if it"
               << std::endl <<"\t\t\t\tis valid" << std::endl;
     std::cout << "-h, --help\t\t\tshow this help" << std::endl;
     std::cout << "-k, --keyfile=FILE\t\tfile to store the generated private key"
@@ -194,7 +194,7 @@ public:
 
             print("Creating certificate file " + cert_file_name);
 
-            // The exact call changed aftert 1.8, adding the
+            // The exact call changed after 1.8, adding the
             // hash function option
 #if BOTAN_VERSION_CODE >= BOTAN_VERSION_CODE_FOR(1,9,0)
             X509_Certificate cert =
@@ -229,7 +229,7 @@ public:
     validateCertificate(const std::string& certfile) {
         // Since we are dealing with a self-signed certificate here, we
         // also use the certificate to check itself; i.e. we add it
-        // as a trusted certificate, then validate the certficate itself.
+        // as a trusted certificate, then validate the certificate itself.
         //const X509_Certificate cert(certfile);
         try {
             X509_Store store;

src/bin/dbutil/dbutil.py.in

@@ -16,7 +16,7 @@
 # WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 
 """
-@file Dabase Utilities
+@file Database Utilities
 
 This file holds the "dbutil" program, a general utility program for doing
 management of the BIND 10 database.  There are two modes of operation:

src/bin/dbutil/tests/dbutil_test.sh.in

@@ -132,7 +132,7 @@ check_no_backup() {
 # .schema command, with spaces removed and upper converted to lowercase.
 #
 # The database is copied before the schema is taken (and removed after)
-# as SQLite3 assummes a writeable database, which may not be the case if
+# as SQLite3 assumes a writeable database, which may not be the case if
 # getting the schema from a reference copy.
 #
 # @param $1 Database for which the schema is required

src/bin/ddns/ddns.py.in

@@ -236,7 +236,7 @@ class DDNSServer:
         '''Exception for internal errors in an update session.
 
         This exception is expected to be caught within the server class,
-        only used for controling the code flow.
+        only used for controlling the code flow.
 
         '''
         pass
@@ -511,7 +511,7 @@ class DDNSServer:
 
         Right now, this is a straightforward subroutine of handle_request(),
         but is intended to be extended evetually so that it can handle more
-        comlicated operations for TCP (which requires asynchronous write).
+        complicated operations for TCP (which requires asynchronous write).
         Further, when we support multiple requests over a single TCP
         connection, this method may even be shared by multiple methods.
 

src/bin/ddns/tests/ddns_test.py

@@ -40,7 +40,7 @@ READ_ZONE_DB_FILE = TESTDATA_PATH + "rwtest.sqlite3" # original, to be copied
 TEST_ZONE_NAME = Name('example.org')
 TEST_ZONE_NAME_STR = TEST_ZONE_NAME.to_text()
 UPDATE_RRTYPE = RRType.SOA
-TEST_QID = 5353                 # arbitrary chosen
+TEST_QID = 5353                 # arbitrarily chosen
 TEST_RRCLASS = RRClass.IN
 TEST_RRCLASS_STR = TEST_RRCLASS.to_text()
 TEST_SERVER6 = ('2001:db8::53', 53, 0, 0)
@@ -53,7 +53,7 @@ TEST_ACL_CONTEXT = isc.acl.dns.RequestContext(
                        socket.IPPROTO_UDP, socket.AI_NUMERICHOST)[0][4])
 # TSIG key for tests when needed.  The key name is TEST_ZONE_NAME.
 TEST_TSIG_KEY = TSIGKey("example.org:SFuWd/q99SzF8Yzd1QbB9g==")
-# TSIG keyring that contanins the test key
+# TSIG keyring that contains the test key
 TEST_TSIG_KEYRING = TSIGKeyRing()
 TEST_TSIG_KEYRING.add(TEST_TSIG_KEY)
 # Another TSIG key not in the keyring, making verification fail
@@ -450,7 +450,7 @@ class TestDDNSServer(unittest.TestCase):
         self.assertEqual(1, isc.config.parse_answer(answer)[0])
         self.assertEqual({}, self.ddns_server._zone_config)
 
-        # the first zone cofig is valid, but not the second.  the first one
+        # the first zone config is valid, but not the second.  the first one
         # shouldn't be installed.
         bad_config = { 'zones': [ { 'origin': TEST_ZONE_NAME_STR,
                                     'class': TEST_RRCLASS_STR,
@@ -856,7 +856,7 @@ class TestDDNSServer(unittest.TestCase):
     def test_select_multi_tcp(self):
         '''Test continuation of sending a TCP response, multiple sockets.'''
         # Check if the implementation still works with multiple outstanding
-        # TCP contexts.  We use three (arbitray choice), of which two will be
+        # TCP contexts.  We use three (arbitrary choice), of which two will be
         # writable after select and complete the send.
         tcp_socks = []
         for i in range(0, 3):

src/bin/dhcp4/ctrl_dhcp4_srv.cc

@@ -156,7 +156,7 @@ void ControlledDhcpv4Srv::establishSession() {
     // Dumy configuration handler is internally invoked by the
     // constructor and on success the constructor updates
     // the current session with the configuration that had been
-    // commited in the previous session. If we did not install
+    // committed in the previous session. If we did not install
     // the dummy handler, the previous configuration would have
     // been lost.
     config_session_ = new ModuleCCSession(specfile, *cc_session_,

src/bin/dhcp4/ctrl_dhcp4_srv.h

@@ -136,7 +136,7 @@ protected:
     /// @brief Helper session object that represents raw connection to msgq.
     isc::cc::Session* cc_session_;
 
-    /// @brief Session that receives configuation and commands
+    /// @brief Session that receives configuration and commands
     isc::config::ModuleCCSession* config_session_;
 };
 

src/bin/dhcp4/dhcp4.dox

@@ -33,7 +33,7 @@ assigned is not implemented in IfaceMgr yet.
 
 DHCPv4 server component is now integrated with BIND10 message queue.
 The integration is performed by establishSession() and disconnectSession()
-functions in isc::dhcp::ControlledDhcpv4Srv class. main() method deifined
+functions in isc::dhcp::ControlledDhcpv4Srv class. main() method defined
 in the src/bin/dhcp4/main.cc file instantiates isc::dhcp::ControlledDhcpv4Srv
 class that establishes connection with msgq and install necessary handlers
 for receiving commands and configuration updates. It is derived from

src/bin/dhcp4/tests/config_parser_unittest.cc

@@ -487,7 +487,7 @@ TEST_F(Dhcp4ParserTest, optionDefIpv4Address) {
     EXPECT_TRUE(def->getEncapsulatedSpace().empty());
 }
 
-// The goal of this test is to check whether an option definiiton
+// The goal of this test is to check whether an option definition
 // that defines an option carrying a record of data fields can
 // be created.
 TEST_F(Dhcp4ParserTest, optionDefRecord) {
@@ -1060,7 +1060,7 @@ TEST_F(Dhcp4ParserTest, optionDataTwoSpaces) {
     ASSERT_TRUE(status);
     checkResult(status, 0);
 
-    // Options should be now availabe for the subnet.
+    // Options should be now available for the subnet.
     Subnet4Ptr subnet = CfgMgr::instance().getSubnet4(IOAddress("192.0.2.200"));
     ASSERT_TRUE(subnet);
     // Try to get the option from the space dhcp4.
@@ -1430,7 +1430,7 @@ TEST_F(Dhcp4ParserTest, optionDataInvalidChar) {
     testInvalidOptionParam("01020R", "data");
 }
 
-// Verify that option data containins '0x' prefix is rejected
+// Verify that option data containing '0x' prefix is rejected
 // by the configuration.
 TEST_F(Dhcp4ParserTest, optionDataUnexpectedPrefix) {
     // Option code 0 is reserved and should not be accepted
@@ -1530,7 +1530,7 @@ TEST_F(Dhcp4ParserTest, stdOptionData) {
     boost::shared_ptr<Option4AddrLst> option_addrs =
         boost::dynamic_pointer_cast<Option4AddrLst>(option);
     // If cast is unsuccessful than option returned was of a
-    // differnt type than Option6IA. This is wrong.
+    // different type than Option6IA. This is wrong.
     ASSERT_TRUE(option_addrs);
 
     // Get addresses from the option.

src/bin/dhcp6/ctrl_dhcp6_srv.cc

@@ -155,7 +155,7 @@ void ControlledDhcpv6Srv::establishSession() {
     // Dumy configuration handler is internally invoked by the
     // constructor and on success the constructor updates
     // the current session with the configuration that had been
-    // commited in the previous session. If we did not install
+    // committed in the previous session. If we did not install
     // the dummy handler, the previous configuration would have
     // been lost.
     config_session_ = new ModuleCCSession(specfile, *cc_session_,

src/bin/dhcp6/ctrl_dhcp6_srv.h

@@ -134,7 +134,7 @@ protected:
     /// @brief Helper session object that represents raw connection to msgq.
     isc::cc::Session* cc_session_;
 
-    /// @brief Session that receives configuation and commands
+    /// @brief Session that receives configuration and commands
     isc::config::ModuleCCSession* config_session_;
 };
 

src/bin/dhcp6/dhcp6.dox

@@ -51,7 +51,7 @@
  list of parsers for each received entry. Parser is an object that is derived
  from a DhcpConfigParser class. Once a parser is created
  (constructor), its value is set (using build() method). Once all parsers are
- build, the configuration is then applied ("commited") and commit() method is
+ build, the configuration is then applied ("committed") and commit() method is
  called.
 
  All parsers are defined in src/bin/dhcp6/config_parser.cc file. Some of them

src/bin/dhcp6/dhcp6_srv.cc

@@ -353,7 +353,7 @@ Dhcpv6Srv::generateServerID() {
         }
 
         // Some interfaces (like lo on Linux) report 6-bytes long
-        // MAC adress 00:00:00:00:00:00. Let's not use such weird interfaces
+        // MAC address 00:00:00:00:00:00. Let's not use such weird interfaces
         // to generate DUID.
         if (isRangeZero(iface->getMac(), iface->getMac() + iface->getMacLen())) {
             continue;

src/bin/dhcp6/tests/config_parser_unittest.cc

@@ -598,7 +598,7 @@ TEST_F(Dhcp6ParserTest, optionDefIpv6Address) {
     EXPECT_EQ(OPT_IPV6_ADDRESS_TYPE, def->getType());
 }
 
-// The goal of this test is to check whether an option definiiton
+// The goal of this test is to check whether an option definition
 // that defines an option carrying a record of data fields can
 // be created.
 TEST_F(Dhcp6ParserTest, optionDefRecord) {
@@ -1176,7 +1176,7 @@ TEST_F(Dhcp6ParserTest, optionDataTwoSpaces) {
     ASSERT_TRUE(status);
     checkResult(status, 0);
 
-    // Options should be now availabe for the subnet.
+    // Options should be now available for the subnet.
     Subnet6Ptr subnet = CfgMgr::instance().getSubnet6(IOAddress("2001:db8:1::5"));
     ASSERT_TRUE(subnet);
     // Try to get the option from the space dhcp6.
@@ -1484,7 +1484,7 @@ TEST_F(Dhcp6ParserTest, optionDataInvalidChar) {
     testInvalidOptionParam("01020R", "data");
 }
 
-// Verify that option data containins '0x' prefix is rejected
+// Verify that option data containing '0x' prefix is rejected
 // by the configuration.
 TEST_F(Dhcp6ParserTest, optionDataUnexpectedPrefix) {
     // Option code 0 is reserved and should not be accepted
@@ -1582,7 +1582,7 @@ TEST_F(Dhcp6ParserTest, stdOptionData) {
     boost::shared_ptr<Option6IA> optionIA =
         boost::dynamic_pointer_cast<Option6IA>(option);
     // If cast is unsuccessful than option returned was of a
-    // differnt type than Option6IA. This is wrong.
+    // different type than Option6IA. This is wrong.
     ASSERT_TRUE(optionIA);
     // If cast was successful we may use accessors exposed by
     // Option6IA to validate that the content of this option

src/bin/loadzone/tests/loadzone_test.py

@@ -41,7 +41,7 @@ ORIG_SOA_TXT = 'example.org. 3600 IN SOA ns1.example.org. ' +\
     'admin.example.org. 1234 3600 1800 2419200 7200\n'
 NEW_SOA_TXT = 'example.org. 3600 IN SOA ns.example.org. ' +\
     'admin.example.org. 1235 3600 1800 2419200 7200\n'
-# This is the brandnew SOA for a newly created zone
+# This is the brand new SOA for a newly created zone
 ALT_NEW_SOA_TXT = 'example.com. 3600 IN SOA ns.example.com. ' +\
     'admin.example.com. 1234 3600 1800 2419200 7200\n'
 

src/bin/msgq/msgq.py.in

@@ -406,7 +406,7 @@ class MsgQ:
             routing, data = self.read_packet(fd, sock)
         except (MsgQReceiveError, MsgQCloseOnReceive) as err:
             # If it's MsgQCloseOnReceive and that happens without reading
-            # any data, it basically means the remote clinet has closed the
+            # any data, it basically means the remote client has closed the
             # socket, so we log it as debug information.  Otherwise, it's
             # a somewhat unexpected event, so we consider it an "error".
             if isinstance(err, MsgQCloseOnReceive) and not err.partial_read:

src/bin/resolver/response_scrubber.h

@@ -25,7 +25,7 @@
 /// unsigned data in a response is more of a problem. (Note that even data from
 /// signed zones may be not be signed, e.g. delegations are not signed.) In
 /// particular, how do we know that the server from which the response was
-/// received was authoritive for the data it returned?
+/// received was authoritative for the data it returned?
 ///
 /// The part of the code that checks for this is the "Data Scrubbing" module.
 /// Although it includes the checking of IP addresses and ports, it is called
@@ -219,7 +219,7 @@
 /// referral been to the com nameservers, it would be a valid response; the com
 /// zone could well be serving all the data for example.com.  Having said that,
 /// the A record for ns1.example.net would still be regarded as being out of
-/// bailiwick becase the nameserver is not authoritative for the .net zone.
+/// bailiwick because the nameserver is not authoritative for the .net zone.
 ///
 /// \subsection DataScrubbingEx4 Example 4: Inconsistent Answer Section
 /// Qu: www.example.com\n

src/bin/sockcreator/tests/sockcreator_tests.cc

@@ -99,7 +99,7 @@ udpCheck(const int socknum) {
 }
 
 // The check function (tcpCheck/udpCheck) is passed as a parameter to the test
-// code, so provide a conveniet typedef.
+// code, so provide a convenient typedef.
 typedef void (*socket_check_t)(const int);
 
 // Address-family-specific scoket checks.

src/bin/stats/tests/b10-stats-httpd_test.py

@@ -173,7 +173,7 @@ class TestItemNameList(unittest.TestCase):
                          stats_httpd.item_name_list({'a':[1,2,3]}, 'a'))
         self.assertEqual(['a', 'a[0]', 'a[1]', 'a[2]'],
                          stats_httpd.item_name_list({'a':[1,2,3]}, ''))
-        # for a list under adict under a dict
+        # for a list under a dict under a dict
         self.assertEqual(['a', 'a/b', 'a/b[0]', 'a/b[1]', 'a/b[2]'],
                          stats_httpd.item_name_list({'a':{'b':[1,2,3]}}, 'a'))
         self.assertEqual(['a', 'a/b', 'a/b[0]', 'a/b[1]', 'a/b[2]'],

src/bin/stats/tests/b10-stats_test.py

@@ -398,8 +398,8 @@ class TestStats(unittest.TestCase):
             (0, "Stats is up. (PID " + str(os.getpid()) + ")"))
 
         # 'showschema' command.  update_modules() will be called, which
-        # (implicitly) cofirms the correct method is called; further details
-        # are tested seprately.
+        # (implicitly) confirms the correct method is called; further details
+        # are tested separately.
         call_log = []
         (rcode, value) = self.__send_command(__stats, 'showschema')
         self.assertEqual([('update_module', ())], call_log)

src/bin/xfrin/tests/xfrin_test.py

@@ -607,7 +607,7 @@ class TestXfrinIXFRAdd(TestXfrinState):
         # signed, rejecting it.
         self.assertRaises(xfrin.XfrinProtocolError, self.state.handle_rr,
                           self.conn, end_soa_rrset)
-        # No diffs were commited
+        # No diffs were committed
         self.assertEqual([], self.conn._datasrc_client.committed_diffs)
 
     def test_handle_out_of_sync(self):

src/bin/xfrin/xfrin.py.in

@@ -432,7 +432,7 @@ class XfrinIXFRAdd(XfrinState):
             if soa_serial == conn._end_serial:
                 # The final part is there. Finish the transfer by
                 # checking the last TSIG (if required), the zone data and
-                # commiting.
+                # committing.
                 conn.finish_transfer()
                 self.set_xfrstate(conn, XfrinIXFREnd())
                 return True
@@ -820,7 +820,7 @@ class XfrinConnection(asyncore.dispatcher):
     def finish_transfer(self):
         """
         Perform any necessary checks after a transfer. Then complete the
-        transfer by commiting the transaction into the data source.
+        transfer by committing the transaction into the data source.
         """
         self._check_response_tsig_last()
         if not check_zone(self._zone_name, self._rrclass,

src/bin/xfrout/tests/xfrout_test.py.in

@@ -191,7 +191,7 @@ class Dbserver:
         self.transfer_counter -= 1
 
 class TestXfroutSessionBase(unittest.TestCase):
-    '''Base classs for tests related to xfrout sessions
+    '''Base class for tests related to xfrout sessions
 
     This class defines common setup/teadown and utility methods.  Actual
     tests are delegated to subclasses.

src/bin/xfrout/xfrout.py.in

@@ -186,7 +186,7 @@ class XfroutSession():
 
         '''
         # Check the xfrout quota.  We do both increase/decrease in this
-        # method so it's clear we always release it once acuired.
+        # method so it's clear we always release it once acquired.
         quota_ok = self._server.increase_transfers_counter()
         ex = None
         try:

src/lib/asiodns/io_fetch.h

@@ -143,7 +143,7 @@ public:
     /// \brief Constructor
     ///  This constructor has one parameter "query_message", which
     ///  is the shared_ptr to a full query message. It's different
-    ///  with above contructor which has only question section. All
+    ///  with above constructor which has only question section. All
     ///  other parameters are same.
     ///
     /// \param query_message the shared_ptr to a full query message

src/lib/asiodns/sync_udp_server.cc

@@ -174,13 +174,13 @@ SyncUDPServer::operator()(asio::error_code, size_t) {
 void
 SyncUDPServer::stop() {
     /// Using close instead of cancel, because cancel
-    /// will only cancel the asynchornized event already submitted
+    /// will only cancel the asynchronized event already submitted
     /// to io service, the events post to io service after
     /// cancel still can be scheduled by io service, if
-    /// the socket is cloesed, all the asynchronized event
+    /// the socket is closed, all the asynchronized event
     /// for it won't be scheduled by io service not matter it is
-    /// submit to io serice before or after close call. And we will
-    //. get bad_descriptor error
+    /// submit to io service before or after close call. And we will
+    /// get bad_descriptor error.
     socket_->close();
     stopped_ = true;
 }

src/lib/asiodns/tests/dns_server_unittest.cc

@@ -51,8 +51,8 @@
 ///   Before the server start to run
 ///   After we get the query and check whether it's valid
 ///   After we lookup the query
-///   After we compoisite the answer
-///   After user get the final result.
+///   After we compose the answer
+///   After user gets the final result.
 
 /// The standard about whether we stop the server successfully or not
 /// is based on the fact that if the server is still running, the io
@@ -83,7 +83,7 @@ const char* const server_port_str = "5553";
 const char* const query_message = "BIND10 is awesome";
 
 // \brief provide capacity to derived class the ability
-// to stop DNSServer at certern point
+// to stop DNSServer at certain point
 class ServerStopper {
     public:
         ServerStopper() : server_to_stop_(NULL) {}
@@ -148,7 +148,7 @@ class SimpleAnswer : public DNSAnswer, public ServerStopper {
 };
 
 // \brief simple client, send one string to server and wait for response
-//  in case, server stopped and client cann't get response, there is a timer wait
+//  in case, server stopped and client can't get response, there is a timer wait
 //  for specified seconds (the value is just a estimate since server process logic is quite
 //  simple, and all the intercommunication is local) then cancel the waiting.
 class SimpleClient : public ServerStopper {

src/lib/asiodns/udp_server.cc

@@ -323,13 +323,13 @@ UDPServer::asyncLookup() {
 void
 UDPServer::stop() {
     /// Using close instead of cancel, because cancel
-    /// will only cancel the asynchornized event already submitted
+    /// will only cancel the asynchronized event already submitted
     /// to io service, the events post to io service after
     /// cancel still can be scheduled by io service, if
-    /// the socket is cloesed, all the asynchronized event
+    /// the socket is closed, all the asynchronized event
     /// for it won't be scheduled by io service not matter it is
-    /// submit to io serice before or after close call. And we will
-    //. get bad_descriptor error
+    /// submit to io service before or after close call. And we will
+    //  get bad_descriptor error.
     data_->socket_->close();
 }
 

src/lib/asiolink/io_asio_socket.h

@@ -158,7 +158,7 @@ public:
     /// (The reason there is a need to know is because the call to open() passes
     /// in the state of the coroutine at the time the call is made.  On an
     /// asynchronous I/O, we need to set the state to point to the statement
-    /// after the call to open() _before_ we pass the corouine to the open()
+    /// after the call to open() _before_ we pass the coroutine to the open()
     /// call.  Unfortunately, the macros that set the state of the coroutine
     /// also yield control - which we don't want to do if the open is
     /// synchronous.  Hence we need to know before we make the call to open()

src/lib/cc/data.h

@@ -504,7 +504,7 @@ public:
 
     // find the Element at 'id', and store the element pointer in t
     // returns true if found, or false if not found (either because
-    // it doesnt exist or one of the elements in the path is not
+    // it doesn't exist or one of the elements in the path is not
     // a MapElement)
     bool find(const std::string& id, ConstElementPtr& t) const;
 

src/lib/cc/session.cc

@@ -260,7 +260,7 @@ SessionImpl::getSocketDesc() {
     /// @todo boost 1.42 uses native() method, but it is deprecated
     /// in 1.49 and native_handle() is recommended instead
     if (!socket_.is_open()) {
-        isc_throw(InvalidOperation, "Can't return socket desciptor: no socket opened.");
+        isc_throw(InvalidOperation, "Can't return socket descriptor: no socket opened.");
     }
     return socket_.native();
 }

src/lib/cc/tests/session_unittests.cc

@@ -46,7 +46,7 @@ TEST(AsioSession, establish) {
     asio::io_service io_service_;
     Session sess(io_service_);
 
-    // can't return socket desciptor before session is established
+    // can't return socket descriptor before session is established
     EXPECT_THROW(sess.getSocketDesc(), isc::InvalidOperation);
 
     EXPECT_THROW(

src/lib/config/documentation.txt

@@ -18,7 +18,7 @@ In order for this to work, it is important that all modules have a way of dynami
 Overview
 --------
 
-Central to the configuration part is the Configuraion Manager b10-cfgmgr. The configuration manager loads any existing configuration, receives configuration updates from user interfaces, and notifies modules about configuration updates.
+Central to the configuration part is the Configuration Manager b10-cfgmgr. The configuration manager loads any existing configuration, receives configuration updates from user interfaces, and notifies modules about configuration updates.
 
 UI  <---UIAPI---> Configuration Manager <---ModuleAPI---> Module
                                         <---ModuleAPI---> Module

src/lib/config/tests/fake_session.cc

@@ -97,7 +97,7 @@ bool
 FakeSession::recvmsg(ConstElementPtr& msg, bool nonblock, int) {
     if (started_ && !nonblock) {
         // This would schedule another read for length, leading to
-        // corputed data
+        // corrupted data
         isc_throw(DoubleRead, "Second read scheduled from recvmsg");
     }
 
@@ -119,7 +119,7 @@ FakeSession::recvmsg(ConstElementPtr& env, ConstElementPtr& msg, bool nonblock,
 {
     if (started_ && !nonblock) {
         // This would schedule another read for length, leading to
-        // corputed data
+        // corrupted data
         isc_throw(DoubleRead, "Second read scheduled from recvmsg");
     }
 

src/lib/datasrc/database.h

@@ -228,7 +228,7 @@ public:
     public:
         /// \brief Destructor
         ///
-        /// Virtual destructor, so any descendand class is destroyed correctly.
+        /// Virtual destructor, so any descendant class is destroyed correctly.
         virtual ~IteratorContext() {}
 
         /// \brief Function to provide next resource record

src/lib/datasrc/master_loader_callbacks.h

@@ -47,7 +47,7 @@ createMasterLoaderCallbacks(const isc::dns::Name& name,
 /// loaded RRsets into a zone updater.
 ///
 /// The zone updater should be opened in the replace mode no changes should
-/// have been done to it yet (but it is not checked). It is not commited
+/// have been done to it yet (but it is not checked). It is not committed
 /// automatically and it is up to the caller to commit the changes (or not).
 /// It must not be destroyed for the whole time of loading.
 ///

src/lib/datasrc/memory/domaintree.h

@@ -42,7 +42,7 @@ namespace isc {
 namespace datasrc {
 namespace memory {
 
-/// Forward declare DomainTree class here is convinent for following
+/// Forward declare DomainTree class here is convenient for following
 /// friend class declare inside DomainTreeNode and DomainTreeNodeChain
 template <typename T>
 class DomainTree;

src/lib/datasrc/memory/rdata_serialization.h

@@ -533,7 +533,7 @@ public:
         }
     }
 
-    /// \brief Rewind the iterator to the beginnig of data.
+    /// \brief Rewind the iterator to the beginning of data.
     ///
     /// The following next() and nextSig() will start iterating from the
     /// beginning again.

src/lib/datasrc/memory/zone_finder.cc

@@ -144,7 +144,7 @@ struct FindState {
 
     // These will be set to a domain node of the highest delegation point,
     // if any.  In fact, we could use a single variable instead of both.
-    // But then we would need to distinquish these two cases by something
+    // But then we would need to distinguish these two cases by something
     // else and it seemed little more confusing when this was written.
     const ZoneNode* zonecut_node_;
     const ZoneNode* dname_node_;

src/lib/datasrc/sqlite3_accessor.cc

@@ -1068,7 +1068,7 @@ private:
     /// \param serial Zone serial number for which an index is being sought.
     /// \param diff Code to delete record additions or deletions
     ///
-    /// \return int ID of the row in the difss table corresponding to the
+    /// \return int ID of the row in the diffs table corresponding to the
     ///         statement.
     ///
     /// \exception TooLittleData Internal error, no result returned when one

src/lib/datasrc/tests/testdata/rrset_toWire2

@@ -18,7 +18,7 @@
 # RDATA: 192.0.2.1
 c0 00 02 01
 #
-# 2nd RR: the owner name is compresed
+# 2nd RR: the owner name is compressed
 c0 00
 00 01 00 01
 00 00 0e 10

src/lib/datasrc/zone_iterator.h

@@ -27,7 +27,7 @@ namespace datasrc {
 /**
  * \brief Read-only iterator to a zone.
  *
- * You can get an instance of (descendand of) ZoneIterator from
+ * You can get an instance of (descendant of) ZoneIterator from
  * DataSourceClient::getIterator() method. The actual concrete implementation
  * will be different depending on the actual data source used. This is the
  * abstract interface.

src/lib/dhcp/dhcp6.h

@@ -134,7 +134,7 @@ extern const int dhcpv6_type_name_max;
 #define LQ_QUERY_OFFSET 17
 
 /*
- * DHCPv6 well-known multicast addressess, from section 5.1 of RFC 3315
+ * DHCPv6 well-known multicast addresses, from section 5.1 of RFC 3315
  */
 #define ALL_DHCP_RELAY_AGENTS_AND_SERVERS "ff02::1:2"
 #define ALL_DHCP_SERVERS "ff05::1:3"

src/lib/dhcp/libdhcp++.h

@@ -61,7 +61,7 @@ public: