From: ANZ1217 <chihun.jeong@samsung.com>
Date: Wed, 16 Aug 2023 08:17:15 +0000 (+0900)
Subject: Fix coverity issue (copy data during iteration)
X-Git-Tag: accepted/tizen/unified/20230821.103303~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=06323c78682eed27c5012a5b1303be6637092020;p=platform%2Fcore%2Fuifw%2Faurum.git

Fix coverity issue (copy data during iteration)

Change-Id: I629a029f987242addf7bf69ec5b3a4af34f0226d
---

diff --git a/libaurum/src/Impl/Accessibility/AtspiAccessibleWatcher.cc b/libaurum/src/Impl/Accessibility/AtspiAccessibleWatcher.cc
index 8684f55..81cdb20 100644
--- a/libaurum/src/Impl/Accessibility/AtspiAccessibleWatcher.cc
+++ b/libaurum/src/Impl/Accessibility/AtspiAccessibleWatcher.cc
@@ -575,7 +575,7 @@ void AtspiAccessibleWatcher::setXMLsync(bool sync)
         mAppXMLLoadedCount = 0;
     } else {
         auto apps = getApplications();
-        for (auto app : apps)
+        for (auto &app : apps)
         {
             app->getAccessibleNode()->updateName();
             mAppCount++;
diff --git a/libaurum/src/Impl/MockDeviceImpl.cc b/libaurum/src/Impl/MockDeviceImpl.cc
index 6c72a24..db933f4 100644
--- a/libaurum/src/Impl/MockDeviceImpl.cc
+++ b/libaurum/src/Impl/MockDeviceImpl.cc
@@ -272,7 +272,7 @@ std::vector<std::shared_ptr<AccessibleNode>> MockDeviceImpl::getWindowRoot() con
 
     auto apps = AccessibleWatcher::getInstance()->getApplications();
 
-    for (auto app : apps)
+    for (auto &app : apps)
     {
         auto wins = app->getWindows();
         std::transform(wins.begin(), wins.end(), std::back_inserter(ret),
diff --git a/libaurum/src/Impl/TizenDeviceImpl.cc b/libaurum/src/Impl/TizenDeviceImpl.cc
index bd29ad2..c99b130 100644
--- a/libaurum/src/Impl/TizenDeviceImpl.cc
+++ b/libaurum/src/Impl/TizenDeviceImpl.cc
@@ -448,7 +448,7 @@ std::vector<std::shared_ptr<AccessibleNode>> TizenDeviceImpl::getWindowRoot() co
     std::unordered_map<int, std::shared_ptr<AccessibleApplication>> pidToAppNode{};
 
     auto apps = AccessibleWatcher::getInstance()->getApplications();
-    for (auto app : apps)
+    for (auto &app : apps)
     {
         app->getAccessibleNode()->updateName();
         app->getAccessibleNode()->updatePid();
@@ -457,7 +457,7 @@ std::vector<std::shared_ptr<AccessibleNode>> TizenDeviceImpl::getWindowRoot() co
         pidToAppNode[app->getAccessibleNode()->getPid()] = app;
     }
 
-    for (auto tWin : mTizenWindows)
+    for (auto &tWin : mTizenWindows)
     {
         LOGI("Visible win (%d) (%d %d %d %d) (%s)", tWin->getPid(), tWin->getWindowGeometry().mTopLeft.x, tWin->getWindowGeometry().mTopLeft.y, tWin->getWindowGeometry().width(),
             tWin->getWindowGeometry().height(), tWin->getName().c_str());