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 | |
| 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
| -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 1d190c6baf7..85abfb54d2a 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; } |