[ofa-general] [PATCH] IB/core - remove redundant NULL pointer check in ib_mad_recv_done_handler()

[Hal Rosenstock]

On Wed, 2007-10-17 at 18:06 -0700, Ralph Campbell wrote:
> In ib_mad_recv_done_handler(), the response pointer is checked for
> NULL after allocating it.  It is then checked again in the local
> process_mad() path but there is no possibility of it changing
> in between.
> 
> Signed-off-by: Ralph Campbell <ralph.campbell at qlogic.com>

Yes, this appears to be no longer needed (and as Suri pointed out was
dropped part of a previous patch). Good catch.

Acked-by: Hal Rosenstock <hal at xsigo.com>

> diff --git a/drivers/infiniband/core/mad.c b/drivers/infiniband/core/mad.c
> index 6f42877..f82900d 100644
> --- a/drivers/infiniband/core/mad.c
> +++ b/drivers/infiniband/core/mad.c
> @@ -1931,15 +1931,6 @@ local:
>  	if (port_priv->device->process_mad) {
>  		int ret;
>  
> -		if (!response) {
> -			printk(KERN_ERR PFX "No memory for response MAD\n");
> -			/*
> -			 * Is it better to assume that
> -			 * it wouldn't be processed ?
> -			 */
> -			goto out;
> -		}
> -
>  		ret = port_priv->device->process_mad(port_priv->device, 0,
>  						     port_priv->port_num,
>  						     wc, &recv->grh,
> 
> 
> _______________________________________________
> general mailing list
> general at lists.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
> 
> To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general