diff options
| author | Giuseppe D'Angelo <dangelog@gmail.com> | 2012-03-24 08:36:52 +0000 |
|---|---|---|
| committer | Qt by Nokia <qt-info@nokia.com> | 2012-04-03 19:12:13 +0200 |
| commit | 33984e72abf6c3aa1fed37740d8731c96f68d6e2 (patch) | |
| tree | 5856aae74d79ee32251bfaa5ec45a867f40ac131 /tools/configure/configure_pch.h | |
| parent | 911eed0f90d84b22db69f43eda33ca4ee4965b52 (diff) | |
QHash security fix (1/2): add global QHash seed
Algorithmic complexity attacks against hash tables have been known
since 2003 (cf. [1, 2]), and they have been left unpatched for years
until the 2011 attacks [3] against many libraries /
(reference) implementations of programming languages.
This patch adds a global integer, to be used as a seed for the hash
function itself. The seed is randomly initialized the first time a
QHash detaches from shared_null.
Right now the seed is not used at all -- another patch will modify
qHash to make use of it.
[1] http://www.cs.rice.edu/~scrosby/hash/CrosbyWallach_UsenixSec2003.pdf
[2] http://perldoc.perl.org/perlsec.html#Algorithmic-Complexity-Attacks
[3] http://www.ocert.org/advisories/ocert-2011-003.html
Task-number: QTBUG-23529
Change-Id: I7519e4c02b9c2794d1c14079b01330eb356e9c65
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'tools/configure/configure_pch.h')
| -rw-r--r-- | tools/configure/configure_pch.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/tools/configure/configure_pch.h b/tools/configure/configure_pch.h index 0831364fe1..36a25dcca8 100644 --- a/tools/configure/configure_pch.h +++ b/tools/configure/configure_pch.h @@ -39,6 +39,11 @@ ** ****************************************************************************/ +// for rand_s, _CRT_RAND_S must be #defined before #including stdlib.h. +// put it at the beginning so some indirect inclusion doesn't break it +#ifndef _CRT_RAND_S +#define _CRT_RAND_S +#endif #include <qplatformdefs.h> #include <qglobal.h> #include <qlist.h> |