diff --git a/liberty/LibertyParser.cc b/liberty/LibertyParser.cc
index 6e9b3442..14f6582b 100644
--- a/liberty/LibertyParser.cc
+++ b/liberty/LibertyParser.cc
@@ -336,11 +336,9 @@ makeLibertyComplexAttr(const char *name,
         group->addAttribute(attr);
         return attr;
       }
-      else {
-        delete attr;
-        return nullptr;
-      }
     }
+    delete attr;
+    return nullptr;
   }
 }
 
diff --git a/sdf/SdfWriter.cc b/sdf/SdfWriter.cc
index 4ffd26e7..7d4c93b2 100644
--- a/sdf/SdfWriter.cc
+++ b/sdf/SdfWriter.cc
@@ -164,8 +164,7 @@ SdfWriter::write(const char *filename,
   sdf_divider_ = sdf_divider;
   include_typ_ = include_typ;
   if (delay_format_ == nullptr)
-    delay_format_ = new char[10];
-  sprintf(delay_format_, "%%.%df", digits);
+    delay_format_ = stringPrint("%%.%df", digits);
 
   LibertyLibrary *default_lib = network_->defaultLibertyLibrary();
   timescale_ = default_lib->units()->timeUnit()->scale();