diff options
author | Bhumika Goyal <bgoyal@redhat.com> | 2018-08-07 15:28:13 +0530 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2018-08-21 05:01:08 +0000 |
commit | 6fa531c57c633827283fba6a3cdaa1c68976ccb7 (patch) | |
tree | 200f300af453ad45e7e7491f989757170d01adb8 /xlators/mgmt/glusterd/src/glusterd-snapshot.c | |
parent | 2348b8ae6ce02f83069d01478a31bb2498cc6183 (diff) | |
download | glusterfs-6fa531c57c633827283fba6a3cdaa1c68976ccb7.tar.gz glusterfs-6fa531c57c633827283fba6a3cdaa1c68976ccb7.tar.xz glusterfs-6fa531c57c633827283fba6a3cdaa1c68976ccb7.zip |
xlators: protocol: Fix deferencing pointer after free coverity issues
The pointer of type struct iobuf * is getting dereferenced after
getting freed by iobuf_unref function. Therefore, move this function
after all the dereferences of this pointer type.
Also, it is useful coding standard to have iobuf_unref just after
iobref_add. So, move iobref_add too.
Occurences found using Coccinelle script:
@@
identifier rsphdr_iobuf;
expression E;
identifier func;
@@
*iobuf_unref(rsphdr_iobuf);
...
*E = func(rsphdr_iobuf);
Fixes CID: 1390517, 1390278, 1388666, 1356588, 1356587 at [1].
and also some more occurences which were found using the above script but not
caught by Coverity.
[1]. https://scan6.coverity.com/reports.htm#v42388/p10714/fileInstanceId=84384920&defectInstanceId=25600709&mergedDefectId=1388666
Change-Id: I579e9d12698f14e9e24bc926c6efef16bac5c06c
updates: bz#789278
Signed-off-by: Bhumika Goyal <bgoyal@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-snapshot.c')
0 files changed, 0 insertions, 0 deletions