Applied.

Charles

On Fri, 2005-11-18 at 11:48 -0500, Shaya Potter wrote:
> here's the code
> 
>       lock_dentry(parent);
>         bstart = dbstart(parent);
>         bend = dbend(parent);
>         wh_dentry = ERR_PTR(-ENOENT);
>         for (bindex = bstart; bindex <= bend; bindex++) {
>                 hidden_parent = dtohd_index(parent, bindex);
>                 wh_dentry = LOOKUP_ONE_LEN(whname, hidden_parent,
> whlen);
> 
> 
> hidden_parent can be null!  I'm thinking right thing to do is this
> patch, perhaps I'm wrong
> 
> --- rename.c.bak        2005-11-18 11:49:02.000000000 +0000
> +++ rename.c    2005-11-18 11:49:28.000000000 +0000
> @@ -802,6 +802,9 @@
>         wh_dentry = ERR_PTR(-ENOENT);
>         for (bindex = bstart; bindex <= bend; bindex++) {
>                 hidden_parent = dtohd_index(parent, bindex);
> +               if (!hidden_parent) {
> +                       continue;
> +               }
>                 wh_dentry = LOOKUP_ONE_LEN(whname, hidden_parent,
> whlen);
>                 if (IS_ERR(wh_dentry))
>                         continue;
> 
> _______________________________________________
> unionfs mailing list
> [email protected]
> http://www.fsl.cs.sunysb.edu/mailman/listinfo/unionfs

_______________________________________________
unionfs mailing list
[email protected]
http://www.fsl.cs.sunysb.edu/mailman/listinfo/unionfs

Reply via email to