diff --git a/attn/attn_handler.cpp b/attn/attn_handler.cpp
index caf5265..a4abf74 100644
--- a/attn/attn_handler.cpp
+++ b/attn/attn_handler.cpp
@@ -52,7 +52,6 @@
     std::vector<Attention> active_attentions;
 
     uint32_t isr_val, isr_mask;
-    uint32_t proc;
 
     std::stringstream ss; // for trace messages
 
@@ -64,7 +63,7 @@
     {
         if (PDBG_TARGET_ENABLED == pdbg_target_probe(target))
         {
-            proc = pdbg_target_index(target); // get processor number
+            auto proc = pdbg_target_index(target); // get processor number
 
             // Use PIB target to determine if a processor is enabled
             char path[16];
diff --git a/attn/attn_logging.cpp b/attn/attn_logging.cpp
index 48ee5be..3aa0889 100644
--- a/attn/attn_logging.cpp
+++ b/attn/attn_logging.cpp
@@ -36,12 +36,7 @@
     createFFDCTuples(const std::vector<util::FFDCFile>& files)
 {
     std::vector<FFDCTuple> ffdcTuples{};
-    for (const util::FFDCFile& file : files)
-    {
-        ffdcTuples.emplace_back(
-            file.getFormat(), file.getSubType(), file.getVersion(),
-            sdbusplus::message::unix_fd(file.getFileDescriptor()));
-    }
+    util::transformFFDC(files, ffdcTuples);
 
     return ffdcTuples;
 }