Date: Mon, 28 Apr 2014 10:40:03 -0700 From: Sean Bruno <sbruno@ignoranthack.me> To: FreeBSD FS <freebsd-fs@freebsd.org> Subject: sys/boot/zfs out of bounds warning Message-ID: <1398706803.1089.3.camel@powernoodle.corp.yahoo.com>
next in thread | raw e-mail | index | archive | help
[-- Attachment #1 --]
looking at sys/boot things this morning, noted that there's a pretty
obvious warning about an out of bounds access. I suspect that I don't
see a layer of abstraction here that makes this ok, but I'm not sure.
It seems like if this wasn't valid memory, things would go pretty badly
in zfs_lookup(). Is there a better way to do this memcpy() such that it
doesn't trip a warning here?
-----------------
In file included from /home/sbruno/bsd/head/sys/boot/zfs/zfs.c:49:
/home/sbruno/bsd/head/sys/boot/zfs/zfsimpl.c:2080:19: warning: array
index 264 is past the end of the array (which contains 192 elements)
[-Warray-bounds]
memcpy(path,
&dn.dn_bonus[sizeof(znode_phys_t)],
^
~~~~~~~~~~~~~~~~~~~~
/home/sbruno/bsd/head/sys/boot/zfs/../../cddl/boot/zfs/zfsimpl.h:790:2:
note: array 'dn_bonus' declared here
uint8_t dn_bonus[DN_MAX_BONUSLEN - sizeof (blkptr_t)];
------
[-- Attachment #2 --]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
iQEcBAABAgAGBQJTXpJpAAoJEBkJRdwI6BaHIugH+wdDO3ncstCGMZP9w30PBviF
2wYwFzE6yMzdrgVK0lY7tkTjl4Q4NUnW0G0poufPN88bgA8BHCbcn/UV5MncTown
85ex5wrb1m56bc7IYMHf8cafSy4imUNRtVn29eXn6Cul2qVlSBFq5uFMti6R8AfZ
t/O63VKpRFdqiS7CdZF+sY432VIPKpmyGbLv5xta5olV4FySNSe2il1wkIX3Hq2f
iTYXYmV3E0iWFt2yexkxd3Ur+g1uYhga9OKXzbBi11cZiofghvXdWspEOPq1/sj+
opVJRnPzjVIvVL7G7+iu2wEbhIH84kYef4L+6a23rofen81rKe7ZE+zENHPolNc=
=M5mD
-----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1398706803.1089.3.camel>