We call kfree(req_task) after we go to abort so it isn't needed here. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> diff --git a/drivers/block/mtip32xx/mtip32xx.c b/drivers/block/mtip32xx/mtip32xx.c index 847b8ff..0d23d8c 100644 --- a/drivers/block/mtip32xx/mtip32xx.c +++ b/drivers/block/mtip32xx/mtip32xx.c @@ -1674,7 +1674,6 @@ static int exec_drive_taskfile(struct driver_data *dd, intotal = compat_tasksize + req_task->out_size; } else { if (copy_from_user(req_task, buf, tasksize)) { - kfree(req_task); err = -EFAULT; goto abort; } -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html