Skip to content

Commit

Permalink
Enable stack-protector-strong for QNX 7.0.0
Browse files Browse the repository at this point in the history
This was originally enabled in the mkspecs for 64-bit QNX 7.0.0
but that broke when the qtConfig change was made.  It looks like
qtConfig shouldn't be used in the platform mkspecs.  I suspect
the stack-protector changes were left out of the 32-bit mkspecs
so that 6.6.0 builds wouldn't be affected.

Ignore the stack-protector/stack-protector-all possibility since
it isn't possible to access it without a command line option.
Specifying both options doesn't even make sense since
stack-protector-all encompasses stack-protector.

For now, leave out command line control of this feature.

Task-number: QTBUG-59644
Change-Id: I99323216be5b592dd2c3bef6d22da195764a6e65
Reviewed-by: Oswald Buddenhagen <[email protected]>
  • Loading branch information
jmcdonnell-blackberry committed Jun 13, 2017
1 parent 876aed3 commit d0706ae
Show file tree
Hide file tree
Showing 8 changed files with 65 additions and 19 deletions.
52 changes: 52 additions & 0 deletions config.tests/unix/stack-protector/stack-protector.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,52 @@
/****************************************************************************
**
** Copyright (C) 2016 The Qt Company Ltd.
** Contact: https://www.qt.io/licensing/
**
** This file is part of the config.tests of the Qt Toolkit.
**
** $QT_BEGIN_LICENSE:LGPL$
** Commercial License Usage
** Licensees holding valid commercial Qt licenses may use this file in
** accordance with the commercial license agreement provided with the
** Software or, alternatively, in accordance with the terms contained in
** a written agreement between you and The Qt Company. For licensing terms
** and conditions see https://www.qt.io/terms-conditions. For further
** information use the contact form at https://www.qt.io/contact-us.
**
** GNU Lesser General Public License Usage
** Alternatively, this file may be used under the terms of the GNU Lesser
** General Public License version 3 as published by the Free Software
** Foundation and appearing in the file LICENSE.LGPL3 included in the
** packaging of this file. Please review the following information to
** ensure the GNU Lesser General Public License version 3 requirements
** will be met: https://www.gnu.org/licenses/lgpl-3.0.html.
**
** GNU General Public License Usage
** Alternatively, this file may be used under the terms of the GNU
** General Public License version 2.0 or (at your option) the GNU General
** Public license version 3 or any later version approved by the KDE Free
** Qt Foundation. The licenses are as published by the Free Software
** Foundation and appearing in the file LICENSE.GPL2 and LICENSE.GPL3
** included in the packaging of this file. Please review the following
** information to ensure the GNU General Public License requirements will
** be met: https://www.gnu.org/licenses/gpl-2.0.html and
** https://www.gnu.org/licenses/gpl-3.0.html.
**
** $QT_END_LICENSE$
**
****************************************************************************/

#if defined(__QNXNTO__)
#include <sys/neutrino.h>

#if _NTO_VERSION < 700
#error stack-protector not used (by default) before QNX 7.0.0.
#endif

#endif

int main()
{
return 0;
}
2 changes: 2 additions & 0 deletions config.tests/unix/stack-protector/stack-protector.pro
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
SOURCES = stack-protector.cpp
QMAKE_CXXFLAGS += -fstack-protector-strong
6 changes: 3 additions & 3 deletions configure.json
Original file line number Diff line number Diff line change
Expand Up @@ -425,8 +425,8 @@
},
"stack_protector": {
"label": "stack protection",
"type": "compilerSupportsFlag",
"test": "-fstack-protector-strong"
"type": "compile",
"test": "unix/stack-protector"
},
"incredibuild_xge": {
"label": "IncrediBuild",
Expand Down Expand Up @@ -930,7 +930,7 @@
"stack-protector-strong": {
"label": "stack protection",
"condition": "config.qnx && tests.stack_protector",
"output": [ "publicQtConfig" ]
"output": [ "privateFeature" ]
},
"system-zlib": {
"label": "Using system zlib",
Expand Down
2 changes: 2 additions & 0 deletions mkspecs/common/qcc-base.conf
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@ QMAKE_CFLAGS_SSE4_2 += -msse4.2
QMAKE_CFLAGS_AVX += -mavx
QMAKE_CFLAGS_AVX2 += -mavx2

QMAKE_CFLAGS_STACK_PROTECTOR_STRONG = -fstack-protector-strong

QMAKE_CXXFLAGS += $$QMAKE_CFLAGS -lang-c++
QMAKE_CXXFLAGS_DEPS += $$QMAKE_CFLAGS_DEPS
QMAKE_CXXFLAGS_WARN_ON += $$QMAKE_CFLAGS_WARN_ON
Expand Down
5 changes: 5 additions & 0 deletions mkspecs/features/default_post.prf
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,11 @@ debug {
QMAKE_LIBFLAGS += $$QMAKE_LIBFLAGS_RELEASE
}

stack_protector_strong {
QMAKE_CFLAGS += $$QMAKE_CFLAGS_STACK_PROTECTOR_STRONG
QMAKE_CXXFLAGS += $$QMAKE_CFLAGS_STACK_PROTECTOR_STRONG
}

# disable special linker flags for host builds (no proper test for host support yet)
!host_build|!cross_compile {
use_gold_linker: QMAKE_LFLAGS += $$QMAKE_LFLAGS_USE_GOLD
Expand Down
1 change: 1 addition & 0 deletions mkspecs/features/qt_common.prf
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ DEFINES *= QT_NO_NARROWING_CONVERSIONS_IN_CONNECT
qtConfig(c++11): CONFIG += c++11 strict_c++
qtConfig(c++14): CONFIG += c++14
qtConfig(c++1z): CONFIG += c++1z
qtConfig(stack-protector-strong): CONFIG += stack_protector_strong
contains(TEMPLATE, .*lib) {
# module and plugins
if(!host_build|!cross_compile):qtConfig(reduce_exports): CONFIG += hide_symbols
Expand Down
8 changes: 0 additions & 8 deletions mkspecs/qnx-aarch64le-qcc/qmake.conf
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,3 @@ DEFINES += _FORTIFY_SOURCE=2
QMAKE_LFLAGS_SHLIB += -Wl,-z,relro -Wl,-z,now

include(../common/qcc-base-qnx-aarch64le.conf)

qtConfig(stack-protector-strong) {
QMAKE_CFLAGS += -fstack-protector-strong
QMAKE_CXXFLAGS += -fstack-protector-strong
} else {
QMAKE_CFLAGS += -fstack-protector -fstack-protector-all
QMAKE_CXXFLAGS += -fstack-protector -fstack-protector-all
}
8 changes: 0 additions & 8 deletions mkspecs/qnx-x86-64-qcc/qmake.conf
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,3 @@
QMAKE_LFLAGS_SHLIB += -Wl,-z,relro -Wl,-z,now

include(../common/qcc-base-qnx-x86-64.conf)

qtConfig(stack-protector-strong) {
QMAKE_CFLAGS += -fstack-protector-strong
QMAKE_CXXFLAGS += -fstack-protector-strong
} else {
QMAKE_CFLAGS += -fstack-protector -fstack-protector-all
QMAKE_CXXFLAGS += -fstack-protector -fstack-protector-all
}

0 comments on commit d0706ae

Please sign in to comment.