s3/vfs_glusterfs_fuse: Avoid using NAME_MAX directly
authorGünther Deschner <gd@samba.org>
Mon, 3 Jun 2019 14:28:36 +0000 (16:28 +0200)
committerGünther Deschner <gd@samba.org>
Tue, 11 Jun 2019 00:29:19 +0000 (00:29 +0000)
BUG: https://bugzilla.samba.org/show_bug.cgi?id=13872

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Günther Deschner <gd@samba.org>
Autobuild-Date(master): Tue Jun 11 00:29:19 UTC 2019 on sn-devel-184

source3/modules/vfs_glusterfs_fuse.c

index 8855cd18d0192b3cef43fcf5f2775cf638c38bd9..d92f5e2b08bc1014ecfc3d8a8e5706052f483bb6 100644 (file)
@@ -21,6 +21,8 @@
 #include "smbd/smbd.h"
 #include "system/filesys.h"
 
+#define GLUSTER_NAME_MAX 255
+
 static int vfs_gluster_fuse_get_real_filename(struct vfs_handle_struct *handle,
                                              const char *path,
                                              const char *name,
@@ -28,19 +30,19 @@ static int vfs_gluster_fuse_get_real_filename(struct vfs_handle_struct *handle,
                                              char **_found_name)
 {
        int ret;
-       char key_buf[NAME_MAX + 64];
-       char val_buf[NAME_MAX + 1];
+       char key_buf[GLUSTER_NAME_MAX + 64];
+       char val_buf[GLUSTER_NAME_MAX + 1];
        char *found_name = NULL;
 
-       if (strlen(name) >= NAME_MAX) {
+       if (strlen(name) >= GLUSTER_NAME_MAX) {
                errno = ENAMETOOLONG;
                return -1;
        }
 
-       snprintf(key_buf, NAME_MAX + 64,
+       snprintf(key_buf, GLUSTER_NAME_MAX + 64,
                 "glusterfs.get_real_filename:%s", name);
 
-       ret = getxattr(path, key_buf, val_buf, NAME_MAX + 1);
+       ret = getxattr(path, key_buf, val_buf, GLUSTER_NAME_MAX + 1);
        if (ret == -1) {
                if (errno == ENOATTR) {
                        errno = EOPNOTSUPP;