Re: [PATCH] trace-cmd: Fix double free in tsync_agent_thread()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 31 Mar 2021 17:05:03 -0400
Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> From: "Steven Rostedt (VMware)" <rostedt@xxxxxxxxxxx>
> 
> tracecmd_tsync_free() already frees the tsync, remove the free of it after
> that is called.
> 
> Signed-off-by: Steven Rostedt (VMware) <rostedt@xxxxxxxxxxx>
> ---
>  lib/trace-cmd/trace-timesync.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/lib/trace-cmd/trace-timesync.c b/lib/trace-cmd/trace-timesync.c
> index e860c335..ad73f08f 100644
> --- a/lib/trace-cmd/trace-timesync.c
> +++ b/lib/trace-cmd/trace-timesync.c
> @@ -887,7 +887,6 @@ static void *tsync_agent_thread(void *data)
>  out:
>  	tracecmd_msg_handle_close(tsync->msg_handle);
>  	tracecmd_tsync_free(tsync);
> -	free(tsync);
>  	close(sd);
>  

Debugging this more, this entire ending is wrong. New patch coming.

-- Steve

>  	pthread_exit(0);




[Index of Archives]     [Linux USB Development]     [Linux USB Development]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux