search

XTLS / Xray-core

Xray, Penetrates Everything. Also the best v2ray-core, with XTLS support. Fully compatible configuration.
https://t.me/projectXray
Mozilla Public License 2.0
25.88k stars 3.98k forks source link

Fix: Apply mutex when visiting `Attributes` as well #3921

Closed Fangliding closed 1 month ago

Fangliding commented 1 month ago

之前就看到了setattr那个函数（毕竟调的最多） 其他函数也可能遇到类似情况 补上了

RPRX commented 1 month ago

这个好像和 mux 那个不是同一个问题？

Fangliding commented 1 month ago

这个好像和 mux 那个不是同一个问题？

一个问题 SetAttribute() 和 Attribute() 都会尝试访问那个map 如果同时访问race了就boom 之前只给 SetAttribute() 加锁了

RPRX commented 1 month ago

这个不是 sniff 引发的吗

RPRX commented 1 month ago

就是说为什么 sniff 也有 race

Fangliding commented 1 month ago

这个不是 sniff 引发的吗

就是 #3908 的问题

RPRX commented 1 month ago

所以还是 mux 导致的？

Fangliding commented 1 month ago

问题还是老问题 对于一条被mux的连接 多个子请求在内部共享一条ctx

RPRX commented 1 month ago

关键代码在哪里，我研究下能不能治本

Fangliding commented 1 month ago

关键代码在哪里，我研究下能不能治本

见 #3718 吧 我也是从那来的（）

RPRX commented 1 month ago

月底了，同样决定先合了治个标