From f40d7afa05430a19cb70b2f2a35b7a0e89451417 Mon Sep 17 00:00:00 2001
From: Warner Losh <imp@FreeBSD.org>
Date: Thu, 10 May 2001 06:55:39 +0000
Subject: [PATCH] Fix the panics for real this time.  When something can't be
 allocated, we need to delete the info from the list as well as zero out the
 res pointer we saved in the code.

Also made a few style(9) changes while I was at it.  Don't use if
(ptr) or if (!ptr), but compare against NULL.  Compare against NULL
rather than 0.  Don't have useless blocks.

There are likely other problems as well, but at least the wi based
wireless card with memory listed in its cis doesn't panic the system
when the card is inserted.
---
 sys/dev/pccard/pccard.c | 51 ++++++++++++++++++++++-------------------
 1 file changed, 27 insertions(+), 24 deletions(-)

diff --git a/sys/dev/pccard/pccard.c b/sys/dev/pccard/pccard.c
index ab3392930a45..992961e049c2 100644
--- a/sys/dev/pccard/pccard.c
+++ b/sys/dev/pccard/pccard.c
@@ -209,9 +209,10 @@ pccard_detach_card(device_t dev, int flags)
 		pccard_function_disable(pf);
 		/*
 		 * XXX must also actually delete resources created by
-		 * pccard_function_init()
+		 * pccard_function_init().  If pccard_function_init
+		 * keeps things allocated it is a bug.
 		 */
-		if (pf->dev)
+		if (pf->dev != NULL)
 			device_delete_child(dev, pf->dev);
 	}
 	return (0);
@@ -330,16 +331,14 @@ pccard_function_init(struct pccard_function *pf)
 			    SYS_RES_IOPORT, &cfe->iorid[i], start, end,
 			    cfe->iospace[i].length,
 			    rman_make_alignment_flags(cfe->iospace[i].length));
-			if (cfe->iores[i] == 0)
+			if (cfe->iores[i] == NULL)
 				goto not_this_one;
 			resource_list_add(rl, SYS_RES_IOPORT, cfe->iorid[i],
 			    rman_get_start(r), rman_get_end(r),
 			    cfe->iospace[i].length);
-			{
-				rle = resource_list_find(rl, SYS_RES_IOPORT, 
-				    cfe->iorid[i]);
-				rle->res = r;
-			}
+			rle = resource_list_find(rl, SYS_RES_IOPORT,
+			    cfe->iorid[i]);
+			rle->res = r;
 		}
 		if (cfe->num_memspace > 0) {
 			goto not_this_one;
@@ -348,17 +347,14 @@ pccard_function_init(struct pccard_function *pf)
 			cfe->irqrid = 0;
 			r = cfe->irqres = bus_alloc_resource(bus, SYS_RES_IRQ,
 			    &cfe->irqrid, 0, ~0, 1, 0);
-			if (cfe->irqres == 0)
+			if (cfe->irqres == NULL)
 				goto not_this_one;
 			resource_list_add(rl, SYS_RES_IRQ, cfe->irqrid,
 			    rman_get_start(r), rman_get_end(r), 1);
-			{
-				rle = resource_list_find(rl, SYS_RES_IRQ,
-				    cfe->irqrid);
-				rle->res = r;
-			}
+			rle = resource_list_find(rl, SYS_RES_IRQ,
+			    cfe->irqrid);
+			rle->res = r;
 		}
-		/* XXX Don't know how to deal with maxtwins */
 		/* If we get to here, we've allocated all we need */
 		pf->cfe = cfe;
 		break;
@@ -370,17 +366,24 @@ pccard_function_init(struct pccard_function *pf)
 		 * from this config entry.
 		 */
 		for (i = 0; i < cfe->num_iospace; i++) {
-			if (cfe->iores[i]) {
-				resource_list_delete(rl, SYS_RES_IOPORT, i);
+			if (cfe->iores[i] != NULL) {
 				bus_release_resource(bus, SYS_RES_IOPORT,
 				    cfe->iorid[i], cfe->iores[i]);
+				rle = resource_list_find(rl, SYS_RES_IOPORT, 
+				    cfe->iorid[i]);
+				rle->res = NULL;
+				resource_list_delete(rl, SYS_RES_IOPORT,
+				    cfe->iorid[i]);
 			}
 			cfe->iores[i] = NULL;
 		}
-		if (cfe->irqmask && cfe->irqres) {
-			resource_list_delete(rl, SYS_RES_IRQ, cfe->irqrid);
+		if (cfe->irqmask && cfe->irqres != NULL) {
 			bus_release_resource(bus, SYS_RES_IRQ,
 			    cfe->irqrid, cfe->irqres);
+			rle = resource_list_find(rl, SYS_RES_IRQ,
+			    cfe->irqrid);
+			rle->res = NULL;
+			resource_list_delete(rl, SYS_RES_IRQ, cfe->irqrid);
 			cfe->irqres = NULL;
 		}
 	}
@@ -731,7 +734,7 @@ pccard_print_resources(struct resource_list *rl, const char *name, int type,
 	printed = 0;
 	for (i = 0; i < count; i++) {
 		rle = resource_list_find(rl, type, i);
-		if (rle) {
+		if (rle != NULL) {
 			if (printed == 0)
 				printf(" %s ", name);
 			else if (printed > 0)
@@ -759,7 +762,7 @@ pccard_print_child(device_t dev, device_t child)
 	retval += bus_print_child_header(dev, child);
 	retval += printf(" at");
 
-	if (devi) {
+	if (devi != NULL) {
 		pccard_print_resources(rl, "port", SYS_RES_IOPORT,
 		    PCCARD_NPORT, "%#lx");
 		pccard_print_resources(rl, "iomem", SYS_RES_MEMORY,
@@ -812,12 +815,12 @@ pccard_get_resource(device_t dev, device_t child, int type, int rid,
 	struct resource_list_entry *rle;
 
 	rle = resource_list_find(rl, type, rid);
-	if (!rle)
+	if (rle == NULL)
 		return (ENOENT);
 
-	if (startp)
+	if (startp != NULL)
 		*startp = rle->start;
-	if (countp)
+	if (countp != NULL)
 		*countp = rle->count;
 
 	return (0);