diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-10-03 17:12:23 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-10-06 14:13:27 +0100 |
| commit | 8f436d3de7e99268a8862664d2cb2574231c3b18 (patch) | |
| tree | 0a07dd3cecbd7bc5b0293006bb8691e33c5cca75 /sw/qa/core/uwriter.cxx | |
| parent | e5ab3685550cf35c3eb9cb47530044f2d86433d5 (diff) | |
use comphelper::rng::uniform_*_distribution everywhere
and automatically seed from time on first use
coverity#1242393 Don't call rand
coverity#1242404 Don't call rand
coverity#1242410 Don't call rand and additionally allow 0xFF as a value
coverity#1242409 Don't call rand
coverity#1242399 Don't call rand
coverity#1242372 Don't call rand
coverity#1242377 Don't call rand
coverity#1242378 Don't call rand
coverity#1242379 Don't call rand
coverity#1242382 Don't call rand
coverity#1242383 Don't call rand
coverity#1242402 Don't call rand
coverity#1242397 Don't call rand
coverity#1242390 Don't call rand
coverity#1242389 Don't call rand
coverity#1242388 Don't call rand
coverity#1242386 Don't call rand
coverity#1242384 Don't call rand
coverity#1242394 Don't call rand
Change-Id: I241feab9cb370e091fd6ccaba2af941eb95bc7cf
Diffstat (limited to 'sw/qa/core/uwriter.cxx')
| -rw-r--r-- | sw/qa/core/uwriter.cxx | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/sw/qa/core/uwriter.cxx b/sw/qa/core/uwriter.cxx index ec62c05366ee..a739ca0c0e96 100644 --- a/sw/qa/core/uwriter.cxx +++ b/sw/qa/core/uwriter.cxx @@ -16,6 +16,7 @@ #include <com/sun/star/i18n/TransliterationModulesExtra.hpp> #include <comphelper/processfactory.hxx> +#include <comphelper/random.hxx> #include <tools/urlobj.hxx> #include <unotools/tempfile.hxx> #include <unotools/transliterationwrapper.hxx> @@ -849,7 +850,7 @@ getRand(int modulus) { if (modulus <= 0) return 0; - return rand() % modulus; + return comphelper::rng::uniform_int_distribution(0, modulus-1); } static OUString @@ -869,12 +870,13 @@ static SwPosition getRandomPosition(SwDoc *pDoc, int /* nOffset */) { const SwPosition aPos(pDoc->GetNodes().GetEndOfContent()); - sal_uLong nNodes = aPos.nNode.GetNode().GetIndex() - aPos.nNode.GetNode().StartOfSectionIndex(); - sal_uLong n = (rand() * nNodes) / RAND_MAX; - SwPaM pam(aPos); - for (sal_uLong i = 0; i < n; ++i) { - pam.Move(fnMoveBackward, fnGoNode); - } + sal_uLong nNodes = aPos.nNode.GetNode().GetIndex() - aPos.nNode.GetNode().StartOfSectionIndex(); + sal_uLong n = comphelper::rng::uniform_int_distribution(static_cast<sal_uLong>(0), nNodes); + SwPaM pam(aPos); + for (sal_uLong i = 0; i < n; ++i) + { + pam.Move(fnMoveBackward, fnGoNode); + } return *pam.GetPoint(); } |