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

Add patched version for CVE-2013-1656 #702

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

v0lck3r
Copy link

@v0lck3r v0lck3r commented Aug 29, 2023

Based on the following commit, which was found here, the patched version is 2.0.0.rc1.
Aslo versions after 1.3.2 and before 2.0.0.rc1 (aka 1.3.3, 1.3.4 and 1.3.5) do not contain updates for the files concerned (See for example payment_methods_controller.rb where the last update dates back to 2012, i.e. before the vulnerability was patched). Thus, we can say that the patched version is 2.0.0.rc1 with a certain confidence.

Copy link
Member

@postmodern postmodern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the commit URL and any other URLs to the related: URLs list. That would be really useful to others!

@@ -22,3 +22,4 @@ patched_versions:
related:
url:
- https://spreecommerce.com/blog/multiple-security-vulnerabilities-fixed
- https://github.com/spree/spree/commit/70092eb55b8be8fe5d21a7658b62da658612fba7
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra newline.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- https://github.com/spree/spree/commit/70092eb55b8be8fe5d21a7658b62da658612fba7
- https://github.com/spree/spree/commit/70092eb55b8be8fe5d21a7658b62da658612fba7

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think your editor is automatically adding the newline.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@v0lck3r GitHub isn't allowing me to use the Suggested change. GitHub is claiming there is no difference between them, even though it's supposed to remove the last newline.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@postmodern yeah I have no idea why is that !

Copy link
Author

@v0lck3r v0lck3r Aug 30, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@postmodern tbh I see no extra line. the file is still 25 lines total after adding the new reference, compared to the 24 lines in the original file .
Can you double check please !

@postmodern postmodern assigned postmodern and v0lck3r and unassigned postmodern Aug 29, 2023
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

Successfully merging this pull request may close these issues.

None yet

2 participants