Lenard Lindstrom avatar Lenard Lindstrom committed 6a2f740

clean up r 2117 bug fix

Comments (0)

Files changed (1)

         test = fopen (filename, "rb");
         if(!test)
         {
+            PyObject *tmp = NULL;
+
             if (!strcmp (filename, font_defaultname)) {
-                Py_DECREF (fileobj);
-                fileobj = font_resource (font_defaultname);
+                tmp = font_resource (font_defaultname);
             }
-            else {
-                Py_DECREF (fileobj);
-                fileobj = NULL;
-            }
-            if (!fileobj)
+            if (!tmp)
             {
-                PyErr_SetString (PyExc_IOError, "unable to read font filename");
+                PyErr_SetString (PyExc_IOError,
+                                 "unable to read font filename");
                 goto error;
             }
+            Py_DECREF (fileobj);
+            fileobj = tmp;
         }
         else
         {
     return 0;
 
 error:
-    Py_XDECREF (fileobj);
+    Py_DECREF (fileobj);
     return -1;
 }
 
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.