fixes from Vadim

request->proxy_listener is NULL after removing it from the proxy
hash, so cache the reverse_coa status.

diff --git a/src/main/process.c b/src/main/process.c
index 0cd6cdb919c3041443fc59c737dc2b46343a14a2..63c924ead5536540d8e2601205d551ceeea6cf32 100644 (file)
--- a/src/main/process.c
+++ b/src/main/process.c
@@ -2439,6 +2439,9 @@ static int process_proxy_reply(REQUEST *request, RADIUS_PACKET *reply)
        int post_proxy_type = 0;
        VALUE_PAIR *vp;
        char const *old_server;
+#ifdef WITH_COA_TUNNEL
+       bool reverse_coa = false;
+#endif
 
        VERIFY_REQUEST(request);
 
@@ -2526,6 +2529,14 @@ static int process_proxy_reply(REQUEST *request, RADIUS_PACKET *reply)
                        rad_assert(!request->in_proxy_hash);
                }
        } else if (request->in_proxy_hash) {
+#ifdef WITH_COA_TUNNEL
+               /*
+                *      Cache this, as request->proxy_listener will be
+                *      NULL after removing the request from the proxy
+                *      hash.
+                */
+               reverse_coa = request->proxy_listener->type != RAD_LISTEN_PROXY;
+#endif
                remove_from_proxy_hash(request);
        }
 
@@ -2540,7 +2551,7 @@ static int process_proxy_reply(REQUEST *request, RADIUS_PACKET *reply)
                request->server = request->home_server->virtual_server;
 
 #ifdef WITH_COA_TUNNEL
-       } else if (request->proxy_listener && (request->proxy_listener->type != RAD_LISTEN_PROXY)) {
+       } else if (reverse_coa) {
                rad_assert((request->proxy->code == PW_CODE_COA_REQUEST) ||
                           (request->proxy->code == PW_CODE_DISCONNECT_REQUEST));
                rad_assert(request->home_server != NULL);