Re: [PATCH 9/12: eCryptfs] Inode operations

From: Andreas Schwab
Date: Thu Nov 03 2005 - 19:24:07 EST


James Morris <jmorris@xxxxxxxxx> writes:

> On Wed, 2 Nov 2005, Phillip Hellewell wrote:
>
>> +static int grow_file(struct dentry *ecryptfs_dentry, struct file *lower_file,
>> + struct inode *inode, struct inode *lower_inode)
>> +{
>> + int rc = 0;
>> + struct file fake_file;
>> + memset(&fake_file, 0, sizeof(fake_file));
>
>
> You don't need these initializations, bss is always initialized to zero
> in this environment.

Automatic variables are not related to the bss segment.

Andreas.

--
Andreas Schwab, SuSE Labs, schwab@xxxxxxx
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
-
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/