[tor-commits] [tor/master] Remove dead code from circpad_machine_remove_token().

2019-08-19 Thread nickm 
commit 0e4753e579c0288a295d1e0a585e1156dd26590a
Author: George Kadianakis 
Date:   Sat Jun 29 19:38:14 2019 +0300

Remove dead code from circpad_machine_remove_token().
---
 changes/bug31027 | 3 +++
 src/core/or/circuitpadding.c | 9 -
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/changes/bug31027 b/changes/bug31027
new file mode 100644
index 0..dd3ce20b6
--- /dev/null
+++ b/changes/bug31027
@@ -0,0 +1,3 @@
+  o Code simplification and refactoring:
+- Remove some dead code from circpad_machine_remove_token() to fix some
+  Coverity warnings (CID 1447298). Fixes bug 31027; bugfix on 
0.4.1.1-alpha.
\ No newline at end of file
diff --git a/src/core/or/circuitpadding.c b/src/core/or/circuitpadding.c
index 0214cc421..626787da7 100644
--- a/src/core/or/circuitpadding.c
+++ b/src/core/or/circuitpadding.c
@@ -1083,8 +1083,11 @@ circpad_machine_remove_token(circpad_machine_runtime_t 
*mi)
 
   state = circpad_machine_current_state(mi);
 
+  /* If we are not in a padding state (like start or end), we're done */
+  if (!state)
+return;
   /* Don't remove any tokens if we're not doing token removal */
-  if (!state || state->token_removal == CIRCPAD_TOKEN_REMOVAL_NONE)
+  if (state->token_removal == CIRCPAD_TOKEN_REMOVAL_NONE)
 return;
 
   current_time = monotime_absolute_usec();
@@ -1103,10 +1106,6 @@ circpad_machine_remove_token(circpad_machine_runtime_t 
*mi)
 timer_disable(mi->padding_timer);
   }
 
-  /* If we are not in a padding state (like start or end), we're done */
-  if (!state)
-return;
-
   /* Perform the specified token removal strategy */
   switch (state->token_removal) {
 case CIRCPAD_TOKEN_REMOVAL_CLOSEST_USEC:



___
tor-commits mailing list
tor-commits@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits

[tor-commits] [tor/master] Remove dead code from circpad_machine_remove_token().

2019-07-25 Thread dgoulet 
commit bd1ac408d854758cc40bde88cc5f71b46661b51c
Author: George Kadianakis 
Date:   Sat Jun 29 19:38:14 2019 +0300

Remove dead code from circpad_machine_remove_token().
---
 changes/bug31027 | 3 +++
 src/core/or/circuitpadding.c | 9 -
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/changes/bug31027 b/changes/bug31027
new file mode 100644
index 0..dd3ce20b6
--- /dev/null
+++ b/changes/bug31027
@@ -0,0 +1,3 @@
+  o Code simplification and refactoring:
+- Remove some dead code from circpad_machine_remove_token() to fix some
+  Coverity warnings (CID 1447298). Fixes bug 31027; bugfix on 
0.4.1.1-alpha.
\ No newline at end of file
diff --git a/src/core/or/circuitpadding.c b/src/core/or/circuitpadding.c
index 0214cc421..626787da7 100644
--- a/src/core/or/circuitpadding.c
+++ b/src/core/or/circuitpadding.c
@@ -1083,8 +1083,11 @@ circpad_machine_remove_token(circpad_machine_runtime_t 
*mi)
 
   state = circpad_machine_current_state(mi);
 
+  /* If we are not in a padding state (like start or end), we're done */
+  if (!state)
+return;
   /* Don't remove any tokens if we're not doing token removal */
-  if (!state || state->token_removal == CIRCPAD_TOKEN_REMOVAL_NONE)
+  if (state->token_removal == CIRCPAD_TOKEN_REMOVAL_NONE)
 return;
 
   current_time = monotime_absolute_usec();
@@ -1103,10 +1106,6 @@ circpad_machine_remove_token(circpad_machine_runtime_t 
*mi)
 timer_disable(mi->padding_timer);
   }
 
-  /* If we are not in a padding state (like start or end), we're done */
-  if (!state)
-return;
-
   /* Perform the specified token removal strategy */
   switch (state->token_removal) {
 case CIRCPAD_TOKEN_REMOVAL_CLOSEST_USEC:



___
tor-commits mailing list
tor-commits@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits