Skip to content

projects: add uint256#6497

Merged
jonathanmetzman merged 4 commits into
google:masterfrom
holiman:uint256
Aug 21, 2022
Merged

projects: add uint256#6497
jonathanmetzman merged 4 commits into
google:masterfrom
holiman:uint256

Conversation

@holiman

@holiman holiman commented Sep 21, 2021

Copy link
Copy Markdown
Contributor

This PR adds https://github.com/holiman/uint256 to oss-fuzz.

uint256 is a 256-bit math library in golang, and is used primarily within the Ethereum ecosystem, by projects such as go-ethereum and erigon

@holiman

holiman commented Sep 21, 2021

Copy link
Copy Markdown
Contributor Author

cc @chfast is that the non-aliased version of your email address?

@chfast

chfast commented Sep 21, 2021

Copy link
Copy Markdown

cc @chfast is that the non-aliased version of your email address?

Yes.

@holiman

holiman commented Oct 27, 2021

Copy link
Copy Markdown
Contributor Author

Fixed the issues, tested the integration locally, fuzzer worked fine. PTAL

@holiman

holiman commented Nov 29, 2021

Copy link
Copy Markdown
Contributor Author

A gentle ping on this?

@holiman

holiman commented Feb 22, 2022

Copy link
Copy Markdown
Contributor Author

Friendly ping on this. The uint256 library is used in several blockchain projects based on go-ethereum. Is there anything in particular preventing this from being approved?

@jonathanmetzman

Copy link
Copy Markdown
Contributor

Are you still interested in this integration? If yes I'll ask the panel to consider. Otherwise maybe you should consider clusterfuzzlite

@holiman

holiman commented Aug 18, 2022

Copy link
Copy Markdown
Contributor Author

Are you still interested in this integration?

Yes, I very much am

@jonathanmetzman

Copy link
Copy Markdown
Contributor

Are you still interested in this integration?

Yes, I very much am

Are there other users? I guess we sort of consider ethereum critical but if there are non-blockchain users we are more likely to accept.

@holiman

holiman commented Aug 19, 2022

Copy link
Copy Markdown
Contributor Author

Not that I am aware of, no

@jonathanmetzman jonathanmetzman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jonathanmetzman jonathanmetzman merged commit 7ce3c5f into google:master Aug 21, 2022
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.

3 participants