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
This commit is contained in:
Toomas Soome 2017-03-01 19:02:43 +00:00
parent d945ed6472
commit d8b59bf47d

View file

@ -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);
}