From: Miss Skeleton (bot) <31488909+miss-islington@users.noreply.github.com>
Date: Wed, 23 Oct 2019 12:07:23 +0000 (-0700)
Subject: bpo-38555: Fix an undefined behavior. (GH-16883)
X-Git-Tag: v3.8.1rc1~139
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=c5d3ea89ee5244714f221dcfcd3be96de2f1da8d;p=thirdparty%2FPython%2Fcpython.git

bpo-38555: Fix an undefined behavior. (GH-16883)

(cherry picked from commit 2e3d873d3bd0ef4708c4fa06b6cd6972574cb9af)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
---

diff --git a/Objects/dictobject.c b/Objects/dictobject.c
index a0208be9246d..76f4fefe0979 100644
--- a/Objects/dictobject.c
+++ b/Objects/dictobject.c
@@ -3840,22 +3840,21 @@ dictreviter_iternext(dictiterobject *di)
     PyDictKeysObject *k = d->ma_keys;
     PyObject *key, *value, *result;
 
+    if (i < 0) {
+        goto fail;
+    }
     if (d->ma_values) {
-        if (i < 0) {
-            goto fail;
-        }
         key = DK_ENTRIES(k)[i].me_key;
         value = d->ma_values[i];
         assert (value != NULL);
     }
     else {
         PyDictKeyEntry *entry_ptr = &DK_ENTRIES(k)[i];
-        while (i >= 0 && entry_ptr->me_value == NULL) {
+        while (entry_ptr->me_value == NULL) {
+            if (--i < 0) {
+                goto fail;
+            }
             entry_ptr--;
-            i--;
-        }
-        if (i < 0) {
-            goto fail;
         }
         key = entry_ptr->me_key;
         value = entry_ptr->me_value;