fix renew/rebind spam on successful renewal0.3.4

This fixes the state not being updated correctly when a Reply to a Renew or Rebind packet is received. It also fixes renewals not being written to disk.

3 files changed, 7 insertions, 4 deletions

diff --git a/Cargo.lock b/Cargo.lock
index 24ff094..9747fbf 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -384,7 +384,7 @@ dependencies = [
 
 [[package]]
 name = "rsdsl_dhcp6"
-version = "0.3.3"
+version = "0.3.4"
 dependencies = [
  "dhcproto",
  "rand",
diff --git a/Cargo.toml b/Cargo.toml
index 61e7126..be8b43a 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "rsdsl_dhcp6"
-version = "0.3.3"
+version = "0.3.4"
 edition = "2021"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
diff --git a/src/client.rs b/src/client.rs
index 5d2fcdb..c5806dd 100644
--- a/src/client.rs
+++ b/src/client.rs
@@ -325,7 +325,11 @@ impl Dhcp6c {
     fn rr(&mut self, t1: Duration, t2: Duration, valid_lifetime: Duration) {
         match self.state {
             Dhcp6cState::Starting | Dhcp6cState::Opened => {} // illegal
-            Dhcp6cState::Soliciting | Dhcp6cState::Requesting | Dhcp6cState::Rerouting => {
+            Dhcp6cState::Soliciting
+            | Dhcp6cState::Requesting
+            | Dhcp6cState::Renewing
+            | Dhcp6cState::Rebinding
+            | Dhcp6cState::Rerouting => {
                 self.upper_status_tx
                     .send(true)
                     .expect("upper status channel is closed");
@@ -339,7 +343,6 @@ impl Dhcp6c {
 
                 self.state = Dhcp6cState::Opened;
             }
-            Dhcp6cState::Renewing | Dhcp6cState::Rebinding => self.state = Dhcp6cState::Opened,
         }
     }
 }