sing-shadowsocks icon indicating copy to clipboard operation
sing-shadowsocks copied to clipboard

Fix multi-user service not working with packet

Open wwqgtxx opened this issue 2 years ago • 6 comments

fix https://github.com/SagerNet/sing-shadowsocks/issues/5

wwqgtxx avatar Aug 16 '23 00:08 wwqgtxx

decrypted或许在continue之前切一下片重置一下len会更好?不重置len的话或许会重新分配底层数组导致潜在的性能损失 Screenshot_20230816_084922

我只看了gcm的实现,不清楚其他的是不是也有类似逻辑

Yuzuki616 avatar Aug 16 '23 00:08 Yuzuki616

@Yuzuki616 他这里校验的是cap,不会导致重分配底层数组的

wwqgtxx avatar Aug 16 '23 00:08 wwqgtxx

但是total是根据dst的len计算的吧

Yuzuki616 avatar Aug 16 '23 00:08 Yuzuki616

但是total是根据dst的len计算的吧

dst传入的始终是len为0的原slice

wwqgtxx avatar Aug 16 '23 00:08 wwqgtxx

ok,是我的理解出了问题,因为我昨天在修复的时候让dst和返回值共用了同一个变量,不好意思,感谢您的修复。

Yuzuki616 avatar Aug 16 '23 01:08 Yuzuki616

Any progress?

dyhkwong avatar Jan 08 '24 09:01 dyhkwong