No point in malloc()ing space if we just immediately overwrite the pointer via strdup. That'll leak some space. Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> --- diff --git a/btreplay/btrecord.c b/btreplay/btrecord.c index 88ab806..3646257 100644 --- a/btreplay/btrecord.c +++ b/btreplay/btrecord.c @@ -365,7 +365,7 @@ static void find_input_files(char *idir) } while ((ent = readdir(dir)) != NULL) { - char *p, *dsf = malloc(256); + char *p, *dsf; if (strstr(ent->d_name, ".blktrace.") == NULL) continue; diff --git a/btreplay/btreplay.c b/btreplay/btreplay.c index f4f5aa0..20494e0 100644 --- a/btreplay/btreplay.c +++ b/btreplay/btreplay.c @@ -596,7 +596,7 @@ static void find_input_devs(char *idir) } while ((ent = readdir(dir)) != NULL) { - char *p, *dsf = malloc(256); + char *p, *dsf; if (strstr(ent->d_name, ".replay.") == NULL) continue; -- To unsubscribe from this list: send the line "unsubscribe linux-btrace" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html