]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
This bug was introduced with the change to use softdep_bp_to_mp() in
authormckusick <mckusick@FreeBSD.org>
Mon, 28 Jan 2019 21:36:45 +0000 (21:36 +0000)
committermckusick <mckusick@FreeBSD.org>
Mon, 28 Jan 2019 21:36:45 +0000 (21:36 +0000)
commit0c506150b7832f1ad3d81ad7ae1d1f4dced55414
tree058ffee0421716a7075da35b2b0254f0b577b13b
parent247f8f74e75cf3c56d4d87841ac102f53d1ba76e
This bug was introduced with the change to use softdep_bp_to_mp() in
January 2018 changes -r327723 and -r327821. The softdep_bp_to_mp()
function failed to include VFIFO as one of the valid cases.

Although fifo's do not allocate blocks in the filesystem, they will
allocate blocks if they use extended attributes (such as ACLs). Thus,
softdep_bp_to_mp() needs to return a non-NULL mount pointer when
presented with a fifo vnode so that the soft updates write complete
will properly process the soft updates structures associated with the
extended attribute blocks. It was the failure to process these soft
updates structures, thus leaving them hanging off the buffer, which
lead to the "panic: softdep_deallocate_dependencies: dangling deps"
when trying to clean up the buffer after it was written.

PR:           230962
Reported by:  2t8mr7kx9f@protonmail.com
Reviewed by:  kib
Tested by:    Peter Holm
MFC after:    1 week
Sponsored by: Netflix
sys/ufs/ffs/ffs_softdep.c