Re: [patch] eCryptfs: remove unneeded checks in virt_to_scatterlist()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2013-01-26 10:48:42, Dan Carpenter wrote:
> This is always called with a valid "sg" pointer.  My static checker
> complains because the call to sg_init_table() dereferences "sg" before
> we reach the checks.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Thanks, Dan! I've tested and pushed this patch to the eCryptfs next
branch.

Tyler

> 
> diff --git a/fs/ecryptfs/crypto.c b/fs/ecryptfs/crypto.c
> index a7b0c2d..d5c25db 100644
> --- a/fs/ecryptfs/crypto.c
> +++ b/fs/ecryptfs/crypto.c
> @@ -301,17 +301,14 @@ int virt_to_scatterlist(const void *addr, int size, struct scatterlist *sg,
>  	while (size > 0 && i < sg_size) {
>  		pg = virt_to_page(addr);
>  		offset = offset_in_page(addr);
> -		if (sg)
> -			sg_set_page(&sg[i], pg, 0, offset);
> +		sg_set_page(&sg[i], pg, 0, offset);
>  		remainder_of_page = PAGE_CACHE_SIZE - offset;
>  		if (size >= remainder_of_page) {
> -			if (sg)
> -				sg[i].length = remainder_of_page;
> +			sg[i].length = remainder_of_page;
>  			addr += remainder_of_page;
>  			size -= remainder_of_page;
>  		} else {
> -			if (sg)
> -				sg[i].length = size;
> +			sg[i].length = size;
>  			addr += size;
>  			size = 0;
>  		}

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux