Re: [patch] caif: remove unneeded null check in caif_connect()

From: Sjur BrÃndeland
Date: Mon May 24 2010 - 15:56:12 EST


Dan Carpenter <error27@xxxxxxxxx> wrote:
> We already dereferenced uaddr towards the start of the function when we
> checked that "uaddr->sa_family != AF_CAIF". ÂBoth the check here and the
> earlier check were added in bece7b2398d0: "caif: Rewritten socket
> implementation". ÂBefore that patch, we assumed that we recieved a valid
> pointer for uaddr, and based on that, I have removed this check.
>
> Signed-off-by: Dan Carpenter <error27@xxxxxxxxx>
Thanks Looks good,
Acked-by: Sjur Braendeland <sjur.brandeland@xxxxxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/