On Mon, May 11, 2009 at 11:17:56PM -0400, Theodore Tso wrote: > On Thu, May 07, 2009 at 04:09:28PM +0530, Aneesh Kumar K.V wrote: > > These buffer_heads are allocated on stack and are > > used only to make get_blocks calls. So we can set the > > b_state to 0 > > > > Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@xxxxxxxxxxxxxxxxxx> > > I've rewritten the commit description to this: > > ext4: Properly initialize the buffer_head state > > From: "Aneesh Kumar K.V" <aneesh.kumar@xxxxxxxxxxxxxxxxxx> > > These struct buffer_heads are allocated on the stack (and hence are > initialized with stack garbage). They are only used to call a > get_blocks() function, so that's mostly OK, but b_state must be > initialized to be 0. > > Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@xxxxxxxxxxxxxxxxxx> > Signed-off-by: "Theodore Ts'o" <tytso@xxxxxxx> > > Comments? > good. -aneesh -- 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
