[PATCH] secur32: Use the ARRAY_SIZE() macro

Michael Stefaniuc mstefani at winehq.org
Wed Jul 18 16:01:32 CDT 2018


Signed-off-by: Michael Stefaniuc <mstefani at winehq.org>
---
 dlls/secur32/schannel.c        | 8 ++++----
 dlls/secur32/schannel_gnutls.c | 2 +-
 dlls/secur32/schannel_macosx.c | 4 ++--
 dlls/secur32/secur32.c         | 4 ++--
 4 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/dlls/secur32/schannel.c b/dlls/secur32/schannel.c
index f0d271e0d8..6d50c97fab 100644
--- a/dlls/secur32/schannel.c
+++ b/dlls/secur32/schannel.c
@@ -195,7 +195,7 @@ static void read_config(void)
     if(res == ERROR_SUCCESS) {
         DWORD type, size, value;
 
-        for(i=0; i < sizeof(protocol_config_keys)/sizeof(*protocol_config_keys); i++) {
+        for(i = 0; i < ARRAY_SIZE(protocol_config_keys); i++) {
             strcpyW(subkey_name, protocol_config_keys[i].key_name);
             strcatW(subkey_name, clientW);
             res = RegOpenKeyExW(protocols_key, subkey_name, 0, KEY_READ, &key);
@@ -229,7 +229,7 @@ static void read_config(void)
         }
     }else {
         /* No config, enable all known protocols. */
-        for(i=0; i < sizeof(protocol_config_keys)/sizeof(*protocol_config_keys); i++) {
+        for(i = 0; i < ARRAY_SIZE(protocol_config_keys); i++) {
             if(protocol_config_keys[i].enabled)
                 enabled |= protocol_config_keys[i].prot_client_flag;
             if(protocol_config_keys[i].disabled_by_default)
@@ -986,7 +986,7 @@ static void *get_alg_name(ALG_ID id, BOOL wide)
     };
     unsigned i;
 
-    for (i = 0; i < sizeof(alg_name_map)/sizeof(alg_name_map[0]); i++)
+    for (i = 0; i < ARRAY_SIZE(alg_name_map); i++)
         if (alg_name_map[i].alg_id == id)
             return wide ? (void*)alg_name_map[i].nameW : (void*)alg_name_map[i].name;
 
@@ -1545,7 +1545,7 @@ void SECUR32_initSchannelSP(void)
         goto fail;
     }
 
-    SECUR32_addPackages(provider, sizeof(info) / sizeof(info[0]), NULL, info);
+    SECUR32_addPackages(provider, ARRAY_SIZE(info), NULL, info);
 
     return;
 
diff --git a/dlls/secur32/schannel_gnutls.c b/dlls/secur32/schannel_gnutls.c
index bc3bbaf67d..05e922dfe8 100644
--- a/dlls/secur32/schannel_gnutls.c
+++ b/dlls/secur32/schannel_gnutls.c
@@ -177,7 +177,7 @@ BOOL schan_imp_create_session(schan_imp_session *session, schan_credentials *cre
     }
 
     p = priority + strlen(priority);
-    for(i=0; i < sizeof(protocol_priority_flags)/sizeof(*protocol_priority_flags); i++) {
+    for(i = 0; i < ARRAY_SIZE(protocol_priority_flags); i++) {
         *p++ = ':';
         *p++ = (cred->enabled_protocols & protocol_priority_flags[i].enable_flag) ? '+' : '-';
         strcpy(p, protocol_priority_flags[i].gnutls_flag);
diff --git a/dlls/secur32/schannel_macosx.c b/dlls/secur32/schannel_macosx.c
index a2f42615e1..bd080dcb41 100644
--- a/dlls/secur32/schannel_macosx.c
+++ b/dlls/secur32/schannel_macosx.c
@@ -429,7 +429,7 @@ static const struct cipher_suite cipher_suites[] = {
 static const struct cipher_suite* get_cipher_suite(SSLCipherSuite cipher_suite)
 {
     int i;
-    for (i = 0; i < sizeof(cipher_suites)/sizeof(cipher_suites[0]); i++)
+    for (i = 0; i < ARRAY_SIZE(cipher_suites); i++)
     {
         if (cipher_suites[i].suite == cipher_suite)
             return &cipher_suites[i];
@@ -767,7 +767,7 @@ BOOL schan_imp_create_session(schan_imp_session *session, schan_credentials *cre
         goto fail;
     }
 
-    for(i=0; i < sizeof(protocol_priority_flags)/sizeof(*protocol_priority_flags); i++) {
+    for(i = 0; i < ARRAY_SIZE(protocol_priority_flags); i++) {
         if(!(protocol_priority_flags[i].enable_flag & supported_protocols))
            continue;
 
diff --git a/dlls/secur32/secur32.c b/dlls/secur32/secur32.c
index 9240c108e5..ec189fda10 100644
--- a/dlls/secur32/secur32.c
+++ b/dlls/secur32/secur32.c
@@ -995,7 +995,7 @@ BOOLEAN WINAPI GetComputerObjectNameW(
         case NameSamCompatible:
             {
                 WCHAR name[MAX_COMPUTERNAME_LENGTH + 1];
-                DWORD size = sizeof(name)/sizeof(name[0]);
+                DWORD size = ARRAY_SIZE(name);
                 if (GetComputerNameW(name, &size))
                 {
                     DWORD len = domainInfo->Name.Length + size + 3;
@@ -1038,7 +1038,7 @@ BOOLEAN WINAPI GetComputerObjectNameW(
             DWORD len, size;
             WCHAR *suffix;
 
-            size = sizeof(name) / sizeof(name[0]);
+            size = ARRAY_SIZE(name);
             if (!GetComputerNameW(name, &size))
             {
                 status = FALSE;
-- 
2.14.4




More information about the wine-devel mailing list