Fix a memleak when throwing away some build times

This was introduced in f7e6e852e80c22b40a8f09bc1c85074726d7078e.
Found by Coverity

diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c
index 54edccc9a7e39e8f62700e326da919ecc58ffd45..fbdbb85b2fe1205bf00324b97daafe24bebb81d9 100644 (file)
--- a/src/or/circuitbuild.c
+++ b/src/or/circuitbuild.c
@@ -454,6 +454,8 @@ circuit_build_times_parse_state(circuit_build_times_t *cbt,
                  "Too many build times in state file. "
                  "Stopping short before %d",
                  loaded_cnt+count);
+        SMARTLIST_FOREACH(args, char*, cp, tor_free(cp));
+        smartlist_free(args);
         break;
       }