Re: [PATCH] slcanpty: declare command buffer static

[Oliver Hartkopp <socketcan@hartkopp.net>]

On 13.01.2014 14:29, yegorslists@googlemail.com wrote:
> From: Yegor Yefremov <yegorslists@googlemail.com>
> 
> ASCII command buffer char buf[200] must be declared as static,
> because it holds incomplete commands between pty2can() calls.
> Without static it is not guaranteed, that buf retains the same
> data between calls.
> 
> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

Acked-by: Oliver Hartkopp <socketcan@hartkopp.net>

Hi Yegor,

indeed the rxoffset is a static variable to handle incomplete ASCII commands
(in buf[]) and therefore it makes no sense when the referenced buffer would
not be static then ...

Thanks for catching this.

Best regards,
Oliver

> ---
>  slcanpty.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/slcanpty.c b/slcanpty.c
> index 2c1cabe..c062a02 100644
> --- a/slcanpty.c
> +++ b/slcanpty.c
> @@ -68,7 +68,7 @@ int pty2can(int pty, int socket, struct can_filter *fi,
>  {
>  	int nbytes;
>  	char cmd;
> -	char buf[200];
> +	static char buf[200];
>  	char replybuf[10]; /* for answers to received commands */
>  	int ptr;
>  	struct can_frame frame;
>