summaryrefslogtreecommitdiffstats
path: root/src/bin/dhcp6/tests/dhcp6_client.cc
diff options
context:
space:
mode:
authorMarcin Siodelski <marcin@isc.org>2015-09-02 15:06:42 +0200
committerMarcin Siodelski <marcin@isc.org>2015-09-02 15:06:42 +0200
commita14f0a33359434ad50e9289deabf664d5f306f75 (patch)
tree18a587d03c68d55584f1e6100f094bc44b653e01 /src/bin/dhcp6/tests/dhcp6_client.cc
parent[3947] Removed spurious config parameter from the Subnet6. (diff)
downloadkea-a14f0a33359434ad50e9289deabf664d5f306f75.tar.xz
kea-a14f0a33359434ad50e9289deabf664d5f306f75.zip
[3947] Addressed review comments.
Diffstat (limited to 'src/bin/dhcp6/tests/dhcp6_client.cc')
-rw-r--r--src/bin/dhcp6/tests/dhcp6_client.cc10
1 files changed, 5 insertions, 5 deletions
diff --git a/src/bin/dhcp6/tests/dhcp6_client.cc b/src/bin/dhcp6/tests/dhcp6_client.cc
index 503f973b3d..d1b9a38995 100644
--- a/src/bin/dhcp6/tests/dhcp6_client.cc
+++ b/src/bin/dhcp6/tests/dhcp6_client.cc
@@ -287,7 +287,7 @@ Dhcp6Client::conditionallyAppendRequestedIA(const Pkt6Ptr& query,
} else if (address_hint_ && (ia_type == D6O_IA_NA)) {
requested_ia->addOption(address_hint_);
- }
+ }
query->addOption(requested_ia);
}
@@ -323,8 +323,8 @@ Dhcp6Client::copyIAsFromLeases(const Pkt6Ptr& dest) const {
for (std::set<uint32_t>::const_iterator iaid = iaids.begin();
iaid != iaids.end(); ++iaid) {
std::vector<Lease6> leases = getLeasesByIAID(*iaid);
- // Only a valid lease should be included. If we have received an
- // error status code, it should not be copied.
+ // Only a valid lease should be included. Do not copy a
+ // lease which have been marked by the server as invalid.
if (leases[0].valid_lft_ == 0) {
continue;
}
@@ -456,8 +456,8 @@ Dhcp6Client::doInfRequest() {
// IA-PD is also not allowed. So it may be useful in testing, too.
if (use_pd_) {
- // Insert IA_PD option with iaid=5678
- Option6IAPtr ia(new Option6IA(D6O_IA_PD, 5678));
+ // Insert IA_PD option.
+ Option6IAPtr ia(new Option6IA(D6O_IA_PD, pd_iaid_));
if (prefix_hint_) {
ia->addOption(prefix_hint_);
}