diff options
| author | Peng Haitao <penght@cn.fujitsu.com> | 2013-04-12 16:58:49 +0800 |
|---|---|---|
| committer | Jeff Layton <jlayton@samba.org> | 2013-04-12 06:19:13 -0400 |
| commit | 21b5bdcb354bc113473347af5e4995fae2285b58 (patch) | |
| tree | 534f849540a5a93f2f05fd128b405b97d7a5ec88 | |
| parent | 6885a6253aa214cbd6f6adbd5c948b74fa4a27a1 (diff) | |
| download | cifs-utils-21b5bdcb354bc113473347af5e4995fae2285b58.tar.gz cifs-utils-21b5bdcb354bc113473347af5e4995fae2285b58.tar.bz2 cifs-utils-21b5bdcb354bc113473347af5e4995fae2285b58.zip | |
setcifsacl.c: fix a bug of goto setcifsacl_facenum_ret
setcifsacl_facenum_ret: is called only if attrlen is equal to -1.
Signed-off-by: Peng Haitao <penght@cn.fujitsu.com>
| -rw-r--r-- | setcifsacl.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/setcifsacl.c b/setcifsacl.c index 67dd29c..4b38459 100644 --- a/setcifsacl.c +++ b/setcifsacl.c @@ -822,9 +822,10 @@ cifsacl: goto setcifsacl_action_ret; attrlen = setxattr(filename, ATTRNAME, ntsdptr, bufsize, 0); - if (attrlen == -1) + if (attrlen == -1) { printf("%s: setxattr error: %s\n", __func__, strerror(errno)); - goto setcifsacl_facenum_ret; + goto setcifsacl_facenum_ret; + } exit_plugin(plugin_handle); return 0; |