continue statement in default case applies when a second pass is wanted as well. As it stands the code drops down to the open re-using the value in name from the previous origin attempt.
Signed-off-by: David Ahern <[email protected]> --- tools/perf/util/symbol.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c index d30136e..861be8b 100644 --- a/tools/perf/util/symbol.c +++ b/tools/perf/util/symbol.c @@ -1488,8 +1488,8 @@ int dso__load(struct dso *self, struct map *map, symbol_filter_t filter) if (want_symtab) { want_symtab = 0; self->origin = DSO__ORIG_BUILD_ID_CACHE; - } else - continue; + } + continue; } /* Name is now the name of the next image to try */ -- 1.7.2.3 -- To unsubscribe from this list: send the line "unsubscribe linux-perf-users" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
