]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
fuse: use c++ allocations for group list 17237/head
authorJeff Layton <jlayton@redhat.com>
Wed, 23 Aug 2017 16:13:14 +0000 (12:13 -0400)
committerPatrick Donnelly <pdonnell@redhat.com>
Thu, 24 Aug 2017 16:47:17 +0000 (09:47 -0700)
commit7ad9933fbf3ab063ffe58467e0d970ab81be0cba
treef6eb1c5705c929f674548720bb1fb4b55623e837
parentd79b443b619a820add2084e110490f3631793437
fuse: use c++ allocations for group list

Valgrind is unhappy about our turning on supplimentary group handling
with fuse by default. The problem is that we end up calling delete to
free the supplimentary gids list, but fuse uses malloc to allocate it.

Note that I was initially concerned that I needed to use malloc and
free there to handle the case of userland calling ceph_userperm_new,
but we leave freeing the pointer up to the caller in that case.

Convert fuse to use new/delete to allocate and free the group lists
instead.

Tracker: http://tracker.ceph.com/issues/21065
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit f4fe5e2d524f8cca74f80a8a80fcd3e82b9effcb)
src/client/fuse_ll.cc