From 3e8d185af326bf183b6f78597d5e3d2eeb2dc40e Mon Sep 17 00:00:00 2001 From: "millert@openbsd.org" Date: Tue, 19 Sep 2017 12:10:30 +0000 Subject: [PATCH] upstream commit Use explicit_bzero() instead of bzero() before free() to prevent the compiler from optimizing away the bzero() call. OK djm@ Upstream-ID: cdc6197e64c9684c7250e23d60863ee1b53cef1d --- channels.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/channels.c b/channels.c index 72f0e7709..89b7d3486 100644 --- a/channels.c +++ b/channels.c @@ -1,4 +1,4 @@ -/* $OpenBSD: channels.c,v 1.370 2017/09/12 07:55:48 djm Exp $ */ +/* $OpenBSD: channels.c,v 1.371 2017/09/19 12:10:30 millert Exp $ */ /* * Author: Tatu Ylonen * Copyright (c) 1995 Tatu Ylonen , Espoo, Finland @@ -573,7 +573,7 @@ channel_free(struct ssh *ssh, Channel *c) if (c->filter_cleanup != NULL && c->filter_ctx != NULL) c->filter_cleanup(ssh, c->self, c->filter_ctx); sc->channels[c->self] = NULL; - bzero(c, sizeof(*c)); + explicit_bzero(c, sizeof(*c)); free(c); }