On Tue, Feb 16, 2016 at 12:49 PM, Gerd Hoffmann <kraxel@xxxxxxxxxx> wrote: > + if (cmd->relocs_num > 65536) > + return -EINVAL; > reloc_info = kmalloc(sizeof(struct qxl_reloc_info) * cmd->relocs_num, GFP_KERNEL); > if (!reloc_info) > return -ENOMEM; Why not just use one of the array alloc functions like kcalloc? -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html