Re: [PATCH 4/5] tmem: Pointless check of NULL against an array

[Andrew Cooper <andrew.cooper3@citrix.com>]

On 25/11/13 17:00, Konrad Rzeszutek Wilk wrote:
> CID 1055632 tells us that we are comparing an array to a NULL
> which is indeed silly. Lets not do it.
>
> CC: Bob Liu <bob.liu@oracle.com>
> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> ---
>  xen/common/tmem.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/xen/common/tmem.c b/xen/common/tmem.c
> index 3bc35fd..1ebed8a 100644
> --- a/xen/common/tmem.c
> +++ b/xen/common/tmem.c
> @@ -1798,8 +1798,6 @@ static int do_tmem_destroy_pool(uint32_t pool_id)
>      struct client *client = tmem_client_from_current();
>      struct tmem_pool *pool;
>  
> -    if ( client->pools == NULL )
> -        return 0;

client->pools certainly can't be null, but client certainly can.  This
should be "if ( !client )"

~Andrew

>      if ( pool_id >= MAX_POOLS_PER_DOMAIN )
>          return 0;
>      if ( (pool = client->pools[pool_id]) == NULL )