Date: Fri, 08 Aug 2008 01:09:11 +0200 From: =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?= <des@des.no> To: Kostik Belousov <kostikbel@gmail.com> Cc: freebsd-fs@freebsd.org, Mateusz Guzik <mjguzik@gmail.com>, bug-followup@freebsd.org Subject: Re: kern/126287: [ufs] [panic] Kernel panics while mounting an UFS filesystem with snapshot enabled Message-ID: <86zlnoe82w.fsf@ds4.des.no> In-Reply-To: <20080806144820.GO97161@deviant.kiev.zoral.com.ua> (Kostik Belousov's message of "Wed, 6 Aug 2008 17:48:20 %2B0300") References: <200808061020.m76AK5NI013323@freefall.freebsd.org> <20080806133441.GM97161@deviant.kiev.zoral.com.ua> <e7799ca20808060652y12525a12ne31cc44f540d7a7a@mail.gmail.com> <20080806144820.GO97161@deviant.kiev.zoral.com.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
Kostik Belousov <kostikbel@gmail.com> writes: > @@ -169,7 +169,8 @@ ffs_mount(struct mount *mp, struct thread *td) > * persist "snapshot" in the options list. > */ > vfs_deleteopt(mp->mnt_optnew, "snapshot"); > - vfs_deleteopt(mp->mnt_opt, "snapshot"); > + if (mp->mnt_opt !=3D NULL) > + vfs_deleteopt(mp->mnt_opt, "snapshot"); > } >=20=20 > MNT_ILOCK(mp); I would suggest also adding a KASSERT to vfs_deleteopt(). DES --=20 Dag-Erling Sm=C3=B8rgrav - des@des.no
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?86zlnoe82w.fsf>