On Tue, Jul 08, 2014 at 11:01:25AM -0700, Steve Beattie wrote:
> [This is an unmodified resend of a patch that was buried deep in a
> thread and may have been overlooked.]
> 
> Use ap_hook_check_access_ex() instead of
> ap_hook_access_checker() for apache 2.4; see
> http://httpd.apache.org/docs/2.4/developer/new_api_2_4.html#http_request
> 
> Signed-off-by: Steve Beattie <[email protected]>

Acked-by: Seth Arnold <[email protected]>

Thanks

> ---
>  changehat/mod_apparmor/mod_apparmor.c |    8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> Index: b/changehat/mod_apparmor/mod_apparmor.c
> ===================================================================
> --- a/changehat/mod_apparmor/mod_apparmor.c
> +++ b/changehat/mod_apparmor/mod_apparmor.c
> @@ -404,7 +404,15 @@ register_hooks(apr_pool_t *p)
>  {
>      ap_hook_post_config(aa_init, NULL, NULL, APR_HOOK_MIDDLE);
>      ap_hook_child_init(aa_child_init, NULL, NULL, APR_HOOK_MIDDLE);
> +
> +#if AP_SERVER_MAJORVERSION_NUMBER == 2 && AP_SERVER_MINORVERSION_NUMBER < 3
> +    /* Compatibility with apache 2.2 */
>      ap_hook_access_checker(aa_enter_hat, NULL, NULL, APR_HOOK_FIRST);
> +#else
> +    /* apache 2.4 mod_authz hook */
> +    ap_hook_check_access_ex(aa_enter_hat, NULL, NULL, APR_HOOK_FIRST, 
> AP_AUTH_INTERNAL_PER_CONF);
> +#endif
> +
>      /* ap_hook_post_read_request(aa_enter_hat, NULL, NULL, APR_HOOK_FIRST); 
> */
>      ap_hook_log_transaction(aa_exit_hat, NULL, NULL, APR_HOOK_LAST);
>  }
> -- 
> Steve Beattie
> <[email protected]>
> http://NxNW.org/~steve/



> -- 
> AppArmor mailing list
> [email protected]
> Modify settings or unsubscribe at: 
> https://lists.ubuntu.com/mailman/listinfo/apparmor

Attachment: signature.asc
Description: Digital signature

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to