mirror of
https://github.com/qTox/qTox.git
synced 2024-03-22 14:00:36 +08:00
chore(build): use -pie
flag only when linking
If used when compiling and not just linking, clang complains about it. Fixes #4101.
This commit is contained in:
parent
bff25ce8b1
commit
63ace85913
|
@ -42,6 +42,23 @@ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11")
|
|||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fno-exceptions")
|
||||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fno-rtti")
|
||||
|
||||
# Hardening flags (ASLR, warnings, etc)
|
||||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fPIE")
|
||||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wstrict-overflow")
|
||||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wstrict-aliasing")
|
||||
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -pie")
|
||||
|
||||
if (NOT WIN32)
|
||||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fstack-protector-all")
|
||||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wstack-protector")
|
||||
endif()
|
||||
|
||||
if (UNIX AND NOT APPLE)
|
||||
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -Wl,-z,now")
|
||||
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -Wl,-z,relro")
|
||||
endif()
|
||||
|
||||
|
||||
# Use ccache when available to speed up builds.
|
||||
find_program(CCACHE_FOUND ccache)
|
||||
if(CCACHE_FOUND)
|
||||
|
|
18
qtox.pro
18
qtox.pro
|
@ -46,18 +46,14 @@ CONFIG += silent
|
|||
|
||||
# Hardening flags (ASLR, warnings, etc)
|
||||
# TODO: add `-Werror` to hardening flags once all warnings are fixed
|
||||
win32 {
|
||||
QMAKE_CXXFLAGS += -pie \
|
||||
-fPIE \
|
||||
-Wstrict-overflow \
|
||||
-Wstrict-aliasing
|
||||
} else {
|
||||
QMAKE_CXXFLAGS += -fPIE \
|
||||
-Wstrict-overflow \
|
||||
-Wstrict-aliasing
|
||||
QMAKE_LFLAGS += -pie
|
||||
|
||||
!win32 {
|
||||
QMAKE_CXXFLAGS += -fstack-protector-all \
|
||||
-pie \
|
||||
-fPIE \
|
||||
-Wstack-protector \
|
||||
-Wstrict-overflow \
|
||||
-Wstrict-aliasing
|
||||
-Wstack-protector
|
||||
}
|
||||
|
||||
# osx & windows cannot into security (build on it fails with those enabled)
|
||||
|
|
Loading…
Reference in New Issue
Block a user