crypto: use ByteSource::Builder in To*Copy

Refs: nodejs#43202
tniessen · Jun 18, 2022 · d14d479 · d14d479
1 parent a36a546
commit d14d479
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/src/crypto/crypto_util.h b/src/crypto/crypto_util.h
@@ -747,19 +747,17 @@ class ArrayBufferOrViewContents {
 
   inline ByteSource ToCopy() const {
     if (size() == 0) return ByteSource();
-    char* buf = MallocOpenSSL<char>(size());
-    CHECK_NOT_NULL(buf);
-    memcpy(buf, data(), size());
-    return ByteSource::Allocated(buf, size());
+    ByteSource::Builder buf(size());
+    memcpy(buf.data<void>(), data(), size());
+    return std::move(buf).release();
   }
 
   inline ByteSource ToNullTerminatedCopy() const {
     if (size() == 0) return ByteSource();
-    char* buf = MallocOpenSSL<char>(size() + 1);
-    CHECK_NOT_NULL(buf);
-    buf[size()] = 0;
-    memcpy(buf, data(), size());
-    return ByteSource::Allocated(buf, size());
+    ByteSource::Builder buf(size() + 1);
+    memcpy(buf.data<char>(), data(), size());
+    buf.data<char>()[size()] = 0;
+    return std::move(buf).release();
   }
 
   template <typename M>