From: Christian Heimes <christian@cheimes.de>
Date: Thu, 14 Nov 2013 00:47:14 +0000 (+0100)
Subject: Issue #17828: _PyObject_GetDictPtr() may return NULL instead of a PyObject**
X-Git-Tag: v3.4.0b1~286
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=6a3db25c70965790893e51febe139215b22b40c5;p=thirdparty%2FPython%2Fcpython.git

Issue #17828: _PyObject_GetDictPtr() may return NULL instead of a PyObject**
CID 1128792: Dereference null return value (NULL_RETURNS)
---

diff --git a/Objects/exceptions.c b/Objects/exceptions.c
index 2f0d5b6400e6..94f581b7b0ee 100644
--- a/Objects/exceptions.c
+++ b/Objects/exceptions.c
@@ -2626,7 +2626,7 @@ _PyErr_TrySetFromCause(const char *format, ...)
     PyObject* msg_prefix;
     PyObject *exc, *val, *tb;
     PyTypeObject *caught_type;
-    PyObject *instance_dict;
+    PyObject **dictptr;
     PyObject *instance_args;
     Py_ssize_t num_args;
     PyObject *new_exc, *new_val, *new_tb;
@@ -2664,8 +2664,10 @@ _PyErr_TrySetFromCause(const char *format, ...)
     }
 
     /* Ensure the instance dict is also empty */
-    instance_dict = *_PyObject_GetDictPtr(val);
-    if (instance_dict != NULL && PyObject_Length(instance_dict) > 0) {
+    dictptr = _PyObject_GetDictPtr(val);
+    if ((dictptr != NULL) && (*dictptr != NULL) &&
+        (PyObject_Length(*dictptr) > 0)
+       ) {
         /* While we could potentially copy a non-empty instance dictionary
          * to the replacement exception, for now we take the more
          * conservative path of leaving exceptions with attributes set