diff options
author | Bipin Kunal <bkunal@redhat.com> | 2016-06-07 17:59:27 +0530 |
---|---|---|
committer | Niels de Vos <ndevos@redhat.com> | 2016-06-07 15:24:10 -0700 |
commit | 8489fe576b6cf148fabcaeee79f298c41858133e (patch) | |
tree | 6253b448ab1b34f1e6811fbbccc4c3eab79e8606 /xlators/mgmt/glusterd/src/glusterd-volgen.c | |
parent | bdde0cfa1d5848e59d20c1adc19ee1b687eec66a (diff) |
xdr/nfs: free complete groupnode structure
The groupnode->gr_next pointer is not traversed upon free. This is
currently not a problem, because the pointer is never used. However the
correct way to free a groupnode should check the ->gr_next pointer and
free any of the groups that it encounters.
This problem was identified while correcting a problem with the MOUNT
protocol. The change "nfs: build exportlist with multiple groups" starts
to use ->gr_next.
Change-Id: I9d04eaf4c65bdb8db136321d60e70789da1739d7
BUG: 1343286
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: http://review.gluster.org/14666
Smoke: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: bipin kunal <kunalbipin@gmail.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-volgen.c')
0 files changed, 0 insertions, 0 deletions