diff options
Diffstat (limited to 'tools/perf/util/config.c')
-rw-r--r-- | tools/perf/util/config.c | 22 |
1 files changed, 13 insertions, 9 deletions
diff --git a/tools/perf/util/config.c b/tools/perf/util/config.c index 780df541006d..8784649109ce 100644 --- a/tools/perf/util/config.c +++ b/tools/perf/util/config.c | |||
@@ -160,17 +160,18 @@ static int get_extended_base_var(char *name, int baselen, int c) | |||
160 | name[baselen++] = '.'; | 160 | name[baselen++] = '.'; |
161 | 161 | ||
162 | for (;;) { | 162 | for (;;) { |
163 | int c = get_next_char(); | 163 | int ch = get_next_char(); |
164 | if (c == '\n') | 164 | |
165 | if (ch == '\n') | ||
165 | return -1; | 166 | return -1; |
166 | if (c == '"') | 167 | if (ch == '"') |
167 | break; | 168 | break; |
168 | if (c == '\\') { | 169 | if (ch == '\\') { |
169 | c = get_next_char(); | 170 | ch = get_next_char(); |
170 | if (c == '\n') | 171 | if (ch == '\n') |
171 | return -1; | 172 | return -1; |
172 | } | 173 | } |
173 | name[baselen++] = c; | 174 | name[baselen++] = ch; |
174 | if (baselen > MAXNAME / 2) | 175 | if (baselen > MAXNAME / 2) |
175 | return -1; | 176 | return -1; |
176 | } | 177 | } |
@@ -530,6 +531,8 @@ static int store_aux(const char* key, const char* value, void *cb __used) | |||
530 | store.offset[store.seen] = ftell(config_file); | 531 | store.offset[store.seen] = ftell(config_file); |
531 | } | 532 | } |
532 | } | 533 | } |
534 | default: | ||
535 | break; | ||
533 | } | 536 | } |
534 | return 0; | 537 | return 0; |
535 | } | 538 | } |
@@ -619,6 +622,7 @@ contline: | |||
619 | switch (contents[offset]) { | 622 | switch (contents[offset]) { |
620 | case '=': equal_offset = offset; break; | 623 | case '=': equal_offset = offset; break; |
621 | case ']': bracket_offset = offset; break; | 624 | case ']': bracket_offset = offset; break; |
625 | default: break; | ||
622 | } | 626 | } |
623 | if (offset > 0 && contents[offset-1] == '\\') { | 627 | if (offset > 0 && contents[offset-1] == '\\') { |
624 | offset_ = offset; | 628 | offset_ = offset; |
@@ -742,9 +746,9 @@ int perf_config_set_multivar(const char* key, const char* value, | |||
742 | goto write_err_out; | 746 | goto write_err_out; |
743 | } else { | 747 | } else { |
744 | struct stat st; | 748 | struct stat st; |
745 | char* contents; | 749 | char *contents; |
746 | ssize_t contents_sz, copy_begin, copy_end; | 750 | ssize_t contents_sz, copy_begin, copy_end; |
747 | int i, new_line = 0; | 751 | int new_line = 0; |
748 | 752 | ||
749 | if (value_regex == NULL) | 753 | if (value_regex == NULL) |
750 | store.value_regex = NULL; | 754 | store.value_regex = NULL; |