Use freezero for better readability
authortobias <tobias@openbsd.org>
Wed, 14 Aug 2024 15:37:11 +0000 (15:37 +0000)
committertobias <tobias@openbsd.org>
Wed, 14 Aug 2024 15:37:11 +0000 (15:37 +0000)
It has the same meaning as the current pair of calling explicit_bzero
and free. Spotted with Benny Baumann (BenBE at geshi dot org).

ok djm@

usr.bin/ssh/sshbuf.c

index f76a2f3..9756867 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: sshbuf.c,v 1.20 2024/08/14 15:35:23 tobias Exp $      */
+/*     $OpenBSD: sshbuf.c,v 1.21 2024/08/14 15:37:11 tobias Exp $      */
 /*
  * Copyright (c) 2011 Damien Miller
  *
@@ -183,10 +183,8 @@ sshbuf_free(struct sshbuf *buf)
        sshbuf_free(buf->parent);
        buf->parent = NULL;
 
-       if (!buf->readonly) {
-               explicit_bzero(buf->d, buf->alloc);
-               free(buf->d);
-       }
+       if (!buf->readonly)
+               freezero(buf->d, buf->alloc);
        freezero(buf, sizeof(*buf));
 }