diff options
author | Sverker Eriksson <[email protected]> | 2017-09-04 20:53:00 +0200 |
---|---|---|
committer | Sverker Eriksson <[email protected]> | 2017-09-05 14:14:35 +0200 |
commit | 98bd5f66a03b945916baa2a91c4aba3c837676d1 (patch) | |
tree | aaa42e530b4fd15fd4f5073b4224935d4472dcd1 /erts/emulator/beam/erl_db.c | |
parent | 040bdce67f88d833bfb59adae130a4ffb4c180f0 (diff) | |
download | otp-98bd5f66a03b945916baa2a91c4aba3c837676d1.tar.gz otp-98bd5f66a03b945916baa2a91c4aba3c837676d1.tar.bz2 otp-98bd5f66a03b945916baa2a91c4aba3c837676d1.zip |
erts: Fix faulty ASSERT of table fixation counter
Cannot read fix->counter safely without table lock.
Diffstat (limited to 'erts/emulator/beam/erl_db.c')
-rw-r--r-- | erts/emulator/beam/erl_db.c | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/erts/emulator/beam/erl_db.c b/erts/emulator/beam/erl_db.c index 17e0f2aeec..e84171649c 100644 --- a/erts/emulator/beam/erl_db.c +++ b/erts/emulator/beam/erl_db.c @@ -3607,14 +3607,8 @@ static SWord proc_cleanup_fixed_table(Process* p, DbFixation* fix) ASSERT(sizeof(DbFixation) == ERTS_ALC_DBG_BLK_SZ(fix)); ERTS_DB_ALC_MEM_UPDATE_(tb, sizeof(DbFixation), 0); } - else { - ASSERT(fix->counter == 0); - } db_unlock(tb, LCK_WRITE_REC); } - else { - ASSERT(fix->counter == 0); - } erts_bin_release(fix->tabs.btid); erts_free(ERTS_ALC_T_DB_FIXATION, fix); @@ -3856,11 +3850,8 @@ static void free_fixations_op(DbFixation* fix, void* vctx) { struct free_fixations_ctx* ctx = (struct free_fixations_ctx*) vctx; erts_aint_t diff; -#ifdef DEBUG - DbTable* dbg_tb = btid2tab(fix->tabs.btid); -#endif - ASSERT(!dbg_tb || dbg_tb == ctx->tb); + ASSERT(!btid2tab(fix->tabs.btid)); ASSERT(fix->counter > 0); ASSERT(ctx->tb->common.status & DB_DELETE); |