diff options
| author | Sage Weil <sage@inktank.com> | 2013-06-15 08:48:37 -0700 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2013-06-17 10:42:29 -0700 |
| commit | 2ad85576850bcb61c3ac8fee38609f9cd51bfac2 (patch) | |
| tree | 165264315a9c4a30dabfd37c54e83b04c8be0940 | |
| parent | d3d88983bceaf508f2585cc6bf8fc304a0943ffe (diff) | |
| download | ceph-2ad85576850bcb61c3ac8fee38609f9cd51bfac2.tar.gz | |
client: fix ancient typo in caps revocation path
If we have dropped all references to a revoked capability, send the ack
to the MDS. This typo has been there since v0.7 (early 2009)!
Backport: cuttlefish
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Greg Farnum <greg@inktank.com>
(cherry picked from commit b7143c2f84daafbe2c27d5b2a2d5dc40c3a68d15)
| -rw-r--r-- | src/client/Client.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/client/Client.cc b/src/client/Client.cc index 0f064636bdb..7f17a3de2a5 100644 --- a/src/client/Client.cc +++ b/src/client/Client.cc @@ -2411,7 +2411,7 @@ void Client::check_caps(Inode *in, bool is_delayed) } /* completed revocation? */ - if (revoking && (revoking && used) == 0) { + if (revoking && (revoking & used) == 0) { ldout(cct, 10) << "completed revocation of " << ccap_string(cap->implemented & ~cap->issued) << dendl; goto ack; } |