cmake: remove util.cc from lib{rados,cephfs},ceph-objectstore-tool

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 <[email protected]>
zmc · Jul 19, 2016 · 3173d2e · 3173d2e
1 parent 741f990
commit 3173d2e
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
@@ -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)

diff --git a/src/librados/CMakeLists.txt b/src/librados/CMakeLists.txt
@@ -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)

diff --git a/src/tools/CMakeLists.txt b/src/tools/CMakeLists.txt
@@ -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)