Subject: [PATCH] ARM: kprobes: Fix compilation error caused by superfluous '*'

From: [email-masked] (Wang Nan)
Date: Tue, 10 Feb 2015 16:13:17 +0800

On 2015/2/10 15:27, Jon Medhurst (Tixy) wrote:
> There is a superfluous '*' in the definition of kprobe_decode_insn_t
> which on older versions of GCC (4.2.4) causes the compilation error:
> 
> In file included from arch/arm/probes/kprobes/core.c:37:
> arch/arm/probes/kprobes/core.h:43: error: '[*]' not allowed in other than a declaration
> 
> Fix this by removing the unneeded character.
> 

Hi Tixy,

I checked C spec when writing this code and found [*] should be a standard way to
represent array of unspecified number elements. But I forgot there are old gccs
which don't support it.

Thank you for your fix!

> Reported-by: Janusz U?ycki <[email-masked]>
> Signed-off-by: Jon Medhurst <[email-masked]>
> ---
> 
> Russell, I also applied this to my branch that you pulled which
> introduced the bug...
> git://git.linaro.org/people/tixy/kernel.git kprobes
> 
>  arch/arm/probes/kprobes/core.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/probes/kprobes/core.h b/arch/arm/probes/kprobes/core.h
> index b3036c5..ec5d1f2 100644
> --- a/arch/arm/probes/kprobes/core.h
> +++ b/arch/arm/probes/kprobes/core.h
> @@ -40,7 +40,7 @@ typedef enum probes_insn (kprobe_decode_insn_t)(probes_opcode_t,
>  						struct arch_probes_insn *,
>  						bool,
>  						const union decode_action *,
> -						const struct decode_checker *[*]);
> +						const struct decode_checker *[]);
>  
>  #ifdef CONFIG_THUMB2_KERNEL
>  
>