diff options
| author | Bipin Kunal <bkunal@redhat.com> | 2016-06-10 16:21:37 +0530 |
|---|---|---|
| committer | Niels de Vos <ndevos@redhat.com> | 2016-07-03 04:15:15 -0700 |
| commit | e6d45abffb861570f9591811c77f0d3d8316f637 (patch) | |
| tree | 7dcaa3cf324cb7256b15955e8a9c1646849436c6 /api | |
| parent | 2687bc68a5f91e8f940a8eef1b47052736116556 (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.
This is backport of below mainline fix -
http://review.gluster.org/#/c/14666/
Change-Id: I9d04eaf4c65bdb8db136321d60e70789da1739d7
BUG: 1343290
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Signed-off-by: Bipin Kunal <bkunal@redhat.com>
Reviewed-on: http://review.gluster.org/14697
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'api')
0 files changed, 0 insertions, 0 deletions