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

[Richard Sharpe]

On Sun, Mar 25, 2012 at 6:16 PM, John H Terpstra <jht at samba.org> wrote:
> On Sun 25 Mar 2012 07:24:17 PM CDT, 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;
>>         }
>>
>>
>
> Richard,
>
> Why are you setting these notifications at loglevel = 10? Would it not
> make sense to set this a little lower (like 5)?

That's a good question ... maybe it would.

-- 
Regards,
Richard Sharpe
(何以解憂？唯有杜康。--曹操)