Fix duplicated if condition in connection.c

Furthermore, fix a test that could returned an uninitialized value.

Signed-off-by: David Goulet <dgoulet@torproject.org>

2 files changed, 2 insertions, 2 deletions

diff --git a/src/or/connection.c b/src/or/connection.c
index 9b583f4243..e3560af4c5 100644
--- a/src/or/connection.c
+++ b/src/or/connection.c
@@ -4511,7 +4511,7 @@ oos_victim_comparator_for_orconns(or_connection_t *a, or_connection_t *b)
   b_circs = connection_or_get_num_circuits(b);
 
   if (a_circs < b_circs) return -1;
-  else if (b_circs > a_circs) return 1;
+  else if (a_circs > b_circs) return 1;
   else return 0;
 }
 
diff --git a/src/test/test_oos.c b/src/test/test_oos.c
index 9451c5c537..3afb68b27a 100644
--- a/src/test/test_oos.c
+++ b/src/test/test_oos.c
@@ -67,7 +67,7 @@ static int pick_oos_mock_last_n = 0;
 static smartlist_t *
 pick_oos_victims_mock(int n)
 {
-  smartlist_t *l;
+  smartlist_t *l = NULL;
   int i;
 
   ++pick_oos_mock_calls;