Skip to content
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

Remove ACE plugin due to multiple CVEs affecting decade old ACEv2.dll #90

Closed
axnsan12 opened this issue Feb 26, 2019 · 4 comments
Closed
Assignees

Comments

@axnsan12
Copy link

Sorry if this was already reported/fixed in any of the russian language discussions.

Basically a serious RCE was discoverd in the ACE archive unpacking code, which can be triggered by simply opening an archive of this format in far.

https://research.checkpoint.com/extracting-code-execution-from-winrar/

https://www.bleepingcomputer.com/news/security/malspam-exploits-winrar-ace-vulnerability-to-install-a-backdoor/

@shmuz
Copy link
Contributor

shmuz commented Feb 26, 2019

by simply opening an archive of this format in far.

Far.exe itself can not open archives, that is done by plugins. If the standard Arclite plugin out of the box handles ACE format then yes, it is a problem. Otherwise it is not Far Manager's concern.

@alabuzhev
Copy link
Contributor

due to multiple CVEs affecting decade old ACEv2.dll

Do you see ACEv2.dll anywhere in the distribution package?

@axnsan12
Copy link
Author

I'm sorry, I saw the ACE dll in the source repo (newarc plugin) and mistakenly assumed it's included in the default distribution.

https://github.com/FarGroup/FarManager/tree/806c80dff3e182c1c043fad9078490a9bf962456/plugins/newarc/bin/plugins

@alabuzhev
Copy link
Contributor

Newarc has never been released (and hardly will ever be), so it's nothing to worry about. However, we should probably remove the dll for good just in case. Thanks for reporting.

alabuzhev added a commit that referenced this issue Feb 26, 2019
@alabuzhev alabuzhev self-assigned this Jun 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants