free() is not a safe function for a signal handler. The next line calls _exit() anyway, so there is no need for resource management. --- text-utils/rev.c | 1 - 1 file changed, 1 deletion(-) diff --git a/text-utils/rev.c b/text-utils/rev.c index cb38482..ace2cd6 100644 --- a/text-utils/rev.c +++ b/text-utils/rev.c @@ -68,7 +68,6 @@ wchar_t *buf; static void sig_handler(int signo __attribute__ ((__unused__))) { - free(buf); _exit(EXIT_SUCCESS); } -- 2.10.0 -- To unsubscribe from this list: send the line "unsubscribe util-linux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html