On Wed, Jul 06, 2011 at 12:02:34AM +0200, Arend van Spriel wrote: > @@ -317,14 +312,16 @@ brcmf_c_iovar_op(struct brcmf_pub *drvr, const char *name, > > BRCMF_TRACE(("%s: Enter\n", __func__)); > > - ASSERT(name); > - ASSERT(len >= 0); > + if (name == NULL || len <= 0) > + return -EINVAL; > In the original code len == 0 was OK but now we return -EINVAL. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html