diff --git a/src/bm_sim/P4Objects.cpp b/src/bm_sim/P4Objects.cpp
index f254dd8c6..c17bdcf7b 100644
--- a/src/bm_sim/P4Objects.cpp
+++ b/src/bm_sim/P4Objects.cpp
@@ -653,7 +653,7 @@ P4Objects::init_header_types(const Json::Value &cfg_root) {
                                              header_type_id);
 
     const Json::Value &cfg_fields = cfg_header_type["fields"];
-    for (const auto cfg_field : cfg_fields) {
+    for (const auto &cfg_field : cfg_fields) {
       const string field_name = cfg_field[0].asString();
       bool is_signed = false;
       if (cfg_field.size() > 2)
diff --git a/src/bm_sim/simple_pre.cpp b/src/bm_sim/simple_pre.cpp
index c60f777c8..eecac0c29 100644
--- a/src/bm_sim/simple_pre.cpp
+++ b/src/bm_sim/simple_pre.cpp
@@ -184,7 +184,7 @@ McSimplePre::mc_node_update(const l1_hdl_t l1_hdl,
 void
 McSimplePre::get_entries_common(Json::Value *root) const {
   Json::Value mgrps(Json::arrayValue);
-  for (const auto p : mgid_entries) {
+  for (const auto &p : mgid_entries) {
     Json::Value mgrp(Json::objectValue);
     mgrp["id"] = Json::Value(Json::UInt(p.first));
 
@@ -212,7 +212,7 @@ McSimplePre::get_entries_common(Json::Value *root) const {
   (*root)["l1_handles"] = l1_handles;
 
   Json::Value l2_handles(Json::arrayValue);
-  for (const auto p : l2_entries) {
+  for (const auto &p : l2_entries) {
     Json::Value handle(Json::objectValue);
     handle["handle"] = Json::Value(Json::UInt(p.first));
     const auto &entry = p.second;