Xray-core icon indicating copy to clipboard operation
Xray-core copied to clipboard

Apply mutex to more attr functions

Open Fangliding opened this issue 1 year ago • 9 comments

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

Fangliding avatar Oct 17 '24 05:10 Fangliding

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

RPRX avatar Oct 17 '24 05:10 RPRX

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

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

Fangliding avatar Oct 17 '24 05:10 Fangliding

这个不是 sniff 引发的吗

RPRX avatar Oct 17 '24 05:10 RPRX

就是说为什么 sniff 也有 race

RPRX avatar Oct 17 '24 05:10 RPRX

这个不是 sniff 引发的吗

就是 #3908 的问题

Fangliding avatar Oct 17 '24 05:10 Fangliding

所以还是 mux 导致的?

RPRX avatar Oct 17 '24 05:10 RPRX

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

Fangliding avatar Oct 17 '24 05:10 Fangliding

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

RPRX avatar Oct 17 '24 05:10 RPRX

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

见 #3718 吧 我也是从那来的()

Fangliding avatar Oct 17 '24 05:10 Fangliding

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

RPRX avatar Oct 30 '24 02:10 RPRX