]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
vfs: Simplify connect_acl_tdb()
authorVolker Lendecke <vl@samba.org>
Thu, 28 Nov 2024 14:48:45 +0000 (15:48 +0100)
committerVolker Lendecke <vl@samba.org>
Tue, 17 Dec 2024 12:30:30 +0000 (12:30 +0000)
We don't need talloc for printing a mask with %o

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Pavel Filipenský <pfilipensky@samba.org>
source3/modules/vfs_acl_tdb.c

index 5ecba7ba75726c2f747aaba4d47ef71a654a51ae..f2d2692159f1a5c86d8610f13ace13f21cf326a0 100644 (file)
@@ -274,23 +274,20 @@ static int connect_acl_tdb(struct vfs_handle_struct *handle,
 
        if (config->ignore_system_acls) {
                mode_t create_mask = lp_create_mask(SNUM(handle->conn));
-               char *create_mask_str = NULL;
 
                if ((create_mask & 0666) != 0666) {
+                       char create_mask_str[16];
+
                        create_mask |= 0666;
-                       create_mask_str = talloc_asprintf(handle, "0%o",
-                                                         create_mask);
-                       if (create_mask_str == NULL) {
-                               DBG_ERR("talloc_asprintf failed\n");
-                               return -1;
-                       }
+                       snprintf(create_mask_str,
+                                sizeof(create_mask_str),
+                                "0%o",
+                                create_mask);
 
                        DBG_NOTICE("setting 'create mask = %s'\n", create_mask_str);
 
                        lp_do_parameter (SNUM(handle->conn),
                                        "create mask", create_mask_str);
-
-                       TALLOC_FREE(create_mask_str);
                }
 
                DBG_NOTICE("setting 'directory mask = 0777', "