[ltt-dev] [UST PATCH] Initialize _res_header variable to zero before use in process_client_cmd

Nils Carlson nils.carlson at ericsson.com
Wed Feb 23 04:11:53 EST 2011



On Wed, 23 Feb 2011, Yannick Brosseau wrote:

> This use of a non-initialized variable was detected by the valgrind test.
>
> Signed-off-by: Yannick Brosseau <yannick.brosseau at gmail.com>

Acked-by: Nils Carlson <nils.carlson at ericsson.com>
> ---
> libustconsumer/libustconsumer.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/libustconsumer/libustconsumer.c b/libustconsumer/libustconsumer.c
> index 25c84ce..c51b106 100644
> --- a/libustconsumer/libustconsumer.c
> +++ b/libustconsumer/libustconsumer.c
> @@ -621,7 +621,7 @@ static void process_client_cmd(int sock, struct ustcomm_header *req_header,
> 			       char *recvbuf, struct ustconsumer_instance *instance)
> {
> 	int result;
> -	struct ustcomm_header _res_header;
> +	struct ustcomm_header _res_header = {0};
> 	struct ustcomm_header *res_header = &_res_header;
> 	struct ustcomm_buffer_info *buf_inf;
>
> -- 
> 1.7.2.3
>
>
> _______________________________________________
> ltt-dev mailing list
> ltt-dev at lists.casi.polymtl.ca
> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>




More information about the lttng-dev mailing list