[EMAIL PROTECTED] wrote:
>
> static inline int autofs4_ispending(struct dentry *dentry)
> {
> struct autofs_info *inf = autofs4_dentry_ino(dentry);
> + int pending;
>
> - return (dentry->d_flags & DCACHE_AUTOFS_PENDING) ||
> - (inf != NULL && inf->flags & AUTOFS_INF_EXPIRING);
> + if (dentry->d_flags & DCACHE_AUTOFS_PENDING)
> + return 1;
> +
> + if (inf) {
> + spin_lock(&inf->sbi->fs_lock);
> + pending = inf->flags & AUTOFS_INF_EXPIRING;
> + spin_unlock(&inf->sbi->fs_lock);
> + }
> +
> + return pending;
This can obviously return an uninitialised variable.
Were these patches very well tested?
-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
