unetmsg: add null check in TX disconnect callback

The network may be deleted before the disconnect callback fires.
Check for null to avoid crash when accessing net.tx_channels.

Signed-off-by: Felix Fietkau <nbd@nbd.name>
(cherry picked from commit f631d1576ddb331ceee726d876ea9c74a3553dbc)

diff --git a/package/network/services/unetmsg/files/usr/share/ucode/unetmsg/unetmsgd-remote.uc b/package/network/services/unetmsg/files/usr/share/ucode/unetmsg/unetmsgd-remote.uc
index 59d9b0c278169ffabd3806411e5d4aa4c07a18fd..63be3cbab04f513b1032e03b5dab66674a8da48f 100644 (file)
--- a/package/network/services/unetmsg/files/usr/share/ucode/unetmsg/unetmsgd-remote.uc
+++ b/package/network/services/unetmsg/files/usr/share/ucode/unetmsg/unetmsgd-remote.uc
@@ -350,6 +350,11 @@ function network_open_channel(net, name, peer)
 
        let disconnect_cb = (req) => {
                let net = networks[sock_data.network];
+               if (!net) {
+                       network_tx_socket_close(sock_data);
+                       return;
+               }
+
                let cur_data = net.tx_channels[sock_data.name];
                if (cur_data == sock_data)
                        delete net.tx_channels[sock_data.name];