[sheepdog] [PATCH] dog/node: remove checking argument in md plug/unplug since root cause is found

Hitoshi Mitake mitake.hitoshi at lab.ntt.co.jp
Thu Aug 7 09:36:07 CEST 2014


At Thu,  7 Aug 2014 12:08:23 +0800,
Ruoyu wrote:
> 
> The purpose of the checking in the function is to ensure the
> argument is not null. But it is not needed any longer because
> the argument should be guaranteed in the framework as long as
> CMD_NEED_ARG is specified in subcommand.
> 
> Since the problem is solved in commit e6d1706, we can remove
> the checking safely.
> 
> Signed-off-by: Ruoyu <liangry at ucweb.com>
> ---
>  dog/node.c | 8 --------
>  1 file changed, 8 deletions(-)

Applied, thanks.
Hitoshi

> 
> diff --git a/dog/node.c b/dog/node.c
> index b6fd929..f9f7351 100644
> --- a/dog/node.c
> +++ b/dog/node.c
> @@ -441,14 +441,6 @@ static int do_plug_unplug(char *disks, bool plug)
>  	struct sd_rsp *rsp = (struct sd_rsp *)&hdr;
>  	int ret;
>  
> -	if (!disks)
> -		return EXIT_USAGE;
> -
> -	if (!strlen(disks)) {
> -		sd_err("Empty path isn't allowed");
> -		return EXIT_FAILURE;
> -	}
> -
>  	if (plug)
>  		sd_init_req(&hdr, SD_OP_MD_PLUG);
>  	else
> -- 
> 1.8.3.2
> 
> 
> -- 
> sheepdog mailing list
> sheepdog at lists.wpkg.org
> http://lists.wpkg.org/mailman/listinfo/sheepdog



More information about the sheepdog mailing list