From: Victor Stinner <victor.stinner@gmail.com>
Date: Thu, 14 Sep 2017 21:48:37 +0000 (-0700)
Subject: bpo-30860: Fix deadcode in obmalloc.c (#3499)
X-Git-Tag: v3.7.0a1~31
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ccb3c7654cc3d031fb93bf443a6ef9cfb11f6b43;p=python

bpo-30860: Fix deadcode in obmalloc.c (#3499)

Fix Coverity CID 1417587: _PyMem_Initialize() contains code which is
never executed.

Replace the runtime check with a build assertion.
---

diff --git a/Objects/obmalloc.c b/Objects/obmalloc.c
index 57edf97e20..f2651d7574 100644
--- a/Objects/obmalloc.c
+++ b/Objects/obmalloc.c
@@ -297,9 +297,9 @@ _PyMem_Initialize(struct _pymem_runtime_state *state)
     state->allocators.obj = _pyobject;
 
 #ifdef WITH_PYMALLOC
+    Py_BUILD_ASSERT(NB_SMALL_SIZE_CLASSES == 64);
+
     for (int i = 0; i < 8; i++) {
-        if (NB_SMALL_SIZE_CLASSES <= i * 8)
-            break;
         for (int j = 0; j < 8; j++) {
             int x = i * 8 + j;
             poolp *addr = &(state->usedpools[2*(x)]);