-
-
Notifications
You must be signed in to change notification settings - Fork 172
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Dependency] update bzip2 1.0.6 -> 1.0.8 #1066
base: master
Are you sure you want to change the base?
Conversation
357f7e6
to
ed83881
Compare
ed83881
to
090d0dc
Compare
@s1lentq hello. We are tested this version in live server with you. Could you please review this pr? |
this update has caused issues with teams and chat command hooks on my cs servers and caused a server hang. rolled back to the first zip bomb exploit fix. |
this commit just updates 10yo dependency (2014year of release) to more fresh version (released in 2019). there is no any exploit fixes. just bzip lib update. are you shure, that previous version from master CI will not cause your problems too as this test build? |
works just fine.
even caused a server hang after a couple of hours. had over 1 month server uptime before so no other mods caused it. when i rolled back everything went back to normal with no issues. |
@di57inct this updated part used by engine only if you download resources without fastdl. how it crash your server, if there is no engine's sources was modified? there is no breaking changes at all. by the way, this pr still in testing. Please, provide more info: ➡️ What version of engine are you seeing the problem on? ➡️ Please, paste output from console command below: ➡️ (Optional) You could also provide info about AMX Mox X (version < 1.9.0 are not supported!): ➡️ Your environment: OS ( 🪟 Windows or 🐧 Linux), OS Vetsion, etc. |
EDIT: I do seem to have a missing resource if that helps with anything; "Error: server failed to transmit file 'sound/radio/bot/im_blind.wav'" Server doesn't crash, it hangs(100% cpu usage and no response). Server engine version is latest steam_legacy(pre-25th). Not running ReGameDLL.
|
having a missing resource needed for work is not very good. I think it should be assumed that this is the reason for the hang, not the library update. in any case, a pull request is not an issue. if you have problems in the future, you should create an issue. let's not flood here without evidence. I have been running three servers for several days and they work fine with this build. testing continues. regards |
never had a server hang because of a missing sound file that isn't used with emit_sound. that should get looked at because it isn't normal. |
i agree. i will test too. |
confirming over 5 days of uptime when no resource is missing. |
@di57inct hello! have any updates? |
Working fine when no resources missing. No crashes or any other issues as far as i know over 2 weeks uptime. |
good news, thank you for feedback! 🤝 |
Just updated bzip2 from
1.0.6
to1.0.8
.No exploit fixes, just library update.
for historical purposes
1.0.8.linux32.zip
1.0.8.win32.zip