Mercurial > illumos > illumos-gate
changeset 13619:1ca8a76b1e84
1802 qlge may wildly overrun a debug buffer
Reviewed by: Jason King <jason.brian.king@gmail.com>
Reviewed by: Joshua M. Clulow <josh@sysmgr.org>
Reviewed by: Robert Mustacchi <rm@joyent.com>
Approved by: Albert Lee <trisk@nexenta.com>
author | Richard Lowe <richlowe@richlowe.net> |
---|---|
date | Mon, 16 May 2011 01:09:06 +0100 |
parents | c6ae14a341e8 |
children | 97ffb0d8de52 |
files | usr/src/uts/common/io/fibre-channel/fca/qlge/qlge_dbg.c |
diffstat | 1 files changed, 6 insertions(+), 6 deletions(-) [+] |
line wrap: on
line diff
--- a/usr/src/uts/common/io/fibre-channel/fca/qlge/qlge_dbg.c Sat Feb 04 05:44:53 2012 +0000 +++ b/usr/src/uts/common/io/fibre-channel/fca/qlge/qlge_dbg.c Mon May 16 01:09:06 2011 +0100 @@ -2552,25 +2552,25 @@ } } - for (i = 0x1c00; i <= 0x1c1f; i++) { + for (i = 0x1c00, j = 0; i <= 0x1c1f; i++) { status = ql_read_serdes_reg(qlge, i, - &mpi_coredump->serdes_xfi_hss_tx[i]); + &mpi_coredump->serdes_xfi_hss_tx[j++]); if (status != DDI_SUCCESS) { goto err; } } - for (i = 0x1c40; i <= 0x1c5f; i++) { + for (i = 0x1c40, j = 0; i <= 0x1c5f; i++) { status = ql_read_serdes_reg(qlge, i, - &mpi_coredump->serdes_xfi_hss_rx[i]); + &mpi_coredump->serdes_xfi_hss_rx[j++]); if (status != DDI_SUCCESS) { goto err; } } - for (i = 0x1e00; i <= 0x1e1f; i++) { + for (i = 0x1e00, j = 0; i <= 0x1e1f; i++) { status = ql_read_serdes_reg(qlge, i, - &mpi_coredump->serdes_xfi_hss_pll[i]); + &mpi_coredump->serdes_xfi_hss_pll[j++]); if (status != DDI_SUCCESS) { goto err; }