All of lore.kernel.org
 help / color / mirror / Atom feed
* [Kernel-janitors] add kfree(ha) to drivers/scsi/ips.c
@ 2004-06-26 13:53 Domen Puncer
  0 siblings, 0 replies; only message in thread
From: Domen Puncer @ 2004-06-26 13:53 UTC (permalink / raw)
  To: kernel-janitors

[-- Attachment #1: Type: text/plain, Size: 777 bytes --]

Hi.

This has been on TODO as long as i remember, lets try to fix it.
Compile tested.


ips_init_phase1 and ips_init_phase2 call ips_abort_init on failure, but
it doesn't free 'ha'.
'ha' should be released on ips_release.

Needs review from someone who knows stuff a bit more.


Signed-off-by: Domen Puncer <domen@coderock.org>


--- c/drivers/scsi/ips.c	2004-06-16 07:18:38.000000000 +0200
+++ a/drivers/scsi/ips.c	2004-06-26 15:36:14.000000000 +0200
@@ -729,6 +729,7 @@ ips_release(struct Scsi_Host *sh)
 	/* free IRQ */
 	free_irq(ha->irq, ha);
 
+	kfree(ha);
 	IPS_REMOVE_HOST(sh);
 	scsi_host_put(sh);
 
@@ -6946,6 +6947,7 @@ ips_abort_init(ips_ha_t * ha, int index)
 {
 	ha->active = 0;
 	ips_free(ha);
+	kfree(ha);
 	ips_ha[index] = 0;
 	ips_sh[index] = 0;
 	return -1;

[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2004-06-26 13:53 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-06-26 13:53 [Kernel-janitors] add kfree(ha) to drivers/scsi/ips.c Domen Puncer

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.