diff options
| author | Simon MacMullen <simon@rabbitmq.com> | 2013-05-30 16:37:14 +0100 |
|---|---|---|
| committer | Simon MacMullen <simon@rabbitmq.com> | 2013-05-30 16:37:14 +0100 |
| commit | 15702b503242a2750e6da46b5cdee7593fb555b9 (patch) | |
| tree | 34faae1b5ed59fe8c7ee5a04f0105442b6387190 | |
| parent | 4baf7769574fe1afb76648d9351f785595191787 (diff) | |
| download | rabbitmq-server-bug25573.tar.gz | |
Don't use andalso here, it breaks LDAP when compiled under R12.bug25573
| -rw-r--r-- | src/rabbit_access_control.erl | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/rabbit_access_control.erl b/src/rabbit_access_control.erl index 16387268..cbe89a17 100644 --- a/src/rabbit_access_control.erl +++ b/src/rabbit_access_control.erl @@ -68,8 +68,11 @@ check_vhost_access(User = #user{ username = Username, auth_backend = Module }, VHostPath) -> check_access( fun() -> - rabbit_vhost:exists(VHostPath) andalso - Module:check_vhost_access(User, VHostPath) + %% TODO this could be an andalso shortcut under >R13A + case rabbit_vhost:exists(VHostPath) of + false -> false; + true -> Module:check_vhost_access(User, VHostPath) + end end, "~s failed checking vhost access to ~s for ~s: ~p~n", [Module, VHostPath, Username], |