Fred Drake avatar Fred Drake committed 6bbe268

There was a real leak in the "export a C API" example; fix that one.
(There are too many initspam() functions; they need to be renamed
post-beta.)

Comments (0)

Files changed (1)

 void
 initspam()
 {
-    PyObject *m, *d;
+    PyObject *m;
     static void *PySpam_API[PySpam_API_pointers];
     PyObject *c_api_object;
+
     m = Py_InitModule("spam", SpamMethods);
 
     /* Initialize the C API pointer array */
     /* Create a CObject containing the API pointer array's address */
     c_api_object = PyCObject_FromVoidPtr((void *)PySpam_API, NULL);
 
-    /* Create a name for this object in the module's namespace */
-    d = PyModule_GetDict(m);
-    PyDict_SetItemString(d, "_C_API", c_api_object);
+    if (c_api_object != NULL) {
+        /* Create a name for this object in the module's namespace */
+        PyObject *d = PyModule_GetDict(m);
+
+        PyDict_SetItemString(d, "_C_API", c_api_object);
+        Py_DECREF(c_api_object);
+    }
 }
 \end{verbatim}
 
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.