On Fri, Feb 10, 2006 at 05:57:51PM +0900, [EMAIL PROTECTED] wrote: > > > This patch for unionfs_d_revalidate() merges two auto variables 'err' > > and 'invalid'. And also checks the return value of > > unionfs_lookup_backend() correctly. > ::: > > --- unionfs-20060208-2346/lookup.c 9 Feb 2006 13:04:42 -0000 1.1 > > +++ unionfs-20060208-2346/lookup.c 10 Feb 2006 07:00:55 -0000 > > @@ -320,8 +320,13 @@ int unionfs_partial_lookup(struct dentry > > struct dentry *tmp; > > > > tmp = unionfs_lookup_backend(dentry, INTERPOSE_PARTIAL); > > - > > - return PTR_ERR(tmp); > > + if (!tmp) > > + return 0; > > + if (IS_ERR(tmp)) > > + return PTR_ERR(tmp); > > + /* need to change the interface */ > > + BUG_ON(1); > > + return -ENOSYS; > > } > > The last BUG_ON() should be > BUG_ON(tmp != dentry);
Applied with the modification. Thanks, Jeff. _______________________________________________ unionfs mailing list [email protected] http://www.fsl.cs.sunysb.edu/mailman/listinfo/unionfs
