diff options
| author | Nick Mathewson <nickm@torproject.org> | 2009-10-26 21:35:26 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2009-10-26 21:35:26 -0400 |
| commit | 8bada1ef67cd7e84f3f22f7e4ef8eb99a8252776 (patch) | |
| tree | 69c4badd07f4660c3c515ca4c4b38d929115b47c /src/or | |
| parent | 56048637a5e34220d0a5c7abbef4eb56c9312f6c (diff) | |
| download | tor-8bada1ef67cd7e84f3f22f7e4ef8eb99a8252776.tar.gz tor-8bada1ef67cd7e84f3f22f7e4ef8eb99a8252776.zip | |
Add missing break statements for Coverity CIDs #406,407.
The code for these was super-wrong, but will only break things when we
reset an option on a platform where sizeof(time_t) is different from
sizeof(int).
Diffstat (limited to 'src/or')
| -rw-r--r-- | src/or/config.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/or/config.c b/src/or/config.c index 4e2a1765dc..128aa4da34 100644 --- a/src/or/config.c +++ b/src/or/config.c @@ -2225,6 +2225,7 @@ option_clear(config_format_t *fmt, or_options_t *options, config_var_t *var) break; case CONFIG_TYPE_ISOTIME: *(time_t*)lvalue = 0; + break; case CONFIG_TYPE_INTERVAL: case CONFIG_TYPE_UINT: case CONFIG_TYPE_BOOL: @@ -2238,6 +2239,7 @@ option_clear(config_format_t *fmt, or_options_t *options, config_var_t *var) routerset_free(*(routerset_t**)lvalue); *(routerset_t**)lvalue = NULL; } + break; case CONFIG_TYPE_CSV: if (*(smartlist_t**)lvalue) { SMARTLIST_FOREACH(*(smartlist_t **)lvalue, char *, cp, tor_free(cp)); |