diff options
author | Toomas Soome <tsoome@FreeBSD.org> | 2017-03-01 19:02:43 +0000 |
---|---|---|
committer | Toomas Soome <tsoome@FreeBSD.org> | 2017-03-01 19:02:43 +0000 |
commit | d8b59bf47d0bea23540f2af49ef71431c6c033f2 (patch) | |
tree | 1d38b49020d5bcf4025b2c863a27d91baa4e3a57 /sys | |
parent | d945ed6472e48d8ad81e2f0c718099873d1f2d35 (diff) | |
download | src-d8b59bf47d0bea23540f2af49ef71431c6c033f2.tar.gz src-d8b59bf47d0bea23540f2af49ef71431c6c033f2.zip |
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
Notes
Notes:
svn path=/head/; revision=314504
Diffstat (limited to 'sys')
-rw-r--r-- | sys/boot/zfs/zfsimpl.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/boot/zfs/zfsimpl.c b/sys/boot/zfs/zfsimpl.c index 44e3bd343e3d..260479e257cb 100644 --- a/sys/boot/zfs/zfsimpl.c +++ b/sys/boot/zfs/zfsimpl.c @@ -2334,7 +2334,7 @@ zfs_lookup(const struct zfsmount *mount, const char *upath, dnode_phys_t *dnode) 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, const char *upath, dnode_phys_t *dnode) *dnode = dn; done: - STAILQ_FOREACH(entry, &on_cache, entry) + STAILQ_FOREACH_SAFE(entry, &on_cache, entry, tentry) free(entry); return (rc); } |