diff options
author | Richard Yao <[email protected]> | 2012-05-30 23:25:31 -0400 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2012-07-17 09:22:43 -0700 |
commit | 0a6b03d3b87cf17ac6159395df93ba3fcfdddc45 (patch) | |
tree | de8b4ffa1f25089e7f1659f2586c4a577c84d08f /include/sys/zfs_acl.h | |
parent | b5a28807cdec3c05aa69cbe4689cd914dc94783a (diff) |
Fix build failures on PaX/GRSecurity patched kernels
Gentoo Hardened kernels include the PaX/GRSecurity patches. They use a
dialect of C that relies on a GCC plugin. In particular, struct
file_operations has been marked do_const in the PaX/GRSecurity dialect,
which causes GCC to consider all instances of it as const. This caused
failures in the autotools checks and the ZFS source code.
To address this, we modify the autotools checks to take into account
differences between the PaX C dialect and the regular C dialect. We also
modify struct zfs_acl's z_ops member to be a pointer to a function
pointer table. Lastly, we modify zpl_put_link() to address a PaX change
to the function prototype of nd_get_link(). This avoids compiler errors
in the PaX/GRSecurity dialect.
Note that the change in zpl_put_link() causes a warning that becomes a
build failure when debugging is enabled. Fixing that warning requires
ryao/spl@5ca50ef459c59bc74b7a7cd3af7311da2b1cd2c3.
Signed-off-by: Richard Yao <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #484
Diffstat (limited to 'include/sys/zfs_acl.h')
-rw-r--r-- | include/sys/zfs_acl.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/sys/zfs_acl.h b/include/sys/zfs_acl.h index b6ed4390f..11fc3351e 100644 --- a/include/sys/zfs_acl.h +++ b/include/sys/zfs_acl.h @@ -165,7 +165,7 @@ typedef struct zfs_acl { uint64_t z_hints; /* ACL hints (ZFS_INHERIT_ACE ...) */ zfs_acl_node_t *z_curr_node; /* current node iterator is handling */ list_t z_acl; /* chunks of ACE data */ - acl_ops_t z_ops; /* ACL operations */ + acl_ops_t *z_ops; /* ACL operations */ } zfs_acl_t; typedef struct acl_locator_cb { |