Re: [PATCH] fix use-after-free in sbp2.c
From: Ben Collins (bcollins_at_debian.org)
Date: 12/03/03
- Previous message: pinotj_at_club-internet.fr: "Re: Re: Re: Re: [Oops] i386 mm/slab.c (cache_flusharray)"
- In reply to: Martin Waitz: "[PATCH] fix use-after-free in sbp2.c"
- Next in thread: Martin Waitz: "Re: [PATCH] fix use-after-free in sbp2.c"
- Reply: Martin Waitz: "Re: [PATCH] fix use-after-free in sbp2.c"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Tue, 2 Dec 2003 18:31:25 -0500 To: Martin Waitz <tali@admingilde.org>
On Mon, Dec 01, 2003 at 10:02:12PM +0100, Martin Waitz wrote:
> hi :)
>
> when using some checking code (CONFIG_DEBUG_{SLAB,SPINLOCK_SLEEP},
> sbp2 fails to log in into my external hd enclosure.
>
> that is because sbp2_agent_reset sends a packet and waits
> for its delivery.
> however, the function used to create the packet activates
> auto-destruct of the packet via hpsb_set_packet_complete_task.
> thus, the semaphore used for synchronization is destroyed
> while the sending task is waiting.
>
> the following patch (against -test11) fixes sbp2 for me
Could you test what's in our repo first? We've already fixed this, but
it was done in a way different way than you did (we got rid of the
semaphore).
-- Debian - http://www.debian.org/ Linux 1394 - http://www.linux1394.org/ Subversion - http://subversion.tigris.org/ WatchGuard - http://www.watchguard.com/ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- Previous message: pinotj_at_club-internet.fr: "Re: Re: Re: Re: [Oops] i386 mm/slab.c (cache_flusharray)"
- In reply to: Martin Waitz: "[PATCH] fix use-after-free in sbp2.c"
- Next in thread: Martin Waitz: "Re: [PATCH] fix use-after-free in sbp2.c"
- Reply: Martin Waitz: "Re: [PATCH] fix use-after-free in sbp2.c"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
|
