Fix E2E protection checker counter increment (#889)

wangdongustc · web-flow · commit 15ceb669990c · 2025-09-15T11:53:49.000+01:00
Bug introduced by 826ebb8. E2E checkers should always update the local counters by the received counters after determining the counter deltas in `verify_counter`.
diff --git a/implementation/e2e_protection/src/e2e/profile/profile04/checker.cpp b/implementation/e2e_protection/src/e2e/profile/profile04/checker.cpp
@@ -77,6 +77,8 @@ bool profile_04_checker::verify_counter(instance_t _instance, uint16_t _received
             its_delta = uint16_t(_received_counter - its_counter);
         else
             its_delta = uint16_t(uint16_t(0xffff) - its_counter + _received_counter);
+
+        find_counter->second = _received_counter;
     } else {
         counter_[_instance] = _received_counter;
     }
diff --git a/implementation/e2e_protection/src/e2e/profile/profile05/checker.cpp b/implementation/e2e_protection/src/e2e/profile/profile05/checker.cpp
@@ -58,6 +58,8 @@ bool profile_05_checker::verify_counter(instance_t _instance, uint8_t _received_
             its_delta = uint8_t(_received_counter - its_counter);
         else
             its_delta = uint8_t(uint8_t(0xff) - its_counter + _received_counter);
+
+        find_counter->second = _received_counter;
     } else {
         counter_[_instance] = _received_counter;
     }
diff --git a/implementation/e2e_protection/src/e2e/profile/profile07/checker.cpp b/implementation/e2e_protection/src/e2e/profile/profile07/checker.cpp
@@ -71,6 +71,8 @@ bool profile_07_checker::verify_counter(instance_t _instance, uint32_t _received
             its_delta = uint32_t(_received_counter - its_counter);
         else
             its_delta = uint32_t(uint32_t(0xffffffff) - its_counter + _received_counter);
+
+        find_counter->second = _received_counter;
     } else {
         counter_[_instance] = _received_counter;
     }

Original file line number	Diff line number	Diff line change
`@@ -77,6 +77,8 @@ bool profile_04_checker::verify_counter(instance_t _instance, uint16_t _received`
`77`	`77`	`its_delta = uint16_t(_received_counter - its_counter);`
`78`	`78`	`else`
`79`	`79`	`its_delta = uint16_t(uint16_t(0xffff) - its_counter + _received_counter);`
	`80`	`+`
	`81`	`+ find_counter->second = _received_counter;`
`80`	`82`	`} else {`
`81`	`83`	`counter_[_instance] = _received_counter;`
`82`	`84`	`}`
Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,8 @@ bool profile_05_checker::verify_counter(instance_t _instance, uint8_t _received_`
`58`	`58`	`its_delta = uint8_t(_received_counter - its_counter);`
`59`	`59`	`else`
`60`	`60`	`its_delta = uint8_t(uint8_t(0xff) - its_counter + _received_counter);`
	`61`	`+`
	`62`	`+ find_counter->second = _received_counter;`
`61`	`63`	`} else {`
`62`	`64`	`counter_[_instance] = _received_counter;`
`63`	`65`	`}`
Original file line number	Diff line number	Diff line change
`@@ -71,6 +71,8 @@ bool profile_07_checker::verify_counter(instance_t _instance, uint32_t _received`
`71`	`71`	`its_delta = uint32_t(_received_counter - its_counter);`
`72`	`72`	`else`
`73`	`73`	`its_delta = uint32_t(uint32_t(0xffffffff) - its_counter + _received_counter);`
	`74`	`+`
	`75`	`+ find_counter->second = _received_counter;`
`74`	`76`	`} else {`
`75`	`77`	`counter_[_instance] = _received_counter;`
`76`	`78`	`}`