diff options
author | Julien Nabet <serval2412@yahoo.fr> | 2013-08-31 21:08:42 +0200 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2013-09-01 09:17:44 +0000 |
commit | 777a659cf8814b7022ce9e1156f3a3b9f3322978 (patch) | |
tree | 9025d2499c8c7f18456e5a8f926250e8fbc1f7bd /setup_native | |
parent | 67ef58d1b338b4a57cae69174de987626250c651 (diff) |
cppcheck: Array index is used before limits check
Change-Id: I531430ce4bc0937a023d3e2849ae07d8f94e3e70
Reviewed-on: https://gerrit.libreoffice.org/5723
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'setup_native')
-rw-r--r-- | setup_native/source/win32/customactions/reg4allmsdoc/reg4allmsi.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/setup_native/source/win32/customactions/reg4allmsdoc/reg4allmsi.cxx b/setup_native/source/win32/customactions/reg4allmsdoc/reg4allmsi.cxx index a0f930ad7d35..40ed84b84bac 100644 --- a/setup_native/source/win32/customactions/reg4allmsdoc/reg4allmsi.cxx +++ b/setup_native/source/win32/customactions/reg4allmsdoc/reg4allmsi.cxx @@ -280,7 +280,7 @@ static bool checkSomeExtensionInRegistry( const int nStart, const int nEnd ) int nIndex = nStart; bool bFound = false; - while ( !bFound && ( g_Extensions[nIndex] != 0 ) && ( nIndex < nEnd ) ) + while ( !bFound && (nIndex < nEnd) && (g_Extensions[nIndex] != 0) ) { bFound = ! CheckExtensionInRegistry( g_Extensions[nIndex] ); @@ -296,7 +296,7 @@ static void registerSomeExtensions( MSIHANDLE handle, const int nStart, const in { // Check all file extensions int nIndex = nStart; - while ( ( g_Extensions[nIndex] != 0 ) && ( nIndex < nEnd ) ) + while ( (nIndex < nEnd) && (g_Extensions[nIndex] != 0) ) { registerForExtension( handle, nIndex++, bRegister ); } |