On Tue, 2009-01-13 at 13:51 +0200, Jouni Malinen wrote: > + if (params->ies) { > + ies = kmalloc(params->ies_len, GFP_KERNEL); > + if (ies == NULL) > + return -ENOMEM; > + ies_len = params->ies_len; > + memcpy(ies, params->ies, ies_len); kmemdup? Might also make sense to not try the allocation if we're going to refuse it anyway, but that might be complicated to do and probably doesn't matter. Otherwise looks good to me. Acked-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> johannes
Attachment:
signature.asc
Description: This is a digitally signed message part