diff options
| author | Richard Sharpe <realrichardsharpe@gmail.com> | 2012-05-16 20:21:34 -0700 |
|---|---|---|
| committer | Richard Sharpe <sharpe@samba.org> | 2012-05-17 07:17:29 +0200 |
| commit | 632af6645376185500820a5be9dbf5c0c3d1e515 (patch) | |
| tree | 77dfd7fe9c9615d76aa1fb1bba5496994d14d412 | |
| parent | 6bafb4ac25989fd5d637db0da4afab5ae36bad1c (diff) | |
| download | samba-632af6645376185500820a5be9dbf5c0c3d1e515.tar.gz samba-632af6645376185500820a5be9dbf5c0c3d1e515.tar.bz2 samba-632af6645376185500820a5be9dbf5c0c3d1e515.zip | |
Check the return from create_acl_blob
Autobuild-User: Richard Sharpe <sharpe@samba.org>
Autobuild-Date: Thu May 17 07:17:29 CEST 2012 on sn-devel-104
| -rw-r--r-- | source3/modules/vfs_acl_common.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/source3/modules/vfs_acl_common.c b/source3/modules/vfs_acl_common.c index 221b43f748..c7bce5710a 100644 --- a/source3/modules/vfs_acl_common.c +++ b/source3/modules/vfs_acl_common.c @@ -592,10 +592,12 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp, NDR_PRINT_DEBUG(security_descriptor, discard_const_p(struct security_descriptor, psd)); } - /* - * Perhaps create_acl_blob should have a status return as well - */ - create_acl_blob(psd, &blob, XATTR_SD_HASH_TYPE_SHA256, hash); + status = create_acl_blob(psd, &blob, XATTR_SD_HASH_TYPE_SHA256, hash); + if (!NT_STATUS_IS_OK(status)) { + DEBUG(10, ("fset_nt_acl_xattr: create_acl_blob failed\n")); + return status; + } + status = store_acl_blob_fsp(handle, fsp, &blob); return status; |