1
0
mirror of https://github.com/samba-team/samba.git synced 2024-12-22 13:34:15 +03:00

smbd: Modernize DEBUGs

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
This commit is contained in:
Volker Lendecke 2024-11-23 14:01:18 +01:00 committed by Martin Schwenke
parent 2af389de9a
commit 8327539651
3 changed files with 23 additions and 28 deletions

View File

@ -211,7 +211,7 @@ NTSTATUS dptr_create(connection_struct *conn,
DBG_INFO("dir=%s\n", fsp_str_dbg(fsp)); DBG_INFO("dir=%s\n", fsp_str_dbg(fsp));
if (sconn == NULL) { if (sconn == NULL) {
DEBUG(0,("dptr_create: called with fake connection_struct\n")); DBG_ERR("called with fake connection_struct\n");
return NT_STATUS_INTERNAL_ERROR; return NT_STATUS_INTERNAL_ERROR;
} }
@ -221,9 +221,8 @@ NTSTATUS dptr_create(connection_struct *conn,
status = check_any_access_fsp(fsp, SEC_DIR_LIST); status = check_any_access_fsp(fsp, SEC_DIR_LIST);
if (!NT_STATUS_IS_OK(status)) { if (!NT_STATUS_IS_OK(status)) {
DBG_INFO("dptr_create: directory %s " DBG_INFO("directory %s not open for LIST access\n",
"not open for LIST access\n", fsp_str_dbg(fsp));
fsp_str_dbg(fsp));
return status; return status;
} }
status = OpenDir_fsp(NULL, conn, fsp, wcard, attr, &dir_hnd); status = OpenDir_fsp(NULL, conn, fsp, wcard, attr, &dir_hnd);
@ -233,7 +232,7 @@ NTSTATUS dptr_create(connection_struct *conn,
dptr = talloc_zero(NULL, struct dptr_struct); dptr = talloc_zero(NULL, struct dptr_struct);
if(!dptr) { if(!dptr) {
DEBUG(0,("talloc fail in dptr_create.\n")); DBG_ERR("talloc failed\n");
TALLOC_FREE(dir_hnd); TALLOC_FREE(dir_hnd);
return NT_STATUS_NO_MEMORY; return NT_STATUS_NO_MEMORY;
} }

View File

@ -103,10 +103,9 @@ bool can_delete_file_in_directory(connection_struct *conn,
if ((get_current_uid(conn) != if ((get_current_uid(conn) !=
smb_fname_parent->st.st_ex_uid) && smb_fname_parent->st.st_ex_uid) &&
(get_current_uid(conn) != smb_fname->st.st_ex_uid)) { (get_current_uid(conn) != smb_fname->st.st_ex_uid)) {
DEBUG(10,("can_delete_file_in_directory: not " DBG_DEBUG("not owner of file %s or directory %s\n",
"owner of file %s or directory %s\n",
smb_fname_str_dbg(smb_fname), smb_fname_str_dbg(smb_fname),
smb_fname_str_dbg(smb_fname_parent))); smb_fname_str_dbg(smb_fname_parent));
ret = false; ret = false;
goto out; goto out;
} }

View File

@ -4248,9 +4248,9 @@ static NTSTATUS smb_file_position_information(connection_struct *conn,
position_information = PULL_LE_U64(pdata, 0); position_information = PULL_LE_U64(pdata, 0);
DEBUG(10,("smb_file_position_information: Set file position " DBG_DEBUG("Set file position information for file %s to %" PRIu64 "\n",
"information for file %s to %.0f\n", fsp_str_dbg(fsp), fsp_str_dbg(fsp),
(double)position_information)); position_information);
fh_set_position_information(fsp->fh, position_information); fh_set_position_information(fsp->fh, position_information);
return NT_STATUS_OK; return NT_STATUS_OK;
} }
@ -4675,8 +4675,7 @@ static NTSTATUS smb_file_rename_information(connection_struct *conn,
return status; return status;
} }
DEBUG(10,("smb_file_rename_information: got name |%s|\n", DBG_DEBUG("got name |%s|\n", newname);
newname));
/* Check the new name has no '/' characters. */ /* Check the new name has no '/' characters. */
if (strchr_m(newname, '/')) { if (strchr_m(newname, '/')) {
@ -4783,10 +4782,10 @@ static NTSTATUS smb_file_rename_information(connection_struct *conn,
} }
if (fsp != NULL && fsp->fsp_flags.is_fsa) { if (fsp != NULL && fsp->fsp_flags.is_fsa) {
DEBUG(10,("smb_file_rename_information: " DBG_DEBUG("SMB_FILE_RENAME_INFORMATION (%s) %s -> %s\n",
"SMB_FILE_RENAME_INFORMATION (%s) %s -> %s\n", fsp_fnum_dbg(fsp),
fsp_fnum_dbg(fsp), fsp_str_dbg(fsp), fsp_str_dbg(fsp),
smb_fname_str_dbg(smb_fname_dst))); smb_fname_str_dbg(smb_fname_dst));
/* /*
* If no pathnames are open below this directory, * If no pathnames are open below this directory,
@ -4805,10 +4804,9 @@ static NTSTATUS smb_file_rename_information(connection_struct *conn,
0, 0,
overwrite); overwrite);
} else { } else {
DEBUG(10,("smb_file_rename_information: " DBG_DEBUG("SMB_FILE_RENAME_INFORMATION %s -> %s\n",
"SMB_FILE_RENAME_INFORMATION %s -> %s\n",
smb_fname_str_dbg(smb_fname_src), smb_fname_str_dbg(smb_fname_src),
smb_fname_str_dbg(smb_fname_dst))); smb_fname_str_dbg(smb_fname_dst));
status = rename_internals(ctx, status = rename_internals(ctx,
conn, conn,
req, req,
@ -4914,8 +4912,7 @@ static NTSTATUS smb_set_info_standard(connection_struct *conn,
/* write time */ /* write time */
ft.mtime = time_t_to_full_timespec(srv_make_unix_date2(pdata+8)); ft.mtime = time_t_to_full_timespec(srv_make_unix_date2(pdata+8));
DEBUG(10,("smb_set_info_standard: file %s\n", DBG_DEBUG("file %s\n", smb_fname_str_dbg(smb_fname));
smb_fname_str_dbg(smb_fname)));
status = check_any_access_fsp(fsp, FILE_WRITE_ATTRIBUTES); status = check_any_access_fsp(fsp, FILE_WRITE_ATTRIBUTES);
if (!NT_STATUS_IS_OK(status)) { if (!NT_STATUS_IS_OK(status)) {
@ -4957,17 +4954,17 @@ static NTSTATUS smb_set_file_allocation_info(connection_struct *conn,
} }
allocation_size = PULL_LE_U64(pdata, 0); allocation_size = PULL_LE_U64(pdata, 0);
DEBUG(10,("smb_set_file_allocation_info: Set file allocation info for " DBG_DEBUG("Set file allocation info for file %s to %" PRIu64 "\n",
"file %s to %.0f\n", smb_fname_str_dbg(smb_fname), smb_fname_str_dbg(smb_fname),
(double)allocation_size)); allocation_size);
if (allocation_size) { if (allocation_size) {
allocation_size = smb_roundup(conn, allocation_size); allocation_size = smb_roundup(conn, allocation_size);
} }
DEBUG(10,("smb_set_file_allocation_info: file %s : setting new " DBG_DEBUG("file %s : setting new allocation size to %" PRIu64 "\n",
"allocation size to %.0f\n", smb_fname_str_dbg(smb_fname), smb_fname_str_dbg(smb_fname),
(double)allocation_size)); allocation_size);
if (!fsp->fsp_flags.is_pathref && (fsp_get_io_fd(fsp) != -1)) { if (!fsp->fsp_flags.is_pathref && (fsp_get_io_fd(fsp) != -1)) {
/* Open file handle. */ /* Open file handle. */