PERFORCE change 119299 for review

Matt Jacob mjacob at FreeBSD.org
Sat May 5 20:05:38 UTC 2007


http://perforce.freebsd.org/chv.cgi?CH=119299

Change 119299 by mjacob at mjexp on 2007/05/05 20:05:20

	misplaced isp_free_pcmd led to a panic at double free.

Affected files ...

.. //depot/projects/mjexp/sys/dev/isp/isp_freebsd.c#24 edit

Differences ...

==== //depot/projects/mjexp/sys/dev/isp/isp_freebsd.c#24 (text+ko) ====

@@ -2478,7 +2478,6 @@
 			    isp_watchdog, ccb);
 			break;
 		case CMD_RQLATER:
-			isp_free_pcmd(isp, ccb);
 			/*
 			 * Handle initial and subsequent loop down cases
 			 */
@@ -2495,6 +2494,7 @@
 				ccb->ccb_h.status =
 				    CAM_SEL_TIMEOUT|CAM_DEV_QFRZN;
 				xpt_freeze_devq(ccb->ccb_h.path, 1);
+				isp_free_pcmd(isp, ccb);
 				xpt_done(ccb);
 				break;
 			}


More information about the p4-projects mailing list