[PATCH v2] ufs: qcom: Properly clear hba priv on failure

2016-11-19 Thread Bjorn Andersson
ufs_qcom_init() sets the hba priv data before attempting to acquire the phy handle, so make sure to clear this in the case of an error. Failing to do this will make ufs_qcom_setup_clocks() operate on the uninitalized host object. Signed-off-by: Bjorn Andersson --- drivers/scsi/ufs/ufs-qcom.c | 6

Re: [PATCH] ufs: qcom: Properly clear hba priv on failure

2016-11-19 Thread Bjorn Andersson
On Sat 19 Nov 12:30 PST 2016, Subhash Jadavani wrote: > On 2016-11-18 12:55, Bjorn Andersson wrote: > >In the case where we fail to acquire the phy the hba priv will be set > >already, so during cleanup ufs_qcom_setup_clocks() will dereference the > >now free, but still "valid looking" pointer "ho

[Bug 179341] mpt3sas: LSISAS3008 don't see Intel 540s SSD

2016-11-19 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=179341 --- Comment #3 from Badalyan Vyacheslav --- Created attachment 245171 --> https://bugzilla.kernel.org/attachment.cgi?id=245171&action=edit Dmes wiyh loh -- You are receiving this mail because: You are watching someone on the CC list of the bu

Re: [PATCH] ufs: qcom: Properly clear hba priv on failure

2016-11-19 Thread Subhash Jadavani
On 2016-11-18 12:55, Bjorn Andersson wrote: In the case where we fail to acquire the phy the hba priv will be set already, so during cleanup ufs_qcom_setup_clocks() will dereference the now free, but still "valid looking" pointer "host". host (ufs_qcom_host) was allocated with devm_kzalloc() so

[PATCH] xen-scsifront: Add a missing call to kfree

2016-11-19 Thread Quentin Lambert
Most error branches following the call to kmalloc contain a call to kfree. This patch add these calls where they are missing. This issue was found with Hector. Signed-off-by: Quentin Lambert --- drivers/scsi/xen-scsifront.c |1 + 1 file changed, 1 insertion(+) --- a/drivers/scsi/xen-scsif

[PATCH] [SCSI] dpt_i2o: Add a missing call to kfree

2016-11-19 Thread Quentin Lambert
Most error branches following the call to kzalloc contain a call to kfree. This patch add these calls where they are missing. This issue was found with Hector. Signed-off-by: Quentin Lambert --- drivers/scsi/dpt_i2o.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) --- a/drivers/s

[PATCH] scsi: pmcraid: Add missing resource releases

2016-11-19 Thread Quentin Lambert
Most error branches following the call to pmcraid_get_free_cmd contain a call to pmcraid_return_cmd. This patch add these calls where they are missing. Moreover, most error branches following the call to class_create contain a call to class_destroy. This patch add these calls where they are missin

[PATCH] isci: Add a missing call to pci_unmap_biosrom

2016-11-19 Thread Quentin Lambert
Most error branches following the call to pci_map_biosrom contain a call to pci_unmap_biosrom. This patch add these calls where they are missing. This issue was found with Hector. Signed-off-by: Quentin Lambert --- drivers/scsi/isci/probe_roms.c |1 + 1 file changed, 1 insertion(+) --- a/

[PATCH] scsi: hisi_sas: Add a missing call to kfree

2016-11-19 Thread Quentin Lambert
Most error branches following the call to hisi_sas_shost_alloc contain a call to kfree. This patch add these calls where they are missing. This issue was found with Hector. Signed-off-by: Quentin Lambert --- drivers/scsi/hisi_sas/hisi_sas_main.c |6 -- 1 file changed, 4 insertions(+),

[PATCH] cxgb4i: Add a missing call to neigh_release

2016-11-19 Thread Quentin Lambert
Most error branches following the call to dst_neigh_lookup contain a call to neigh_release. This patch add these calls where they are missing. This issue was found with Hector. Signed-off-by: Quentin Lambert --- drivers/scsi/cxgbi/cxgb4i/cxgb4i.c |2 +- 1 file changed, 1 insertion(+), 1 de

[PATCH] scsi: aic94xx: Add a missing call to kfree

2016-11-19 Thread Quentin Lambert
Most error branches following the call to kzalloc contain a call to kfree. This patch add these calls where they are missing. This issue was found with Hector. Signed-off-by: Quentin Lambert --- drivers/scsi/aic94xx/aic94xx_hwi.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) ---