diff options
author | Prashanth Pai <ppai@redhat.com> | 2015-06-14 16:40:58 +0530 |
---|---|---|
committer | Raghavendra Bhat <raghavendra@redhat.com> | 2015-06-26 03:43:33 -0700 |
commit | 1da4ba883721e1613c4a0685e0fdd177a55eb07e (patch) | |
tree | 6691661392130cb97c44401853f40b2cab2ab8ae /extras/peer_add_secret_pub.in | |
parent | bbde491b47f240836d454585d10c5623aef45662 (diff) |
posix: Fix fgetxattr() crash when key name is NULL
This is used to get the size of xattr list (to later allocate buffer):
glfs_flistxattr(glfd, NULL, 0);
glfs_flistxattr() internally has the following call:
syncop_fgetxattr (subvol, fd, &xattr, NULL, NULL, NULL);
strncmp() segfaults as name is NULL in posix_fgetxattr()
Turns out this was a coverity fix in master branch that was not
backported to 3.7.x tree.
http://review.gluster.org/#/c/10252/
BUG: 1235904
Change-Id: I2ec4715f1ea2f0e9c5314b2dc358bc01ad7b7d45
Signed-off-by: Prashanth Pai <ppai@redhat.com>
Reviewed-on: http://review.gluster.org/11213
Reviewed-by: Raghavendra Talur <rtalur@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Raghavendra Bhat <raghavendra@redhat.com>
Diffstat (limited to 'extras/peer_add_secret_pub.in')
0 files changed, 0 insertions, 0 deletions