diff --git a/lib/include/srsran/upper/pdcp_entity_base.h b/lib/include/srsran/upper/pdcp_entity_base.h
index 55b5d768e..75bb8f0b8 100644
--- a/lib/include/srsran/upper/pdcp_entity_base.h
+++ b/lib/include/srsran/upper/pdcp_entity_base.h
@@ -57,8 +57,8 @@ public:
   pdcp_entity_base(pdcp_entity_base&&) = default;
   virtual ~pdcp_entity_base();
   virtual bool configure(const pdcp_config_t& cnfg_) = 0;
-  virtual void reset()       = 0;
-  virtual void reestablish() = 0;
+  virtual void reset()                               = 0;
+  virtual void reestablish()                         = 0;
 
   bool is_active() { return active; }
   bool is_srb() { return cfg.rb_type == PDCP_RB_IS_SRB; }
@@ -106,7 +106,7 @@ public:
     }
   }
 
-  void config_security(as_security_config_t sec_cfg_);
+  void config_security(const as_security_config_t& sec_cfg_);
 
   // GW/SDAP/RRC interface
   virtual void write_sdu(unique_byte_buffer_t sdu, int sn = -1) = 0;
diff --git a/lib/src/upper/pdcp_entity_base.cc b/lib/src/upper/pdcp_entity_base.cc
index e3706c962..22e3966ad 100644
--- a/lib/src/upper/pdcp_entity_base.cc
+++ b/lib/src/upper/pdcp_entity_base.cc
@@ -23,7 +23,7 @@ pdcp_entity_base::pdcp_entity_base(task_sched_handle task_sched_, srslog::basic_
 
 pdcp_entity_base::~pdcp_entity_base() {}
 
-void pdcp_entity_base::config_security(as_security_config_t sec_cfg_)
+void pdcp_entity_base::config_security(const as_security_config_t& sec_cfg_)
 {
   sec_cfg = sec_cfg_;