fix(ua2f.c): possible deadlock in the queue, causing network disconnec…

[miny1233]

…tion

在不支持内核上运行会造成队列死锁。当返回Operation not supported错误的时候，如果继续执行，就会死锁。

[miny1233]

还有就是我觉得，像这种错误之类的，最好能直接在控制台上输出，最开始运行搞的我都没搞懂哪弄错了，还以为是iptables的问题

[miny1233]

补充一点就是，这个死锁问题，就算加--by-pass也是没用的。

[Zxilly]

我仔细看了一下，感觉碰到 IO_ERROR 直接退出好了。设置一下 should_exit。已经收到没法解析的东西了，再往下执行也不合适。

[miny1233]

我仔细看了一下，感觉碰到 IO_ERROR 直接退出好了。设置一下 should_exit。已经收到没法解析的东西了，再往下执行也不合适。

我最早是这样想的，但是就是错误类型很多，我也没仔细看，就怕存在那种可以忽略的错误。起码目前能确定这个错误是必须要程序退出的。

[Zxilly]

你现在这样写的话，IO_NOTREADY 的时候也直接退出了，我直接改吧。

[miny1233]

你现在这样写的话，IO_NOTREADY 的时候也直接退出了，我直接改吧。

没有的，只会在ERROR的时候退出

[Zxilly]

while (status = nfqueue_next(buf, packet)) {

返回一个 IO_NOTREADY 为 0，这 while 直接退出去了

[miny1233]

while (status = nfqueue_next(buf, packet)) {

返回一个 IO_NOTREADY 为 0，这 while 直接退出去了

while (status = nfqueue_next(buf, packet)) {

返回一个 IO_NOTREADY 为 0，这 while 直接退出去了

原本就是收到IO_NOTREADY的时候，跳出这个循环去做nfqueue_receive(queue, buf, 0) ，不过你这用switch确实比原来好看很多。

[Zxilly]

nfqueue_receive 只需要成功一次的，拿到 queue 的 handler 就行了。是 packet 因为加了 async 的 flag，所以读的时候可能 NOTREADY

[miny1233]

你确定吗？你的这个修改和之前的行为都不一样了
https://github.com/Zxilly/UA2F/blob/448c047601bb150d3ffced6e63efd5fb8ca1ca46/src/ua2f.c

 while (!should_exit) {
        if (nfqueue_receive(queue, buf, 0) == IO_READY) {
            struct nf_packet packet[1];
            while (nfqueue_next(buf, packet) == IO_READY) {
                handle_packet(queue, packet);
            }
        }
    }

原来在nfqueue_next收到非IO_READY的情况下会执行一遍nfqueue_receive，但你修改的最新版本只会执行一次nfqueue_receive后就一直循环执行nfqueue_next

[Zxilly]

确实，太久没写我把queue和packet弄混了，我再改改。
应该写个集成测试的。