setcifsacl_facenum_ret: is called only if attrlen is equal to -1. Signed-off-by: Peng Haitao <penght@xxxxxxxxxxxxxx> --- setcifsacl.c | 5 +++-- 1 file 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; -- 1.8.1.4 -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html