On 09/04/2011 09:57 PM, Ted Ts'o wrote: > On Thu, Sep 01, 2011 at 12:00:52AM +0800, Tao Ma wrote: >> From: Tao Ma <boyu.mt@xxxxxxxxxx> >> >> For all the callers of dx_probe, frame_in is passed in uninitialized. >> So we should set frame->bh to NULL in the beginning of while, not >> the end. Otherwise, the first "goto fail2" will try to brelse(frame->bh) >> while frame->bh is still invalid. > > There's a "frame->bh = NULL" statement at the very beginning of > dx_probe(). Shouldn't that do the trick? Oh, sorry for the false alarm. Thanks Tao -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html