aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKarol Latecki <karolx.latecki@intel.com>2016-10-21 12:21:38 -0400
committerKarol Latecki <karolx.latecki@intel.com>2017-01-19 13:32:18 +0000
commitc603d3ed138c0fe8a0944baf499c271aae620bc2 (patch)
treee2eec73c47192cc5cc63fea3f2ece7581229e7cf
parentaf7f90313b4f97cdfa401615b99beb2aeaa67f96 (diff)
tldk: fix possibility of tle_event_active() from DOWN to UP
Docstring in tle_event.h for tle_event_active() only describes possibility of activating event from IDLE state. Also re-added previously failing test case. Change-Id: Ib9e19b6b3e64cc2836337b6a86884f312df73858 Signed-off-by: Karol Latecki <karolx.latecki@intel.com> Signed-off-by: Mohammad Abdul Awal <mohammad.abdul.awal@intel.com>
-rw-r--r--lib/libtle_udp/tle_event.h2
-rw-r--r--test/gtest/test_tle_udp_event.cpp1
2 files changed, 2 insertions, 1 deletions
diff --git a/lib/libtle_udp/tle_event.h b/lib/libtle_udp/tle_event.h
index 1a5c436..9357def 100644
--- a/lib/libtle_udp/tle_event.h
+++ b/lib/libtle_udp/tle_event.h
@@ -168,7 +168,7 @@ tle_event_active(struct tle_event *ev, enum tle_ev_state st)
{
struct tle_evq *q;
- if (st == ev->state)
+ if (ev->state != TLE_SEV_IDLE)
return;
q = ev->head;
diff --git a/test/gtest/test_tle_udp_event.cpp b/test/gtest/test_tle_udp_event.cpp
index c7c25a0..c012ccc 100644
--- a/test/gtest/test_tle_udp_event.cpp
+++ b/test/gtest/test_tle_udp_event.cpp
@@ -125,6 +125,7 @@ INSTANTIATE_TEST_CASE_P(Default, udp_event_state_active_twice,
event_state_active_twice{TLE_SEV_IDLE, TLE_SEV_DOWN, TLE_SEV_DOWN},
event_state_active_twice{TLE_SEV_IDLE, TLE_SEV_UP, TLE_SEV_UP},
event_state_active_twice{TLE_SEV_DOWN, TLE_SEV_IDLE, TLE_SEV_DOWN},
+ event_state_active_twice{TLE_SEV_DOWN, TLE_SEV_UP, TLE_SEV_DOWN},
event_state_active_twice{TLE_SEV_DOWN, TLE_SEV_DOWN, TLE_SEV_DOWN},
event_state_active_twice{TLE_SEV_UP, TLE_SEV_IDLE, TLE_SEV_UP},
event_state_active_twice{TLE_SEV_UP, TLE_SEV_DOWN, TLE_SEV_UP},