diff options
| author | Robert Newson <robert.newson@uk.ibm.com> | 2020-08-13 14:23:24 +0100 |
|---|---|---|
| committer | Robert Newson <robert.newson@uk.ibm.com> | 2020-08-14 16:47:59 +0100 |
| commit | 5004f997a6a30831e65c864b96ca82c926f3dbac (patch) | |
| tree | 7517a4f4d11a9ca0db81a1bc84b2be23da3cdf60 | |
| parent | 7c9094cfdd9d5069b903b27105bb156c621a624b (diff) | |
| download | couchdb-couch_index_server_crash2.tar.gz | |
Don't crash couch_index_server if the db isn't known yetcouch_index_server_crash2
If a ddoc is added immediately after database creation (_users and
_replicator when couchdb is used in a multi-tenant fashion), we can
crash couch_index_server in handle_db_event, as mem3_shards:local
throws an error.
| -rw-r--r-- | src/couch_index/src/couch_index_server.erl | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/couch_index/src/couch_index_server.erl b/src/couch_index/src/couch_index_server.erl index 67f0f8c6f..6bebff2d8 100644 --- a/src/couch_index/src/couch_index_server.erl +++ b/src/couch_index/src/couch_index_server.erl @@ -284,7 +284,11 @@ handle_db_event(<<"shards/", _/binary>> = DbName, {ddoc_updated, DDocResult = couch_util:with_db(DbName, fun(Db) -> couch_db:open_doc(Db, DDocId, [ejson_body, ?ADMIN_CTX]) end), - DbShards = [mem3:name(Sh) || Sh <- mem3:local_shards(mem3:dbname(DbName))], + LocalShards = try mem3:local_shards(mem3:dbname(DbName)) + catch error:database_does_not_exist -> + [] + end, + DbShards = [mem3:name(Sh) || Sh <- LocalShards], lists:foreach(fun(DbShard) -> lists:foreach(fun({_DbShard, {_DDocId, Sig}}) -> % check if there are other ddocs with the same Sig for the same db |