Subject: Re: [Tux3] logmap inode fix

From: OGAWA Hirofumi <hirofumi@...> · Date: Wednesday, January 7, 2009 - 3:37 am

Daniel Phillips  writes:
> The iattr passed here seem to be uninitialized:

quoted text>

> 	sbi->logmap = tux_new_inode(sbi->rootdir, &iattr, 0);

>

> Why not pass &(struct iattr){ }?
I think "static iattr" is more prefer, it would be in .bss, and it would

be zeroed by initialization of program.

--

OGAWA Hirofumi 
_______________________________________________

Tux3 mailing list

Tux3@tux3.org

http://mailman.tux3.org/cgi-bin/mailman/listinfo/tux3


linux-kernel:
Parag Warudkar	BUG: soft lockup - CPU#1 stuck for 15s! [swapper:0]
Tarkan Erimer	Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3
Bart Van Assche	Integration of SCST in the mainstream Linux kernel
Greg Kroah-Hartman	[PATCH 001/196] Chinese: Add the known_regression URI to the HOWTO
git:

linux-netdev:
Gerrit Renker	[PATCH 27/37] dccp: Integration of dynamic feature activation - part 2 (server side)
David Miller	Re: [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock().
Arjan van de Ven	Re: [GIT]: Networking
David Miller	Re: [BUG] New Kernel Bugs
openbsd-misc:

Re: [Tux3] logmap inode fix

Online users