diff options
author | jmallett <jmallett@FreeBSD.org> | 2012-03-29 03:13:43 +0000 |
---|---|---|
committer | jmallett <jmallett@FreeBSD.org> | 2012-03-29 03:13:43 +0000 |
commit | 0b5862156580ae2abf520982aca1e1085730d500 (patch) | |
tree | ea722f27f2dcefc1a0ae7ba943dd42b20b9207a5 /usr.bin/top | |
parent | 299c847f0b109bdb3c5e20596f1ba2dc6f72e468 (diff) | |
download | FreeBSD-src-0b5862156580ae2abf520982aca1e1085730d500.zip FreeBSD-src-0b5862156580ae2abf520982aca1e1085730d500.tar.gz |
Fix 32-bit libgeom consumers run on 64-bit kernels with COMPAT_FREEBSD32.
Kernel pointer values are used as opaque unique identifiers, which are then
used to reconstruct references between various providers, classes, etc., inside
libgeom from the source XML. Unfortunately, they're converted to pointer-width
integers (in the form of pointers) to do this, and 32-bit userland pointers
cannot hold sensible representations (however opaque) of 64-bit kernel pointers
on all systems.
In the case where the leading bits are zero and 32 distinct bits of pointer can
be identified, this will happen to work. On systems where the upper 32-bits of
kernel pointers are non-zero and the same for all kernel pointers, this will
result in double frees and all kinds of bizarre crashes and linkage between
objects inside libgeom.
To mitigate this problem, treat the opaque identifiers in the XML as C strings
instead, and internalize them to give unique and consistent per-object pointer
values in userland for each identifier in the XML. This allows us to keep the
libgeom logic the same with only minor changes to initial setup and parsing.
It might be more sensible for speed reasons to treat the identifiers as numbers
of a large size (uintmax_t, say) rather than strings, but strings seem fine for
now.
(As an added side-effect, this makes it slightly easier to identify unresolved
references, but nothing has been added to inform the user of those.)
Diffstat (limited to 'usr.bin/top')
0 files changed, 0 insertions, 0 deletions