diff options
| author | Volker Lendecke <vl@samba.org> | 2014-10-28 15:20:26 -0700 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2014-10-31 03:47:40 +0100 |
| commit | 151b9caeef7dc4fa4816035a406acb9f1c5812c3 (patch) | |
| tree | 4c49ac449f7fd1279ff9f93ad738e4ad8a08abf4 /source3/locking | |
| parent | 8334428666b8282d2cfbcfd411acab0c338ae390 (diff) | |
| download | samba-151b9caeef7dc4fa4816035a406acb9f1c5812c3.tar.gz samba-151b9caeef7dc4fa4816035a406acb9f1c5812c3.tar.xz samba-151b9caeef7dc4fa4816035a406acb9f1c5812c3.zip | |
s3:locking: Change from ndr_pull_struct_blob() to ndr_pull_struct_blob_all() so we fail if not all bytes are consumed.
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'source3/locking')
| -rw-r--r-- | source3/locking/share_mode_lock.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/locking/share_mode_lock.c b/source3/locking/share_mode_lock.c index da16d1acf0..65409acf8f 100644 --- a/source3/locking/share_mode_lock.c +++ b/source3/locking/share_mode_lock.c @@ -133,7 +133,7 @@ static struct share_mode_data *parse_share_modes(TALLOC_CTX *mem_ctx, blob.data = dbuf.dptr; blob.length = dbuf.dsize; - ndr_err = ndr_pull_struct_blob( + ndr_err = ndr_pull_struct_blob_all( &blob, d, d, (ndr_pull_flags_fn_t)ndr_pull_share_mode_data); if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) { DEBUG(1, ("ndr_pull_share_mode_lock failed: %s\n", @@ -476,7 +476,7 @@ static int share_mode_traverse_fn(struct db_record *rec, void *_state) blob.data = value.dptr; blob.length = value.dsize; - ndr_err = ndr_pull_struct_blob( + ndr_err = ndr_pull_struct_blob_all( &blob, d, d, (ndr_pull_flags_fn_t)ndr_pull_share_mode_data); if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) { DEBUG(1, ("ndr_pull_share_mode_lock failed\n")); |