Re: 2.6.24.2: 4KSTACKS + pcdrw + dm + mount -> stack overflow:ide-cd related? dm-related?

From: Jan Kara
Date: Tue Feb 26 2008 - 11:41:35 EST


On Tue 26-02-08 12:37:17, Jiri Kosina wrote:
> On Tue, 26 Feb 2008, Ingo Molnar wrote:
>
> > > + name = kmalloc(sizeof(char) * UDF_NAME_LEN, GFP_KERNEL);
> > > + fname = kmalloc(sizeof(char) * UDF_NAME_LEN, GFP_KERNEL);
> > > +
> > > + if (!name || !fname) {
> > > + *err = -ENOMEM;
> > > + return NULL;
> > > + }
> > > +
> > > if (dentry) {
> > > if (!dentry->d_name.len) {
> > > *err = -EINVAL;
> > this bit is missing i think:
> > if (name)
> > kfree(name);
> > if (fname)
> > kfree(fname);
>
> Ergh, of course, stupid me, sorry, it should be freed on all exit paths. I
> am not sending updated patch, as Jan is probably working on complete
> removal of one of those fields ... ?
Yes, I'll convert one variable to kmalloc and the other one remove
completely. Stay tuned ;).

Honza
--
Jan Kara <jack@xxxxxxx>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/