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 /module/zfs/vdev_removal.c | |
| 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 'module/zfs/vdev_removal.c')
| -rw-r--r-- | module/zfs/vdev_removal.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/module/zfs/vdev_removal.c b/module/zfs/vdev_removal.c index 99d67b7be..f2d18d925 100644 --- a/module/zfs/vdev_removal.c +++ b/module/zfs/vdev_removal.c @@ -340,7 +340,7 @@ vdev_remove_initiate_sync(void *arg, dmu_tx_t *tx) */ vdev_config_dirty(vd); - zfs_dbgmsg("starting removal thread for vdev %llu (%p) in txg %llu " + zfs_dbgmsg("starting removal thread for vdev %llu (%px) in txg %llu " "im_obj=%llu", vd->vdev_id, vd, dmu_tx_get_txg(tx), vic->vic_mapping_object); |