Does anyone have a problem if I add the following additional DEBUG statements?

[Jeremy Allison]

On Sun, Mar 25, 2012 at 05:24:17PM -0700, Richard Sharpe wrote:
> diff --git a/source3/smbd/nttrans.c b/source3/smbd/nttrans.c
> index fc52ee5..20379ac 100644
> --- a/source3/smbd/nttrans.c
> +++ b/source3/smbd/nttrans.c
> @@ -1868,11 +1868,13 @@ NTSTATUS smbd_do_query_security_desc(connection_struct *
> 
>         if ((security_info_wanted & SECINFO_SACL) &&
>                         !(fsp->access_mask & SEC_FLAG_SYSTEM_SECURITY)) {
> +               DEBUG(10, ("Access to SACL denied.\n"));
>                 return NT_STATUS_ACCESS_DENIED;
>         }
> 
>         if ((security_info_wanted & (SECINFO_DACL|SECINFO_OWNER|SECINFO_GROUP))
>                         !(fsp->access_mask & SEC_STD_READ_CONTROL)) {
> +               DEBUG(10, ("Access to DACL, OWNER, or GROUP denied.\n"));
>                 return NT_STATUS_ACCESS_DENIED;
>         }

No, push them please !