diff options
| author | Amar Tumballi <amar@gluster.com> | 2009-11-25 15:13:37 +0000 |
|---|---|---|
| committer | Anand V. Avati <avati@dev.gluster.com> | 2009-11-26 02:49:07 -0800 |
| commit | 3429e8755756191d010896083ca9a7141682ead9 (patch) | |
| tree | a529b7348b8bde711a2564746d43249ec6353529 | |
| parent | 28e2901806d35055ca58470bf26d72edb803ba01 (diff) | |
add an extra check for loc->parent when loc->ino is not 1
This extra check is needed only in case where setup is
"server -> chain -> client", where for the root inode, inode->ino
is not updated in the first lookup call, hence used to crash.
Signed-off-by: Amar Tumballi <amar@gluster.com>
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
BUG: 377 (segfault in protocol/client with chaining)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=377
| -rw-r--r-- | xlators/protocol/client/src/client-protocol.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/protocol/client/src/client-protocol.c b/xlators/protocol/client/src/client-protocol.c index 2723126ca95..d0f33e730f8 100644 --- a/xlators/protocol/client/src/client-protocol.c +++ b/xlators/protocol/client/src/client-protocol.c @@ -3268,7 +3268,7 @@ client_lookup (call_frame_t *frame, xlator_t *this, loc_t *loc, GF_VALIDATE_OR_GOTO (this->name, loc, unwind); GF_VALIDATE_OR_GOTO (this->name, loc->path, unwind); - if (loc->ino != 1) { + if (loc->ino != 1 && loc->parent) { ret = inode_ctx_get2 (loc->parent, this, &par, &gen); if (loc->parent->ino && ret < 0) { gf_log (this->name, GF_LOG_TRACE, |