diff --git a/src/lib/deskflow/unix/AppUtilUnix.cpp b/src/lib/deskflow/unix/AppUtilUnix.cpp index bd31a0bf1..3f3990a80 100644 --- a/src/lib/deskflow/unix/AppUtilUnix.cpp +++ b/src/lib/deskflow/unix/AppUtilUnix.cpp @@ -77,7 +77,7 @@ std::vector AppUtilUnix::getKeyboardLayoutList() layoutLanguages = (CFArrayRef)TISGetInputSourceProperty(keyboardLayout, kTISPropertyInputSourceLanguages); } char temporaryCString[128] = {0}; - for (CFIndex index = 0; index < CFArrayGetCount(layoutLanguages) && layoutLanguages; index++) { + for (CFIndex index = 0; layoutLanguages && index < CFArrayGetCount(layoutLanguages); index++) { auto languageCode = (CFStringRef)CFArrayGetValueAtIndex(layoutLanguages, index); if (!languageCode || !CFStringGetCString(languageCode, temporaryCString, 128, kCFStringEncodingUTF8)) { continue; @@ -163,7 +163,7 @@ std::string AppUtilUnix::getCurrentLanguageCode() layoutLanguages = (CFArrayRef)TISGetInputSourceProperty(source.get(), kTISPropertyInputSourceLanguages); } char temporaryCString[128] = {0}; - for (CFIndex index = 0; index < CFArrayGetCount(layoutLanguages) && layoutLanguages; index++) { + for (CFIndex index = 0; layoutLanguages && index < CFArrayGetCount(layoutLanguages); index++) { auto languageCode = (CFStringRef)CFArrayGetValueAtIndex(layoutLanguages, index); if (!languageCode || !CFStringGetCString(languageCode, temporaryCString, 128, kCFStringEncodingUTF8)) { continue;