diff options
| author | Mark Felder <feld@FreeBSD.org> | 2014-11-19 13:57:39 +0000 |
|---|---|---|
| committer | Mark Felder <feld@FreeBSD.org> | 2014-11-19 13:57:39 +0000 |
| commit | 46ad509bf8c5e3fd881a70be4e4bc2cced054b86 (patch) | |
| tree | 6fc0b555641c30cccd037dd8aeaef10136a2a0e1 /sbin/ifconfig/ifgroup.c | |
| parent | 7b56cc430a261630677f614b1cf4283e6ebb34fc (diff) | |
| download | src-46ad509bf8c5e3fd881a70be4e4bc2cced054b86.tar.gz src-46ad509bf8c5e3fd881a70be4e4bc2cced054b86.zip | |
Expose groups by default in ifconfig output. This was never hidden by
OpenBSD; unsure why we chose to do so. As groups are a requirement for
pf, exposing them by default will make our pf implementation less
confusing.
While here add a missing free() that OpenBSD fixed 7 years ago.
PR: 194925
Differential Revision: https://reviews.freebsd.org/D1185
Approved by: des
Obtained from: OpenBSD
Notes
Notes:
svn path=/head/; revision=274710
Diffstat (limited to 'sbin/ifconfig/ifgroup.c')
| -rw-r--r-- | sbin/ifconfig/ifgroup.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/sbin/ifconfig/ifgroup.c b/sbin/ifconfig/ifgroup.c index f8b18b4a9835..e3f271d4429f 100644 --- a/sbin/ifconfig/ifgroup.c +++ b/sbin/ifconfig/ifgroup.c @@ -86,9 +86,6 @@ getifgroups(int s) struct ifgroupreq ifgr; struct ifg_req *ifg; - if (!verbose) - return; - memset(&ifgr, 0, sizeof(ifgr)); strlcpy(ifgr.ifgr_name, name, IFNAMSIZ); @@ -121,6 +118,8 @@ getifgroups(int s) } if (cnt) printf("\n"); + + free(ifgr.ifgr_groups); } static void |