git.samba.org
/
samba.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
6bafb4a
)
Check the return from create_acl_blob
author
Richard Sharpe
<realrichardsharpe@gmail.com>
Thu, 17 May 2012 03:21:34 +0000
(20:21 -0700)
committer
Richard Sharpe
<sharpe@samba.org>
Thu, 17 May 2012 05:17:29 +0000
(07:17 +0200)
Autobuild-User: Richard Sharpe <sharpe@samba.org>
Autobuild-Date: Thu May 17 07:17:29 CEST 2012 on sn-devel-104
source3/modules/vfs_acl_common.c
patch
|
blob
|
history
diff --git
a/source3/modules/vfs_acl_common.c
b/source3/modules/vfs_acl_common.c
index 221b43f7481e0bdcec7f334968249eb6a4c64e79..c7bce5710aa38e0f1890c3ac0b73784168b756df 100644
(file)
--- 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;