]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
cmake: remove util.cc from lib{rados,cephfs},ceph-objectstore-tool
authorKefu Chai <kchai@redhat.com>
Tue, 19 Jul 2016 06:24:45 +0000 (14:24 +0800)
committerKefu Chai <kchai@redhat.com>
Tue, 19 Jul 2016 14:25:36 +0000 (22:25 +0800)
util.cc is included by both librados and libcephfs, the `lvm` static
variable in `lsb_release_parse()` will be free twice by them. this
could lead to double free issue. and util.cc is not used by client at all, so
remove it from them.

Signed-off-by: Kefu Chai <kchai@redhat.com>
src/CMakeLists.txt
src/librados/CMakeLists.txt
src/tools/CMakeLists.txt

index 72e210668529f44d25d421960ccd316ed56c90b4..d54cd683ed127f2a43bf8f7537f83a7fa735546a 100644 (file)
@@ -748,8 +748,7 @@ add_library(parse_secret_objs OBJECT ${parse_secret_files})
 if(WITH_LIBCEPHFS)
   add_subdirectory(client)
   set(libcephfs_srcs libcephfs.cc)
-  add_library(cephfs ${CEPH_SHARED} ${libcephfs_srcs}
-    $<TARGET_OBJECTS:common_util_obj>)
+  add_library(cephfs ${CEPH_SHARED} ${libcephfs_srcs})
   target_link_libraries(cephfs LINK_PRIVATE client
     ${CRYPTO_LIBS} ${EXTRALIBS})
   if(ENABLE_SHARED)
index 7a99ef44f34a59af1a2267d43b4d015929c7363d..b4335a606b86df8383ee774206b5036e46a825dc 100644 (file)
@@ -4,7 +4,6 @@ add_library(librados_objs OBJECT
   RadosClient.cc)
 add_library(librados ${CEPH_SHARED}
   librados.cc
-  $<TARGET_OBJECTS:common_util_obj>
   $<TARGET_OBJECTS:librados_objs>)
 add_dependencies(librados osdc)
 if(WITH_LTTNG)
index 5a0ff5cb67586d1e74931e41ec48909e6072c6d9..a93a96add5291be8b8523cb89ebf5028a6812994 100644 (file)
@@ -33,8 +33,7 @@ install(PROGRAMS
 
 add_executable(ceph-objectstore-tool
   ceph_objectstore_tool.cc
-  RadosDump.cc
-  $<TARGET_OBJECTS:common_util_obj>)
+  RadosDump.cc)
 target_link_libraries(ceph-objectstore-tool osd os global ${Boost_PROGRAM_OPTIONS_LIBRARY} ${CMAKE_DL_LIBS} fuse)
 install(TARGETS ceph-objectstore-tool DESTINATION bin)