[lttng-dev] [PATCH lttng-tools 2/3] Fix: Typo in LTTNG_FILTER_MAX_LEN

[Mathieu Desnoyers]

* Christian Babeux (christian.babeux at efficios.com) wrote:
> 
> Signed-off-by: Christian Babeux <christian.babeux at efficios.com>
> ---
>  src/bin/lttng-sessiond/main.c            | 2 +-
>  src/common/sessiond-comm/sessiond-comm.h | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/bin/lttng-sessiond/main.c b/src/bin/lttng-sessiond/main.c
> index 318da74..730ac65 100644
> --- a/src/bin/lttng-sessiond/main.c
> +++ b/src/bin/lttng-sessiond/main.c
> @@ -2659,7 +2659,7 @@ skip_domain:
>  	{
>  		struct lttng_filter_bytecode *bytecode;
>  
> -		if (cmd_ctx->lsm->u.filter.bytecode_len > 65336) {
> +		if (cmd_ctx->lsm->u.filter.bytecode_len > LTTNG_FILTER_MAX_LEN) {
>  			ret = LTTNG_ERR_FILTER_INVAL;
>  			goto error;
>  		}
> diff --git a/src/common/sessiond-comm/sessiond-comm.h b/src/common/sessiond-comm/sessiond-comm.h
> index 32ce384..ff22875 100644
> --- a/src/common/sessiond-comm/sessiond-comm.h
> +++ b/src/common/sessiond-comm/sessiond-comm.h
> @@ -208,7 +208,7 @@ struct lttcomm_session_msg {
>  	} u;
>  };
>  
> -#define LTTNG_FILTER_MAX_LEN	65336
> +#define LTTNG_FILTER_MAX_LEN	65535

This should be 65536.

Thanks,

Mathieu

>  
>  /*
>   * Filter bytecode data. The reloc table is located at the end of the
> -- 
> 1.7.11.4
> 

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com