Addressed @andrepuschmann comments

srsue/src/stack/mac/mux.cc

@@ -395,7 +395,7 @@ uint8_t* mux::msg3_get(srslte::byte_buffer_t* payload, uint32_t pdu_sz)
     return payload->msg;
   } else {
     Error("Msg3 size exceeds buffer\n");
-    return NULL;
+    return nullptr;
   }
 }
 

srsue/src/stack/mac/proc_ra.cc

@@ -488,7 +488,7 @@ bool ra_proc::is_contention_resolution()
 /* Perform the actions upon completition of the RA procedure as defined in 5.1.6 */
 void ra_proc::complete()
 {
-  /* Start looking for PDCCH CRNTI */
+  // Start looking for PDCCH CRNTI
   if (!transmitted_crnti) {
     rntis->crnti = rntis->temp_rnti;
   }

srsue/test/mac_test.cc

@@ -31,7 +31,7 @@
 using namespace srsue;
 using namespace srslte;
 
-#define HAVE_PCAP 1
+#define HAVE_PCAP 0
 
 static std::unique_ptr<srslte::mac_pcap> pcap_handle = nullptr;
 
@@ -206,8 +206,7 @@ public:
   {
 
     // Generate RAR to MAC
-    uint8_t grant[SRSLTE_RAR_GRANT_LEN] = {};
+    uint8_t grant[SRSLTE_RAR_GRANT_LEN] = {1};
-    memset(grant, 1, SRSLTE_RAR_GRANT_LEN);
 
     uint32_t rar_timeadv = 16;
 
@@ -1302,9 +1301,8 @@ int mac_random_access_test()
   uint32 tti = 0;
 
   // Structure that defines the test to be executed
-  struct ra_test my_test;
+  struct ra_test my_test           = {};
   uint32_t       test_id = 1;
-  ZERO_OBJECT(my_test);
   my_test.temp_rnti                = 100;
   my_test.assume_prach_transmitted = -1;