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

Potential security issue #7

Closed
psmoros opened this issue Jun 21, 2023 · 8 comments
Closed

Potential security issue #7

psmoros opened this issue Jun 21, 2023 · 8 comments

Comments

@psmoros
Copy link

psmoros commented Jun 21, 2023

Hello 👋

I run a security community that finds and fixes vulnerabilities in OSS. A researcher (@haile01) has found a potential issue, which I would be eager to share with you.

Could you add a SECURITY.md file with an e-mail address for me to send further details to? GitHub recommends a security policy to ensure issues are responsibly disclosed, and it would help direct researchers in the future.

Looking forward to hearing from you 👍

(cc @huntr-helper)

@jmcnamara
Copy link
Owner

jmcnamara commented Dec 28, 2023

See also runrig#33

@jmcnamara jmcnamara reopened this Dec 29, 2023
@MichaelDaum
Copy link

Thanks for fixing CVE-2023-7101. Yet memory bombing as reported in https://github.com/haile01/perl_spreadsheet_excel_rce_poc is still possible.

@jmcnamara
Copy link
Owner

@MichaelDaum The memory bomb part of the POC is against Spreadsheet::ParseXLSX which uses Spreadsheet::ParseExcel but isn't part of it. That isn't within my control.

That part of the issue seems to be raised here: doy/spreadsheet-parsexlsx#103

@jmcnamara
Copy link
Owner

Fixed in version 0.66 which is now on CPAN.

@haile01
Copy link

haile01 commented Dec 29, 2023 via email

@jmcnamara
Copy link
Owner

Thanks but to give credit where it is due the fix was from @ruoso, I just tested and packaged it.

@MichaelDaum
Copy link

@MichaelDaum The memory bomb part of the POC is against Spreadsheet::ParseXLSX which uses Spreadsheet::ParseExcel but isn't part of it. That isn't within my control.

Oic. Sorry.

Gave it a spin in doy/spreadsheet-parsexlsx#104

@timlegge
Copy link

timlegge commented Jan 8, 2024

@psmoros as a member of the cpan security working group I am attempting to do a lesson's learned for the issue you referenced (that eventually became CVE-2023-7101). Would you mind contacting me at "timlegge at cpan dot org" to answer a few questions?

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

5 participants