diff options
| author | Jonathan Gray <jsg@cvs.openbsd.org> | 2009-07-12 15:36:42 +0000 |
|---|---|---|
| committer | Jonathan Gray <jsg@cvs.openbsd.org> | 2009-07-12 15:36:42 +0000 |
| commit | 91ec1b1ff68b3a4c7976633d11967ce1c32930c2 (patch) | |
| tree | 5c3165ea49f7c6de342637c5fcfab29774049679 /usr.sbin | |
| parent | 4b971934c121486f374cf6c11dbd7d35a6c9f7e9 (diff) | |
Remove a semicolon at the end of an if statement so the
test if a buffer isn't NULL before freeing has effect.
This was harmless as it is in a failure case and free(NULL) is
valid in userland.
Diffstat (limited to 'usr.sbin')
| -rw-r--r-- | usr.sbin/bgpd/mrt.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/usr.sbin/bgpd/mrt.c b/usr.sbin/bgpd/mrt.c index 5fa497a2b26..932fa8d3392 100644 --- a/usr.sbin/bgpd/mrt.c +++ b/usr.sbin/bgpd/mrt.c @@ -1,4 +1,4 @@ -/* $OpenBSD: mrt.c,v 1.63 2009/06/29 12:22:16 claudio Exp $ */ +/* $OpenBSD: mrt.c,v 1.64 2009/07/12 15:36:41 jsg Exp $ */ /* * Copyright (c) 2003, 2004 Claudio Jeker <claudio@openbsd.org> @@ -300,7 +300,7 @@ mrt_dump_entry_mp(struct mrt *mrt, struct prefix *p, u_int16_t snum, fail: if (hbuf) buf_free(hbuf); - if (h2buf); + if (h2buf) buf_free(h2buf); buf_free(buf); return (-1); |