The log->list always has "strdup_strings" activated, hence strdup'ing namebuf was unnecessary. This change also removes a latent memory leak in the old code. Signed-off-by: Adeodato Simó <dato@xxxxxxxxxxxxxx> --- builtin-shortlog.c | 5 +---- 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/builtin-shortlog.c b/builtin-shortlog.c index 4c5d761..90e76ae 100644 --- a/builtin-shortlog.c +++ b/builtin-shortlog.c @@ -67,12 +67,9 @@ static void insert_one_record(struct shortlog *log, snprintf(namebuf + len, room, " %.*s", maillen, boemail); } - buffer = xstrdup(namebuf); - item = string_list_insert(buffer, &log->list); + item = string_list_insert(namebuf, &log->list); if (item->util == NULL) item->util = xcalloc(1, sizeof(struct string_list)); - else - free(buffer); /* Skip any leading whitespace, including any blank lines. */ while (*oneline && isspace(*oneline)) -- 1.6.0.4 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html