On Sun, Jul 20, 2003, Divacky Roman wrote: > Hi, > > I might be wrong but this: > > free(mp->mnt_data, M_UNIONFSMNT); /* XXX */ > mp->mnt_data = 0; > > seems to me wrong.... and might cause crashes etc. > am I correct or wrong? > > its from union_vfsops.c:384 What's wrong with it? It's just freeing the memory it allocated earlier. FFS does the same thing.