]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[5682] addLease() parameter is const again
authorTomek Mrugalski <tomasz@isc.org>
Mon, 30 Jul 2018 14:32:01 +0000 (16:32 +0200)
committerTomek Mrugalski <tomasz@isc.org>
Mon, 30 Jul 2018 17:05:19 +0000 (19:05 +0200)
src/lib/dhcpsrv/cql_lease_mgr.cc
src/lib/dhcpsrv/cql_lease_mgr.h
src/lib/dhcpsrv/lease_mgr.h
src/lib/dhcpsrv/memfile_lease_mgr.cc
src/lib/dhcpsrv/memfile_lease_mgr.h
src/lib/dhcpsrv/mysql_lease_mgr.cc
src/lib/dhcpsrv/mysql_lease_mgr.h
src/lib/dhcpsrv/pgsql_lease_mgr.cc
src/lib/dhcpsrv/pgsql_lease_mgr.h
src/lib/dhcpsrv/tests/lease_mgr_unittest.cc

index 01e18af705153626b3f81c5df3192fb0d3b26189..7085ff436c8efd7d0e7410ffd0fae6c6c636b8ee 100644 (file)
@@ -2013,7 +2013,7 @@ CqlLeaseMgr::getDBVersion() {
 }
 
 bool
-CqlLeaseMgr::addLease(Lease4Ptr &lease) {
+CqlLeaseMgr::addLease(const Lease4Ptr &lease) {
     LOG_DEBUG(dhcpsrv_logger, DHCPSRV_DBG_TRACE_DETAIL, DHCPSRV_CQL_ADD_ADDR4)
         .arg(lease->addr_.toText());
 
@@ -2032,7 +2032,7 @@ CqlLeaseMgr::addLease(Lease4Ptr &lease) {
 }
 
 bool
-CqlLeaseMgr::addLease(Lease6Ptr &lease) {
+CqlLeaseMgr::addLease(const Lease6Ptr &lease) {
     LOG_DEBUG(dhcpsrv_logger, DHCPSRV_DBG_TRACE_DETAIL, DHCPSRV_CQL_ADD_ADDR6)
         .arg(lease->addr_.toText());
 
index 653e487abe69e4b44c8917b5aa886a9d70b85522..39382b2f4efaaced27a8ada18270916d4413b55b 100644 (file)
@@ -87,7 +87,7 @@ public:
     ///
     /// @throw isc::dhcp::DbOperationError An operation on the open database has
     ///        failed.
-    virtual bool addLease(Lease4Ptr& lease) override;
+    virtual bool addLease(const Lease4Ptr& lease) override;
 
     /// @brief Adds an IPv6 lease
     ///
@@ -98,7 +98,7 @@ public:
     ///
     /// @throw isc::dhcp::DbOperationError An operation on the open database has
     ///        failed.
-    virtual bool addLease(Lease6Ptr& lease) override;
+    virtual bool addLease(const Lease6Ptr& lease) override;
 
     /// @brief Basic lease access methods. Obtain leases from the database using
     ///     various criteria.
index 099754875e485cf79f1300c0b9d341b13d405637..642bb6b759f02a0ca0addaccd8eade294aafd9f4 100644 (file)
@@ -246,7 +246,7 @@ public:
     ///
     /// @result true if the lease was added, false if not (because a lease
     ///         with the same address was already there or failed sanity checks)
-    virtual bool addLease(Lease4Ptr& lease) = 0;
+    virtual bool addLease(const Lease4Ptr& lease) = 0;
 
     /// @brief Adds an IPv6 lease.
     ///
@@ -259,7 +259,7 @@ public:
     ///
     /// @result true if the lease was added, false if not (because a lease
     ///         with the same address was already there or failed sanity checks)
-    virtual bool addLease(Lease6Ptr& lease) = 0;
+    virtual bool addLease(const Lease6Ptr& lease) = 0;
 
     /// @brief Returns an IPv4 lease for specified IPv4 address
     ///
index 80384af91f164ec915522719166a6574359564f7..a4b4a1114c3417029aeb8cdbf263416332073ad7 100644 (file)
@@ -690,7 +690,7 @@ Memfile_LeaseMgr::getDBVersion() {
 }
 
 bool
-Memfile_LeaseMgr::addLease(Lease4Ptr& lease) {
+Memfile_LeaseMgr::addLease(const Lease4Ptr& lease) {
     LOG_DEBUG(dhcpsrv_logger, DHCPSRV_DBG_TRACE_DETAIL,
               DHCPSRV_MEMFILE_ADD_ADDR4).arg(lease->addr_.toText());
 
@@ -711,7 +711,7 @@ Memfile_LeaseMgr::addLease(Lease4Ptr& lease) {
 }
 
 bool
-Memfile_LeaseMgr::addLease(Lease6Ptr& lease) {
+Memfile_LeaseMgr::addLease(const Lease6Ptr& lease) {
     LOG_DEBUG(dhcpsrv_logger, DHCPSRV_DBG_TRACE_DETAIL,
               DHCPSRV_MEMFILE_ADD_ADDR6).arg(lease->addr_.toText());
 
index 196ebb3ad35f7d0ff4138b47f93ebf9bc0e147e4..66dedccb0d659cc14960191e16d47c571b68e519 100644 (file)
@@ -141,12 +141,12 @@ public:
     /// @brief Adds an IPv4 lease.
     ///
     /// @param lease lease to be added
-    virtual bool addLease(Lease4Ptr& lease);
+    virtual bool addLease(const Lease4Ptr& lease);
 
     /// @brief Adds an IPv6 lease.
     ///
     /// @param lease lease to be added
-    virtual bool addLease(Lease6Ptr& lease);
+    virtual bool addLease(const Lease6Ptr& lease);
 
     /// @brief Returns existing IPv4 lease for specified IPv4 address.
     ///
index 377ef8c2edefcfe87bd559d9f26ce7ff48eedac6..16c3327ce42aaa2f3684d8ca432be3832a597521 100644 (file)
@@ -1699,7 +1699,7 @@ MySqlLeaseMgr::addLeaseCommon(StatementIndex stindex,
 }
 
 bool
-MySqlLeaseMgr::addLease(Lease4Ptr& lease) {
+MySqlLeaseMgr::addLease(const Lease4Ptr& lease) {
     LOG_DEBUG(dhcpsrv_logger, DHCPSRV_DBG_TRACE_DETAIL,
               DHCPSRV_MYSQL_ADD_ADDR4).arg(lease->addr_.toText());
 
@@ -1711,7 +1711,7 @@ MySqlLeaseMgr::addLease(Lease4Ptr& lease) {
 }
 
 bool
-MySqlLeaseMgr::addLease(Lease6Ptr& lease) {
+MySqlLeaseMgr::addLease(const Lease6Ptr& lease) {
     LOG_DEBUG(dhcpsrv_logger, DHCPSRV_DBG_TRACE_DETAIL,
               DHCPSRV_MYSQL_ADD_ADDR6).arg(lease->addr_.toText())
               .arg(lease->type_);
index 3a777c1b8ce34134d58014f9cc45543e9a6dd7f2..64fe4710a379d6ceca41ae6db74b8f6466a55978 100644 (file)
@@ -74,7 +74,7 @@ public:
     ///
     /// @throw isc::dhcp::DbOperationError An operation on the open database has
     ///        failed.
-    virtual bool addLease(Lease4Ptr& lease);
+    virtual bool addLease(const Lease4Ptr& lease);
 
     /// @brief Adds an IPv6 lease
     ///
@@ -85,7 +85,7 @@ public:
     ///
     /// @throw isc::dhcp::DbOperationError An operation on the open database has
     ///        failed.
-    virtual bool addLease(Lease6Ptr& lease);
+    virtual bool addLease(const Lease6Ptr& lease);
 
     /// @brief Returns an IPv4 lease for specified IPv4 address
     ///
index c187e3aa400d7552f79372a36c6c0c80a67a6950..fc3bc0dfd0c4dc108faeeecd7acc3c621995165f 100644 (file)
@@ -1135,7 +1135,7 @@ PgSqlLeaseMgr::addLeaseCommon(StatementIndex stindex,
 }
 
 bool
-PgSqlLeaseMgr::addLease(Lease4Ptr& lease) {
+PgSqlLeaseMgr::addLease(const Lease4Ptr& lease) {
     LOG_DEBUG(dhcpsrv_logger, DHCPSRV_DBG_TRACE_DETAIL,
               DHCPSRV_PGSQL_ADD_ADDR4).arg(lease->addr_.toText());
 
@@ -1145,7 +1145,7 @@ PgSqlLeaseMgr::addLease(Lease4Ptr& lease) {
 }
 
 bool
-PgSqlLeaseMgr::addLease(Lease6Ptr& lease) {
+PgSqlLeaseMgr::addLease(const Lease6Ptr& lease) {
     LOG_DEBUG(dhcpsrv_logger, DHCPSRV_DBG_TRACE_DETAIL,
               DHCPSRV_PGSQL_ADD_ADDR6).arg(lease->addr_.toText());
     PsqlBindArray bind_array;
index 549827dad8902f17c75f746f1bedbe17043734e7..8b69c46b97dea9c5d6a3170330ec3294ec6ad9b9 100644 (file)
@@ -71,7 +71,7 @@ public:
     ///
     /// @throw isc::dhcp::DbOperationError An operation on the open database has
     ///        failed.
-    virtual bool addLease(Lease4Ptr& lease);
+    virtual bool addLease(const Lease4Ptr& lease);
 
     /// @brief Adds an IPv6 lease
     ///
@@ -82,7 +82,7 @@ public:
     ///
     /// @throw isc::dhcp::DbOperationError An operation on the open database has
     ///        failed.
-    virtual bool addLease(Lease6Ptr& lease);
+    virtual bool addLease(const Lease6Ptr& lease);
 
     /// @brief Returns an IPv4 lease for specified IPv4 address
     ///
index 66a6c8ddc2d456bb2fecc7ba07d7ec7865059009..d32379fe8dd2e0a3c538ed8788ce723ef5b65075 100644 (file)
@@ -46,14 +46,14 @@ public:
     /// @brief Adds an IPv4 lease.
     ///
     /// @param lease lease to be added
-    virtual bool addLease(Lease4Ptr&) {
+    virtual bool addLease(const Lease4Ptr&) {
         return (false);
     }
 
     /// @brief Adds an IPv6 lease.
     ///
     /// @param lease lease to be added
-    virtual bool addLease(Lease6Ptr&) {
+    virtual bool addLease(const Lease6Ptr&) {
         return (false);
     }