diff options
| author | Sara Hartse <[email protected]> | 2019-04-04 18:57:06 -0700 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2019-04-04 18:57:06 -0700 |
| commit | a887d653b32aaba3fe04c7b25ff0091b9ea9c64e (patch) | |
| tree | 6d0bd45a6d2080fce9223ae252f5c93782418dd7 /cmd | |
| parent | af6507930051e9ee481cad1130df84636c4a770c (diff) | |
Restrict kstats and print real pointers
There are several places where we use zfs_dbgmsg and %p to
print pointers. In the Linux kernel, these values obfuscated
to prevent information leaks which means the pointers aren't
very useful for debugging crash dumps. We decided to restrict
the permissions of dbgmsg (and some other kstats while we were
at it) and print pointers with %px in zfs_dbgmsg as well as
spl_dumpstack
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: John Gallagher <[email protected]>
Signed-off-by: sara hartse <[email protected]>
Closes #8467
Closes #8476
Diffstat (limited to 'cmd')
| -rw-r--r-- | cmd/ztest/ztest.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cmd/ztest/ztest.c b/cmd/ztest/ztest.c index a1ab56bd2..9c2cf9501 100644 --- a/cmd/ztest/ztest.c +++ b/cmd/ztest/ztest.c @@ -6468,7 +6468,7 @@ ztest_initialize(ztest_ds_t *zd, uint64_t id) char *path = strdup(rand_vd->vdev_path); boolean_t active = rand_vd->vdev_initialize_thread != NULL; - zfs_dbgmsg("vd %p, guid %llu", rand_vd, guid); + zfs_dbgmsg("vd %px, guid %llu", rand_vd, guid); spa_config_exit(spa, SCL_VDEV, FTAG); uint64_t cmd = ztest_random(POOL_INITIALIZE_FUNCS); |