diff options
author | Lars Knoll <lars.knoll@qt.io> | 2020-08-05 10:21:21 +0200 |
---|---|---|
committer | Lars Knoll <lars.knoll@qt.io> | 2020-08-07 13:35:45 +0200 |
commit | 096e268764397f781e72619e3b5f546d5786cae1 (patch) | |
tree | bdd0db7dc4e682125792a78b781b1f3064051a37 /src/tools | |
parent | 3f64d770037a0e3adc9685eb0a1d326515f87b42 (diff) |
Avoid UB in moc generated code
Introduce a Q_OFFSETOF() macro that uses the optional support of
offsetof() for non standard layout types and disables the
corresponding compiler warnings. All our supported compilers
support offsetof() on non standard layout types.
Use the macro to do the offset calculations required in moc
generated code to replace a manual offset calculation that
was dereferencing a null pointer.
Change-Id: I4aab3af3c8bbaa90372f2234aa1cf8399d023c22
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'src/tools')
-rw-r--r-- | src/tools/moc/generator.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/tools/moc/generator.cpp b/src/tools/moc/generator.cpp index da3f22aa96..06b2ffc06f 100644 --- a/src/tools/moc/generator.cpp +++ b/src/tools/moc/generator.cpp @@ -1613,7 +1613,7 @@ void Generator::generateQPropertyApi() for (const PrivateQPropertyDef &property: cdef->privateQProperties) { auto printAccessor = [this, property](bool constAccessor = false) { const char *constOrNot = constAccessor ? "const " : " "; - fprintf(out, " const size_t propertyMemberOffset = reinterpret_cast<size_t>(&(static_cast<%s *>(nullptr)->%s));\n", cdef->qualified.constData(), property.name.constData()); + fprintf(out, " const size_t propertyMemberOffset = Q_OFFSETOF(%s, %s);\n", cdef->qualified.constData(), property.name.constData()); fprintf(out, " %sauto *thisPtr = reinterpret_cast<%s%s *>(reinterpret_cast<%schar *>(this) - propertyMemberOffset);\n", constOrNot, constOrNot, cdef->qualified.constData(), constOrNot); }; |