]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
scsi: be2iscsi: Revert "Fix a theoretical leak in beiscsi_create_eqs()"
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 3 Dec 2020 12:18:26 +0000 (15:18 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 29 Dec 2020 12:46:44 +0000 (13:46 +0100)
commitbb4ee532e955c8d3c66e57ce5dd72027994664bd
treeabdce61d4c2c9d1bd43abb0af3f4ffab5ef1bb7c
parent436b01c4447e74d729a3ed6090ae223bb7d56ffc
scsi: be2iscsi: Revert "Fix a theoretical leak in beiscsi_create_eqs()"

commit eeaf06af6f87e1dba371fbe42674e6f963220b9c upstream.

My patch caused kernel Oopses and delays in boot.  Revert it.

The problem was that I moved the "mem->dma = paddr;" before the call to
be_fill_queue().  But the first thing that the be_fill_queue() function
does is memset the whole struct to zero which overwrites the assignment.

Link: https://lore.kernel.org/r/X8jXkt6eThjyVP1v@mwanda
Fixes: 38b2db564d9a ("scsi: be2iscsi: Fix a theoretical leak in beiscsi_create_eqs()")
Cc: stable <stable@vger.kernel.org>
Reported-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/scsi/be2iscsi/be_main.c