diff options
author | kib <kib@FreeBSD.org> | 2008-12-29 13:25:58 +0000 |
---|---|---|
committer | kib <kib@FreeBSD.org> | 2008-12-29 13:25:58 +0000 |
commit | 0deec7e423cd2663a58a55b2073aba84a7c000eb (patch) | |
tree | 5cd3d5d9ed06ee1895073bb9c722780923946890 /sys/fs/pseudofs/pseudofs_vncache.c | |
parent | f3a9a195cb5b2d1f5e0a7779c33cce89b9539695 (diff) | |
download | FreeBSD-src-0deec7e423cd2663a58a55b2073aba84a7c000eb.zip FreeBSD-src-0deec7e423cd2663a58a55b2073aba84a7c000eb.tar.gz |
When the insmntque() in the pfs_vncache_alloc() fails, vop_reclaim calls
pfs_vncache_free() that removes pvd from the list, while it is not yet
put on the list.
Prevent the invalid removal from the list by clearing pvd_next and
pvd_prev for the newly allocated pvd, and only move pfs_vncache list
head when the pvd was at the head.
Suggested and approved by: des
MFC after: 2 weeks
Diffstat (limited to 'sys/fs/pseudofs/pseudofs_vncache.c')
-rw-r--r-- | sys/fs/pseudofs/pseudofs_vncache.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sys/fs/pseudofs/pseudofs_vncache.c b/sys/fs/pseudofs/pseudofs_vncache.c index fd2bce1..0bacc03 100644 --- a/sys/fs/pseudofs/pseudofs_vncache.c +++ b/sys/fs/pseudofs/pseudofs_vncache.c @@ -149,6 +149,7 @@ retry: /* nope, get a new one */ pvd = malloc(sizeof *pvd, M_PFSVNCACHE, M_WAITOK); + pvd->pvd_next = pvd->pvd_prev = NULL; error = getnewvnode("pseudofs", mp, &pfs_vnodeops, vpp); if (error) { free(pvd, M_PFSVNCACHE); @@ -245,7 +246,7 @@ pfs_vncache_free(struct vnode *vp) pvd->pvd_next->pvd_prev = pvd->pvd_prev; if (pvd->pvd_prev) pvd->pvd_prev->pvd_next = pvd->pvd_next; - else + else if (pfs_vncache == pvd) pfs_vncache = pvd->pvd_next; --pfs_vncache_entries; mtx_unlock(&pfs_vncache_mutex); |