Browse Source

Add another assertion to check for 24086 root causes

In cdm_diff_ht_set_status(), we shouldn't have been allowing the
status CDM_DIFF_PRESENT to be set if there wasn't actually a handle.
Nick Mathewson 6 years ago
parent
commit
68c21860e3
1 changed files with 4 additions and 0 deletions
  1. 4 0
      src/or/consdiffmgr.c

+ 4 - 0
src/or/consdiffmgr.c

@@ -283,6 +283,10 @@ cdm_diff_ht_set_status(consensus_flavor_t flav,
                        int status,
                        consensus_cache_entry_handle_t *handle)
 {
+  if (handle == NULL) {
+    tor_assert_nonfatal(status != CDM_DIFF_PRESENT);
+  }
+
   struct cdm_diff_t search, *ent;
   memset(&search, 0, sizeof(cdm_diff_t));
   search.flavor = flav;