diff options
| author | Nick Mathewson <nickm@torproject.org> | 2014-09-16 10:57:00 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2014-09-16 10:57:00 -0400 |
| commit | ef9a0d204838768055194bd47acdb0074c2f13cb (patch) | |
| tree | b4536c7f79495fca681ba1ff7fe610e27f8695f0 | |
| parent | 0bd220adcb82670893c85b1cc24d64e615855b4f (diff) | |
| download | tor-ef9a0d204838768055194bd47acdb0074c2f13cb.tar.gz tor-ef9a0d204838768055194bd47acdb0074c2f13cb.zip | |
Add script to detect and remove unCish malloc-then-cast pattern
Also, apply it.
| -rw-r--r-- | scripts/coccinelle/malloc_cast.cocci | 38 | ||||
| -rw-r--r-- | src/or/circuitstats.c | 2 |
2 files changed, 39 insertions, 1 deletions
diff --git a/scripts/coccinelle/malloc_cast.cocci b/scripts/coccinelle/malloc_cast.cocci new file mode 100644 index 0000000000..20321d4fd0 --- /dev/null +++ b/scripts/coccinelle/malloc_cast.cocci @@ -0,0 +1,38 @@ +@cast_malloc@ +expression e; +type T; +@@ +- (T *)tor_malloc(e) ++ tor_malloc(e) + +@cast_malloc_zero@ +expression e; +type T; +identifier func; +@@ +- (T *)tor_malloc_zero(e) ++ tor_malloc_zero(e) + +@cast_calloc@ +expression a, b; +type T; +identifier func; +@@ +- (T *)tor_calloc(a, b) ++ tor_calloc(a, b) + +@cast_realloc@ +expression e; +expression p; +type T; +@@ +- (T *)tor_realloc(p, e) ++ tor_realloc(p, e) + +@cast_reallocarray@ +expression a,b; +expression p; +type T; +@@ +- (T *)tor_reallocarray(p, a, b) ++ tor_reallocarray(p, a, b) diff --git a/src/or/circuitstats.c b/src/or/circuitstats.c index 88a1f9b46c..c24259c22c 100644 --- a/src/or/circuitstats.c +++ b/src/or/circuitstats.c @@ -691,7 +691,7 @@ circuit_build_times_get_xm(circuit_build_times_t *cbt) if (cbt->total_build_times < CBT_NCIRCUITS_TO_OBSERVE) num_modes = 1; - nth_max_bin = (build_time_t*)tor_calloc(num_modes, sizeof(build_time_t)); + nth_max_bin = tor_calloc(num_modes, sizeof(build_time_t)); /* Determine the N most common build times */ for (i = 0; i < nbins; i++) { |