[lttng-dev] [PATCH lttng-tools] Fix: Remove overly restrictive asserts in create_session()

dgoulet at efficios.com dgoulet at efficios.com
Wed Mar 19 13:28:28 EDT 2014


Merged and the two subsequent fixes also!

David

On 19 Mar (11:25:26), Jérémie Galarneau wrote:
> The current assertions only allow sessions containing all three
> domain types to be created. The pointers are already checked at
> ln 1242.
> 
> Signed-off-by: Jérémie Galarneau <jeremie.galarneau at efficios.com>
> ---
>  src/common/config/config.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/src/common/config/config.c b/src/common/config/config.c
> index a890c9d..13af2f1 100644
> --- a/src/common/config/config.c
> +++ b/src/common/config/config.c
> @@ -1186,9 +1186,6 @@ int create_session(const char *name,
>  	xmlNodePtr consumer_output_node;
>  
>  	assert(name);
> -	assert(kernel_domain);
> -	assert(ust_domain);
> -	assert(jul_domain);
>  
>  	if (output_node) {
>  		consumer_output_node = xmlFirstElementChild(output_node);
> -- 
> 1.9.0
> 
-------------- 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/20140319/4e34abe7/attachment.pgp>


More information about the lttng-dev mailing list