[lttng-dev] [PATCH lttng-tools] Fix: Don't report an error if no sessions are found with default options

David Goulet dgoulet at efficios.com
Mon May 12 10:17:53 EDT 2014


Merged!

And all three other fixes that followed this one:

Print an error message when a session fails to load
Fix: Possible out-of-bounds read
Fix: Ressource leak, missing fclose

Cheers!
David

On 08 May (12:12:28), Jérémie Galarneau wrote:
> Signed-off-by: Jérémie Galarneau <jeremie.galarneau at efficios.com>
> ---
>  src/common/config/config.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/src/common/config/config.c b/src/common/config/config.c
> index 8b34d02..df6f6a3 100644
> --- a/src/common/config/config.c
> +++ b/src/common/config/config.c
> @@ -2544,5 +2544,12 @@ int config_load_session(const char *path, const char *session_name,
>  	}
>  end:
>  	fini_session_config_validation_ctx(&validation_ctx);
> +	if (ret == -LTTNG_ERR_LOAD_SESSION_NOENT && !session_name && !path) {
> +		/*
> +		 * Don't report an error if no sessions are found when called
> +		 * without a session_name or a search path.
> +		 */
> +		ret = 0;
> +	}
>  	return ret;
>  }
> -- 
> 1.9.2
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 603 bytes
Desc: Digital signature
URL: <http://lists.lttng.org/pipermail/lttng-dev/attachments/20140512/f3e436fa/attachment.sig>


More information about the lttng-dev mailing list