diff options
| author | mmcc@openbsd.org <mmcc@openbsd.org> | 2015-12-11 05:21:11 +0100 |
|---|---|---|
| committer | Damien Miller <djm@mindrot.org> | 2015-12-18 04:50:48 +0100 |
| commit | 52d7078421844b2f88329f5be3de370b0a938636 (patch) | |
| tree | 1241462a841b14259755de181563cc83acd42109 /sshbuf.c | |
| parent | upstream commit (diff) | |
| download | openssh-52d7078421844b2f88329f5be3de370b0a938636.tar.xz openssh-52d7078421844b2f88329f5be3de370b0a938636.zip | |
upstream commit
Remove NULL-checks before sshbuf_free().
ok djm@
Upstream-ID: 5ebed00ed5f9f03b119a345085e8774565466917
Diffstat (limited to 'sshbuf.c')
| -rw-r--r-- | sshbuf.c | 8 |
1 files changed, 3 insertions, 5 deletions
@@ -1,4 +1,4 @@ -/* $OpenBSD: sshbuf.c,v 1.4 2015/10/05 17:11:21 djm Exp $ */ +/* $OpenBSD: sshbuf.c,v 1.5 2015/12/11 04:21:12 mmcc Exp $ */ /* * Copyright (c) 2011 Damien Miller * @@ -163,10 +163,8 @@ sshbuf_free(struct sshbuf *buf) * If we are a child, the free our parent to decrement its reference * count and possibly free it. */ - if (buf->parent != NULL) { - sshbuf_free(buf->parent); - buf->parent = NULL; - } + sshbuf_free(buf->parent); + buf->parent = NULL; /* * If we are a parent with still-extant children, then don't free just * yet. The last child's call to sshbuf_free should decrement our |