diff --git a/handler/minidump_to_upload_parameters.cc b/handler/minidump_to_upload_parameters.cc
index 9e26d877..6b64081e 100644
--- a/handler/minidump_to_upload_parameters.cc
+++ b/handler/minidump_to_upload_parameters.cc
@@ -50,7 +50,7 @@ std::map<std::string, std::string> BreakpadHTTPFormParametersFromMinidump(
       }
     }
 
-    for (std::string annotation : module->AnnotationsVector()) {
+    for (const std::string& annotation : module->AnnotationsVector()) {
       list_annotations.append(annotation);
       list_annotations.append("\n");
     }
diff --git a/minidump/minidump_unloaded_module_writer.cc b/minidump/minidump_unloaded_module_writer.cc
index 855e1967..c8a5f0fd 100644
--- a/minidump/minidump_unloaded_module_writer.cc
+++ b/minidump/minidump_unloaded_module_writer.cc
@@ -123,7 +123,8 @@ void MinidumpUnloadedModuleListWriter::InitializeFromSnapshot(
   DCHECK_EQ(state(), kStateMutable);
   DCHECK(unloaded_modules_.empty());
 
-  for (auto unloaded_module_snapshot : unloaded_module_snapshots) {
+  for (const UnloadedModuleSnapshot& unloaded_module_snapshot :
+       unloaded_module_snapshots) {
     auto unloaded_module = std::make_unique<MinidumpUnloadedModuleWriter>();
     unloaded_module->InitializeFromSnapshot(unloaded_module_snapshot);
     AddUnloadedModule(std::move(unloaded_module));