On Wed, 2013-10-09 at 09:13 +0000, Undekari, Sunil Dutt wrote: > > It's just a question of where to address this - in the kernel or the supplicant. In a way, I'd rather have the kernel reject it and do the policy of ignoring invalid stuff in the supplicant. > The above statement from you made me to rethink for the following check in the patch sent. > > + if (params->supported_oper_classes_len < 2 || > + params->supported_oper_classes_len > 253) > + return -EINVAL; > > Are you fine to return a failure for the invalid length for the supported_oper_classes_len? If yes, then shouldn't we also do the similar check for invalid supported_channels_len ( < 2 ). Certainly works for me - but there should also be something like % 2 I guess johannes -- 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
