diff options
author | Pranith Kumar K <pranithk@gluster.com> | 2012-07-25 15:08:46 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2012-07-25 08:16:27 -0700 |
commit | 8180d474a9f4aad495168102399b0465ad8efb06 (patch) | |
tree | 10fd223134847c6dc3c06a9938b5932fad0c0a5d /COPYING-LGPLV3 | |
parent | 9d3dff4cf2b4f29f514b9eae00330dfe3e144fff (diff) |
features/index: Fix race in this->private initialization
RCA:
In index_worker function at the time of assigning priv from
this->private, this->private may not be set in init() function
of index xlator.
Fix:
Set this->private before creating the thread.
Additional Changes:
Added code to handle error path completely.
Test cases:
Attached the process to gdb and simulated failures.
executed fini in gdb using call fini(this) after init
was successful.
Change-Id: I1874a30d009a35352173b827574cf83daf431453
BUG: 843071
Signed-off-by: Pranith Kumar K <pranithk@gluster.com>
Reviewed-on: http://review.gluster.com/3728
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Jeff Darcy <jdarcy@redhat.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'COPYING-LGPLV3')
0 files changed, 0 insertions, 0 deletions