sha256/md5: leak: don't use borrowed buffer for hash.

It's incorrect to use a borrowed buf, because a borrowed buf
assumes that the memory isn't dynamically allocated, and will
not free it when it is finalized by the garbage collector.
The buffer which holds the hash returned by sha256 and md5
does in fact own the memory.

* buf.c (make_owned_buf): Static function becomes external.

* buf.h (make_owned_buf): Declared.

* chksum.c (chksum_ensure_buf): Use make_owned_buf rather than
make_borrowed_buf.

1 files changed, 1 insertions, 1 deletions

diff --git a/buf.c b/buf.c
index 4305952e..5d0fb268 100644
--- a/buf.c
+++ b/buf.c
@@ -130,7 +130,7 @@ val make_duplicate_buf(val len, mem_t *data)
   return obj;
 }
 
-static val make_owned_buf(val len, mem_t *data)
+val make_owned_buf(val len, mem_t *data)
 {
   val buf = make_borrowed_buf(len, data);
   buf->b.size = len;