Python: fixing incorrect function object dereference.

The __call__ method can be native and not be a PyFunction type.  A type check
is thus required before accessing op_code and other fields.

Reproduced on Ubuntu 21.04, Python 3.9.4 and Falcon framework: here, the
App.__call__ method is compiled with Cython, so accessing op_code->co_flags is
invalid; accidentally, the COROUTINE bit is set which forces the Python module
into the ASGI mode.

The workaround is explicit protocol specification.

Note: it is impossible to specify the legacy mode for ASGI.

1 files changed, 12 insertions, 6 deletions

diff --git a/src/python/nxt_python_asgi.c b/src/python/nxt_python_asgi.c
index 354e3a81..2d3efc7d 100644
--- a/src/python/nxt_python_asgi.c
+++ b/src/python/nxt_python_asgi.c
@@ -117,15 +117,20 @@ nxt_python_asgi_get_func(PyObject *obj)
     if (PyMethod_Check(call)) {
         obj = PyMethod_GET_FUNCTION(call);
 
-        Py_INCREF(obj);
-        Py_DECREF(call);
+        if (PyFunction_Check(obj)) {
+            Py_INCREF(obj);
 
-        return obj;
+        } else {
+            obj = NULL;
+        }
+
+    } else {
+        obj = NULL;
     }
 
     Py_DECREF(call);
 
-    return NULL;
+    return obj;
 }
 
 
@@ -161,8 +166,9 @@ nxt_python_asgi_init(nxt_unit_init_t *init, nxt_python_proto_t *proto)
     for (i = 0; i < nxt_py_targets->count; i++) {
         func = nxt_python_asgi_get_func(nxt_py_targets->target[i].application);
         if (nxt_slow_path(func == NULL)) {
-            nxt_unit_alert(NULL, "Python cannot find function for callable");
-            return NXT_UNIT_ERROR;
+            nxt_unit_debug(NULL, "asgi: cannot find function for callable, "
+                                 "unable to check for legacy mode (#%d)", i);
+            continue;
         }
 
         code = (PyCodeObject *) PyFunction_GET_CODE(func);