summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2006-06-20 04:14:37 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 14:09:12 -0500
commit59670b2f9762a2a7c6764f9aba9012abec7f5e25 (patch)
tree89ea7232a4ddff29802219ad5678a8013065762c
parentf03e039e88ae3ea5c1d5e00c43b338a41ca2de66 (diff)
downloadsamba-59670b2f9762a2a7c6764f9aba9012abec7f5e25.tar.gz
samba-59670b2f9762a2a7c6764f9aba9012abec7f5e25.tar.bz2
samba-59670b2f9762a2a7c6764f9aba9012abec7f5e25.zip
r16399: - split out pushing smb_fsinfo into a DATA_BLOB
- add error checks metze (This used to be commit 2aaca1631f4e9f312a8de506b21290b9452c2a43)
-rw-r--r--source4/smb_server/smb/trans2.c143
1 files changed, 76 insertions, 67 deletions
diff --git a/source4/smb_server/smb/trans2.c b/source4/smb_server/smb/trans2.c
index c29e1020e1..ecc5165274 100644
--- a/source4/smb_server/smb/trans2.c
+++ b/source4/smb_server/smb/trans2.c
@@ -237,131 +237,122 @@ static NTSTATUS trans2_align_data(struct smb_trans2 *trans)
return NT_STATUS_OK;
}
-
-/*
- trans2 qfsinfo implementation send
-*/
-static NTSTATUS trans2_qfsinfo_send(struct trans_op *op)
+static NTSTATUS trans2_push_fsinfo(struct smbsrv_request *req,
+ union smb_fsinfo *fsinfo,
+ TALLOC_CTX *mem_ctx,
+ DATA_BLOB *blob)
{
- struct smbsrv_request *req = op->req;
- struct smb_trans2 *trans = op->trans;
- union smb_fsinfo *fsinfo;
- NTSTATUS status;
uint_t i;
DATA_BLOB guid_blob;
- TRANS2_CHECK_ASYNC_STATUS(fsinfo, union smb_fsinfo);
-
switch (fsinfo->generic.level) {
case SMB_QFS_ALLOCATION:
- trans2_setup_reply(trans, 0, 18, 0);
+ TRANS2_CHECK(trans2_grow_data(mem_ctx, blob, 18));
- SIVAL(trans->out.data.data, 0, fsinfo->allocation.out.fs_id);
- SIVAL(trans->out.data.data, 4, fsinfo->allocation.out.sectors_per_unit);
- SIVAL(trans->out.data.data, 8, fsinfo->allocation.out.total_alloc_units);
- SIVAL(trans->out.data.data, 12, fsinfo->allocation.out.avail_alloc_units);
- SSVAL(trans->out.data.data, 16, fsinfo->allocation.out.bytes_per_sector);
+ SIVAL(blob->data, 0, fsinfo->allocation.out.fs_id);
+ SIVAL(blob->data, 4, fsinfo->allocation.out.sectors_per_unit);
+ SIVAL(blob->data, 8, fsinfo->allocation.out.total_alloc_units);
+ SIVAL(blob->data, 12, fsinfo->allocation.out.avail_alloc_units);
+ SSVAL(blob->data, 16, fsinfo->allocation.out.bytes_per_sector);
return NT_STATUS_OK;
case SMB_QFS_VOLUME:
- trans2_setup_reply(trans, 0, 5, 0);
+ TRANS2_CHECK(trans2_grow_data(mem_ctx, blob, 5));
- SIVAL(trans->out.data.data, 0, fsinfo->volume.out.serial_number);
+ SIVAL(blob->data, 0, fsinfo->volume.out.serial_number);
/* w2k3 implements this incorrectly for unicode - it
* leaves the last byte off the string */
- trans2_append_data_string(req, trans, &trans->out.data,
- &fsinfo->volume.out.volume_name,
- 4, STR_LEN8BIT|STR_NOALIGN);
+ TRANS2_CHECK(trans2_append_data_string(req, mem_ctx, blob,
+ &fsinfo->volume.out.volume_name,
+ 4, STR_LEN8BIT|STR_NOALIGN));
return NT_STATUS_OK;
case SMB_QFS_VOLUME_INFO:
case SMB_QFS_VOLUME_INFORMATION:
- trans2_setup_reply(trans, 0, 18, 0);
+ TRANS2_CHECK(trans2_grow_data(mem_ctx, blob, 18));
- push_nttime(trans->out.data.data, 0, fsinfo->volume_info.out.create_time);
- SIVAL(trans->out.data.data, 8, fsinfo->volume_info.out.serial_number);
- SSVAL(trans->out.data.data, 16, 0); /* padding */
- trans2_append_data_string(req, trans, &trans->out.data,
- &fsinfo->volume_info.out.volume_name,
- 12, STR_UNICODE);
+ push_nttime(blob->data, 0, fsinfo->volume_info.out.create_time);
+ SIVAL(blob->data, 8, fsinfo->volume_info.out.serial_number);
+ SSVAL(blob->data, 16, 0); /* padding */
+ TRANS2_CHECK(trans2_append_data_string(req, mem_ctx, blob,
+ &fsinfo->volume_info.out.volume_name,
+ 12, STR_UNICODE));
return NT_STATUS_OK;
case SMB_QFS_SIZE_INFO:
case SMB_QFS_SIZE_INFORMATION:
- trans2_setup_reply(trans, 0, 24, 0);
+ TRANS2_CHECK(trans2_grow_data(mem_ctx, blob, 24));
- SBVAL(trans->out.data.data, 0, fsinfo->size_info.out.total_alloc_units);
- SBVAL(trans->out.data.data, 8, fsinfo->size_info.out.avail_alloc_units);
- SIVAL(trans->out.data.data, 16, fsinfo->size_info.out.sectors_per_unit);
- SIVAL(trans->out.data.data, 20, fsinfo->size_info.out.bytes_per_sector);
+ SBVAL(blob->data, 0, fsinfo->size_info.out.total_alloc_units);
+ SBVAL(blob->data, 8, fsinfo->size_info.out.avail_alloc_units);
+ SIVAL(blob->data, 16, fsinfo->size_info.out.sectors_per_unit);
+ SIVAL(blob->data, 20, fsinfo->size_info.out.bytes_per_sector);
return NT_STATUS_OK;
case SMB_QFS_DEVICE_INFO:
case SMB_QFS_DEVICE_INFORMATION:
- trans2_setup_reply(trans, 0, 8, 0);
- SIVAL(trans->out.data.data, 0, fsinfo->device_info.out.device_type);
- SIVAL(trans->out.data.data, 4, fsinfo->device_info.out.characteristics);
- return NT_STATUS_OK;
+ TRANS2_CHECK(trans2_grow_data(mem_ctx, blob, 8));
+
+ SIVAL(blob->data, 0, fsinfo->device_info.out.device_type);
+ SIVAL(blob->data, 4, fsinfo->device_info.out.characteristics);
+ return NT_STATUS_OK;
case SMB_QFS_ATTRIBUTE_INFO:
case SMB_QFS_ATTRIBUTE_INFORMATION:
- trans2_setup_reply(trans, 0, 12, 0);
+ TRANS2_CHECK(trans2_grow_data(mem_ctx, blob, 12));
- SIVAL(trans->out.data.data, 0, fsinfo->attribute_info.out.fs_attr);
- SIVAL(trans->out.data.data, 4, fsinfo->attribute_info.out.max_file_component_length);
+ SIVAL(blob->data, 0, fsinfo->attribute_info.out.fs_attr);
+ SIVAL(blob->data, 4, fsinfo->attribute_info.out.max_file_component_length);
/* this must not be null terminated or win98 gets
confused! also note that w2k3 returns this as
unicode even when ascii is negotiated */
- trans2_append_data_string(req, trans, &trans->out.data,
- &fsinfo->attribute_info.out.fs_type,
- 8, STR_UNICODE);
+ TRANS2_CHECK(trans2_append_data_string(req, mem_ctx, blob,
+ &fsinfo->attribute_info.out.fs_type,
+ 8, STR_UNICODE));
return NT_STATUS_OK;
case SMB_QFS_QUOTA_INFORMATION:
- trans2_setup_reply(trans, 0, 48, 0);
+ TRANS2_CHECK(trans2_grow_data(mem_ctx, blob, 48));
- SBVAL(trans->out.data.data, 0, fsinfo->quota_information.out.unknown[0]);
- SBVAL(trans->out.data.data, 8, fsinfo->quota_information.out.unknown[1]);
- SBVAL(trans->out.data.data, 16, fsinfo->quota_information.out.unknown[2]);
- SBVAL(trans->out.data.data, 24, fsinfo->quota_information.out.quota_soft);
- SBVAL(trans->out.data.data, 32, fsinfo->quota_information.out.quota_hard);
- SBVAL(trans->out.data.data, 40, fsinfo->quota_information.out.quota_flags);
+ SBVAL(blob->data, 0, fsinfo->quota_information.out.unknown[0]);
+ SBVAL(blob->data, 8, fsinfo->quota_information.out.unknown[1]);
+ SBVAL(blob->data, 16, fsinfo->quota_information.out.unknown[2]);
+ SBVAL(blob->data, 24, fsinfo->quota_information.out.quota_soft);
+ SBVAL(blob->data, 32, fsinfo->quota_information.out.quota_hard);
+ SBVAL(blob->data, 40, fsinfo->quota_information.out.quota_flags);
return NT_STATUS_OK;
case SMB_QFS_FULL_SIZE_INFORMATION:
- trans2_setup_reply(trans, 0, 32, 0);
+ TRANS2_CHECK(trans2_grow_data(mem_ctx, blob, 32));
- SBVAL(trans->out.data.data, 0, fsinfo->full_size_information.out.total_alloc_units);
- SBVAL(trans->out.data.data, 8, fsinfo->full_size_information.out.call_avail_alloc_units);
- SBVAL(trans->out.data.data, 16, fsinfo->full_size_information.out.actual_avail_alloc_units);
- SIVAL(trans->out.data.data, 24, fsinfo->full_size_information.out.sectors_per_unit);
- SIVAL(trans->out.data.data, 28, fsinfo->full_size_information.out.bytes_per_sector);
+ SBVAL(blob->data, 0, fsinfo->full_size_information.out.total_alloc_units);
+ SBVAL(blob->data, 8, fsinfo->full_size_information.out.call_avail_alloc_units);
+ SBVAL(blob->data, 16, fsinfo->full_size_information.out.actual_avail_alloc_units);
+ SIVAL(blob->data, 24, fsinfo->full_size_information.out.sectors_per_unit);
+ SIVAL(blob->data, 28, fsinfo->full_size_information.out.bytes_per_sector);
return NT_STATUS_OK;
case SMB_QFS_OBJECTID_INFORMATION:
- trans2_setup_reply(trans, 0, 64, 0);
+ TRANS2_CHECK(trans2_grow_data(mem_ctx, blob, 64));
- status = ndr_push_struct_blob(&guid_blob, req,
- &fsinfo->objectid_information.out.guid,
- (ndr_push_flags_fn_t)ndr_push_GUID);
- if (!NT_STATUS_IS_OK(status)) {
- return status;
- }
-
- memcpy(trans->out.data.data, guid_blob.data, guid_blob.length);
+ TRANS2_CHECK(ndr_push_struct_blob(&guid_blob, mem_ctx,
+ &fsinfo->objectid_information.out.guid,
+ (ndr_push_flags_fn_t)ndr_push_GUID));
+ memcpy(blob->data, guid_blob.data, guid_blob.length);
for (i=0;i<6;i++) {
- SBVAL(trans->out.data.data, 16 + 8*i, fsinfo->objectid_information.out.unknown[i]);
+ SBVAL(blob->data, 16 + 8*i, fsinfo->objectid_information.out.unknown[i]);
}
+
return NT_STATUS_OK;
}
@@ -369,6 +360,24 @@ static NTSTATUS trans2_qfsinfo_send(struct trans_op *op)
}
/*
+ trans2 qfsinfo implementation send
+*/
+static NTSTATUS trans2_qfsinfo_send(struct trans_op *op)
+{
+ struct smbsrv_request *req = op->req;
+ struct smb_trans2 *trans = op->trans;
+ union smb_fsinfo *fsinfo;
+
+ TRANS2_CHECK_ASYNC_STATUS(fsinfo, union smb_fsinfo);
+
+ TRANS2_CHECK(trans2_setup_reply(trans, 0, 0, 0));
+
+ TRANS2_CHECK(trans2_push_fsinfo(req, fsinfo, trans, &trans->out.data));
+
+ return NT_STATUS_OK;
+}
+
+/*
trans2 qfsinfo implementation
*/
static NTSTATUS trans2_qfsinfo(struct smbsrv_request *req, struct trans_op *op)