[lttng-dev] [PATCH lttng-tools] Fix: initialize the cur_event variable before using it

[lttng-dev] [PATCH lttng-tools] Fix: initialize the cur_event variable before using it

[Jérémie Galarneau]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1061 bytes --]

Merged, thanks!

Jérémie

On Tue, May 17, 2016 at 11:52 AM, Jonathan Rajotte
<jonathan.rajotte-julien at efficios.com> wrote:
> CID 1243041 (#1 of 1): Uninitialized scalar variable (UNINIT)
> uninit_use_in_call: Using uninitialized element of array *cur_event.name when
> calling strcmp.
>
> Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien at efficios.com>
> ---
>  src/bin/lttng/commands/list.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/src/bin/lttng/commands/list.c b/src/bin/lttng/commands/list.c
> index 0dd74e5..418fccf 100644
> --- a/src/bin/lttng/commands/list.c
> +++ b/src/bin/lttng/commands/list.c
> @@ -621,6 +621,8 @@ static int mi_list_ust_event_fields(struct lttng_event_field *fields, int count,
>         int event_element_open = 0;
>         struct lttng_event cur_event;
>
> +       memset(&cur_event, 0, sizeof(cur_event));
> +
>         /* Open domains element */
>         ret = mi_lttng_domains_open(writer);
>         if (ret) {
> --
> 2.7.4
>



-- 
Jérémie Galarneau
EfficiOS Inc.
http://www.efficios.com