diff options
| author | Raghavendra G <raghavendra@gluster.com> | 2010-10-13 01:38:20 +0000 |
|---|---|---|
| committer | Anand V. Avati <avati@dev.gluster.com> | 2010-10-26 03:01:35 -0700 |
| commit | 681bfd7ff32202314a5cdf8a8e13478c6c45acdb (patch) | |
| tree | c2d8b2fdcb85841054d9ee49a5fbbcaa53984cf5 /xlators/protocol/server/src/server.c | |
| parent | a8212606e23312fabd5c2068ade05efacae9c086 (diff) | |
server: Don't free conf in fini.
- This will fix race condition between handling DISCONNECT event and doing
fini. Though conf need to be freed, this will be done in future where
finis of all translators are properly implemented to make sure no
corruptions.
Signed-off-by: Raghavendra G <raghavendra@gluster.com>
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
BUG: 1758 ()
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1758
Diffstat (limited to 'xlators/protocol/server/src/server.c')
| -rw-r--r-- | xlators/protocol/server/src/server.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xlators/protocol/server/src/server.c b/xlators/protocol/server/src/server.c index 1d190c6ba..85abfb54d 100644 --- a/xlators/protocol/server/src/server.c +++ b/xlators/protocol/server/src/server.c @@ -771,6 +771,7 @@ out: void fini (xlator_t *this) { +#if 0 server_conf_t *conf = NULL; conf = this->private; @@ -794,7 +795,7 @@ fini (xlator_t *this) } this->private = NULL; - +#endif return; } |