Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pull] master from torvalds:master #1

Merged
merged 23 commits into from
Nov 14, 2018
Merged
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
5841734
scsi: target/core: Avoid that a kernel oops is triggered when COMPARE…
bvanassche Nov 6, 2018
f8f4adc
scsi: myrb: fix sprintf buffer overflow warning
arndb Nov 2, 2018
f8d2943
scsi: lpfc: fix remoteport access
arndb Nov 2, 2018
77409c4
scsi: myrs: avoid stack overflow warning
arndb Nov 2, 2018
a3ecf48
scsi: myrs: only build on little-endian platforms
arndb Nov 2, 2018
e34ff8e
scsi: hisi_sas: Remove set but not used variable 'dq_list'
Oct 26, 2018
0d52e64
scsi: qla2xxx: Fix a typo in MODULE_PARM_DESC
standby24x7 Oct 28, 2018
96edebd
scsi: NCR5380: Return false instead of NULL
Oct 24, 2018
86d4d06
parisc: Revert "Release spinlocks using ordered store"
danglin44 Nov 6, 2018
aca49ee
Revert "scsi: ufs: Disable blk-mq for now"
martinkpetersen Nov 6, 2018
f635e48
scsi: qla2xxx: Initialize port speed to avoid setting lower speed
Nov 6, 2018
fbb974b
rtc: cmos: Do not export alarm rtc_ops when we do not support alarms
jwrdegoede Sep 4, 2018
7ce9a99
rtc: hctosys: Add missing range error reporting
Nov 5, 2018
9bde0af
rtc: pcf2127: fix a kmemleak caused in pcf2127_i2c_gather_write
xulinsun Nov 6, 2018
1e9c75f
mnt: fix __detach_mounts infinite loop
bcodding-rh Oct 3, 2018
5df7a99
ARM: 8810/1: vfp: Fix wrong assignement to ufp_exc
Nov 8, 2018
0d76bcc
Revert "ACPI/PCI: Pay attention to device-specific _PXM node values"
bjorn-helgaas Nov 13, 2018
e2f8b47
Merge branch 'spectre' of git://git.armlinux.org.uk/~rmk/linux-arm
torvalds Nov 14, 2018
857c34c
Merge branch 'parisc-4.20-3' of git://git.kernel.org/pub/scm/linux/ke…
torvalds Nov 14, 2018
47e624c
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel…
torvalds Nov 14, 2018
b7bbf99
Merge tag 'rtc-4.20-2' of git://git.kernel.org/pub/scm/linux/kernel/g…
torvalds Nov 14, 2018
dbcec2e
Merge tag 'scsi-fixes' of git://git.kernel.org/pub/scm/linux/kernel/g…
torvalds Nov 14, 2018
d41217a
Merge tag 'pci-v4.20-fixes-1' of git://git.kernel.org/pub/scm/linux/k…
torvalds Nov 14, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
scsi: lpfc: fix remoteport access
The addition of a spinlock in lpfc_debugfs_nodelist_data() introduced
a bug that lets us not skip NULL pointers correctly, as noticed by
gcc-8:

drivers/scsi/lpfc/lpfc_debugfs.c: In function 'lpfc_debugfs_nodelist_data.constprop':
drivers/scsi/lpfc/lpfc_debugfs.c:728:13: error: 'nrport' may be used uninitialized in this function [-Werror=maybe-uninitialized]
   if (nrport->port_role & FC_PORT_ROLE_NVME_INITIATOR)

This changes the logic back to what it was, while keeping the added
spinlock.

Fixes: 9e21017 ("scsi: lpfc: Synchronize access to remoteport via rport")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
  • Loading branch information
arndb authored and martinkpetersen committed Nov 6, 2018
commit f8d294324598ec85bea2779512e48c94cbe4d7c6
2 changes: 2 additions & 0 deletions drivers/scsi/lpfc/lpfc_debugfs.c
Original file line number Diff line number Diff line change
Expand Up @@ -698,6 +698,8 @@ lpfc_debugfs_nodelist_data(struct lpfc_vport *vport, char *buf, int size)
rport = lpfc_ndlp_get_nrport(ndlp);
if (rport)
nrport = rport->remoteport;
else
nrport = NULL;
spin_unlock(&phba->hbalock);
if (!nrport)
continue;
Expand Down