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;
 		}