diff --git a/plugins/sms/smsplugin.cpp b/plugins/sms/smsplugin.cpp
index 27074584f..335447544 100644
--- a/plugins/sms/smsplugin.cpp
+++ b/plugins/sms/smsplugin.cpp
@@ -62,7 +62,7 @@ void SmsPlugin::sendSms(const QDBusVariant& addresses, const QString& messageBod
     QList<ConversationAddress> addressList = ConversationAddress::listfromDBus(addresses);
 
     QVariantList addressMapList;
-    for (const ConversationAddress address : addressList) {
+    for (const ConversationAddress& address : addressList) {
         QVariantMap addressMap({{QStringLiteral("address"), address.address()}});
         addressMapList.append(addressMap);
         qDebug() <<address.address();
diff --git a/smsapp/conversationssortfilterproxymodel.cpp b/smsapp/conversationssortfilterproxymodel.cpp
index f90a26b32..6ca2432c0 100644
--- a/smsapp/conversationssortfilterproxymodel.cpp
+++ b/smsapp/conversationssortfilterproxymodel.cpp
@@ -67,8 +67,8 @@ bool ConversationsSortFilterProxyModel::filterAcceptsRow(int sourceRow, const QM
         }
 
         // This block of code compares each address in the multi target conversation to find a match
-        QList<ConversationAddress> addressList = sourceModel()->data(index, ConversationListModel::AddressesRole).value<QList<ConversationAddress>>();
-        for (const ConversationAddress address : addressList) {
+        const QList<ConversationAddress> addressList = sourceModel()->data(index, ConversationListModel::AddressesRole).value<QList<ConversationAddress>>();
+        for (const ConversationAddress& address : addressList) {
             QString canonicalAddress = SmsHelper::canonicalizePhoneNumber(address.address());
             if (canonicalAddress.contains(filterRegExp())) {
                 return true;