Date: Wed, 1 Mar 2017 19:02:43 +0000 (UTC) From: Toomas Soome <tsoome@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r314504 - head/sys/boot/zfs Message-ID: <201703011902.v21J2hV7067360@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: tsoome Date: Wed Mar 1 19:02:43 2017 New Revision: 314504 URL: https://svnweb.freebsd.org/changeset/base/314504 Log: loader: r314112 did introduce dereference freed pointer entry CID: 1371675 Reported by: Coverity Reviewed by: jhb, allanjude Approved by: allanjude (mentor) Differential Revision: https://reviews.freebsd.org/D9846 Modified: head/sys/boot/zfs/zfsimpl.c Modified: head/sys/boot/zfs/zfsimpl.c ============================================================================== --- head/sys/boot/zfs/zfsimpl.c Wed Mar 1 18:53:05 2017 (r314503) +++ head/sys/boot/zfs/zfsimpl.c Wed Mar 1 19:02:43 2017 (r314504) @@ -2334,7 +2334,7 @@ zfs_lookup(const struct zfsmount *mount, char path[1024]; int symlinks_followed = 0; struct stat sb; - struct obj_list *entry; + struct obj_list *entry, *tentry; STAILQ_HEAD(, obj_list) on_cache = STAILQ_HEAD_INITIALIZER(on_cache); spa = mount->spa; @@ -2482,7 +2482,7 @@ zfs_lookup(const struct zfsmount *mount, *dnode = dn; done: - STAILQ_FOREACH(entry, &on_cache, entry) + STAILQ_FOREACH_SAFE(entry, &on_cache, entry, tentry) free(entry); return (rc); }
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201703011902.v21J2hV7067360>