From: Victor Stinner <victor.stinner@haypocalc.com>
Date: Sun, 11 Dec 2011 19:04:56 +0000 (+0100)
Subject: Fix error handling in resize_compact()
X-Git-Tag: v3.3.0a1~639
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=84def3774d2079ea2a812e0220507ff0e27247e7;p=python

Fix error handling in resize_compact()
---

diff --git a/Objects/unicodeobject.c b/Objects/unicodeobject.c
index 9670ae84fc..2846bd228d 100644
--- a/Objects/unicodeobject.c
+++ b/Objects/unicodeobject.c
@@ -630,6 +630,7 @@ resize_compact(PyObject *unicode, Py_ssize_t length)
     Py_ssize_t struct_size;
     Py_ssize_t new_size;
     int share_wstr;
+    PyObject *new_unicode;
 
     assert(PyUnicode_IS_READY(unicode));
     char_size = PyUnicode_KIND(unicode);
@@ -639,22 +640,25 @@ resize_compact(PyObject *unicode, Py_ssize_t length)
         struct_size = sizeof(PyCompactUnicodeObject);
     share_wstr = _PyUnicode_SHARE_WSTR(unicode);
 
-    _Py_DEC_REFTOTAL;
-    _Py_ForgetReference(unicode);
-
     if (length > ((PY_SSIZE_T_MAX - struct_size) / char_size - 1)) {
+        Py_DECREF(unicode);
         PyErr_NoMemory();
         return NULL;
     }
     new_size = (struct_size + (length + 1) * char_size);
 
-    unicode = (PyObject *)PyObject_REALLOC((char *)unicode, new_size);
-    if (unicode == NULL) {
+    _Py_DEC_REFTOTAL;
+    _Py_ForgetReference(unicode);
+
+    new_unicode = (PyObject *)PyObject_REALLOC((char *)unicode, new_size);
+    if (new_unicode == NULL) {
         PyObject_Del(unicode);
         PyErr_NoMemory();
         return NULL;
     }
+    unicode = new_unicode;
     _Py_NewReference(unicode);
+
     _PyUnicode_LENGTH(unicode) = length;
     if (share_wstr) {
         _PyUnicode_WSTR(unicode) = PyUnicode_DATA(unicode);