Re: [PATCH] libata: clear drvdata in ata_host_release(), take#2

2007-03-01 Thread Jeff Garzik
Tejun Heo wrote: Clearing drvdata in ->remove_one causes NULL pointer deference. Clear drvdata only in ata_host_release() after all resources are freed. Signed-off-by: Tejun Heo <[EMAIL PROTECTED]> --- pata_pcmcia is fixed by Alan's patch. This fixes other drivers and make libata always clear

[PATCH] libata: clear drvdata in ata_host_release(), take#2

2007-02-27 Thread Tejun Heo
Clearing drvdata in ->remove_one causes NULL pointer deference. Clear drvdata only in ata_host_release() after all resources are freed. Signed-off-by: Tejun Heo <[EMAIL PROTECTED]> --- pata_pcmcia is fixed by Alan's patch. This fixes other drivers and make libata always clear drvdata after detac

Re: [PATCH] libata: clear drvdata in ata_host_release()

2007-02-25 Thread Alan
On Sat, 24 Feb 2007 20:51:27 -0500 Jeff Garzik <[EMAIL PROTECTED]> wrote: > Tejun Heo wrote: > > ata_host_release() uses drvdata to determine ata_host to release and > > clearing drvdata in ->remove_one causes NULL pointer deference. Clear > > drvdata only in ata_host_release() after all resource

Re: [PATCH] libata: clear drvdata in ata_host_release()

2007-02-24 Thread Jeff Garzik
Tejun Heo wrote: ata_host_release() uses drvdata to determine ata_host to release and clearing drvdata in ->remove_one causes NULL pointer deference. Clear drvdata only in ata_host_release() after all resources are freed. This bug was first analyzed by Alan Cox for pata_pcmcia. Signed-off-by: