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/chksum.c b/chksum.c
index ad12d7d4..215d4f57 100644
--- a/chksum.c
+++ b/chksum.c
@@ -97,7 +97,7 @@ static val chksum_ensure_buf(val self, val buf_in,
 {
   if (null_or_missing_p(buf_in)) {
     *phash = chk_malloc(c_unum(len, self));
-    return make_borrowed_buf(len, *phash);
+    return make_owned_buf(len, *phash);
   } else {
     *phash = buf_get(buf_in, self);
     if (lt(length_buf(buf_in), len))