search

AI-Protocol-Official / Gitcoin-Bounties

6 stars 7 forks source link

iNFT Protocol - ERC-20 Contract Audit #1

Open ai-protocol-dev opened 3 years ago

ai-protocol-dev commented 3 years ago

Description

Context

Alethea AI is building a protocol to create, share and monetize Synthetic Media. As originators of the iNFT standard, Alethea AI is embedding state-of-the-art large language models like OpenAI’s GPT-3 engine into popular NFT Standards, to create interactive and intelligent NFTs.

Recently, Alethea AI partnered with the Artist Robert Alice to launch the sale of the first iNFT. The first iNFT is a virtual character known as Alice who can engage, interact and respond in real-time to complex questions about the Metaverse. The iNFT sold through Sotheby's Natively Digital on June 10, 2021.

Bounty Description

We have finished coding the smart contract for our ERC-20. Before sending the contract to the leading auditing DAOs/firms, and to also start our Gitcoin journey, we are looking for people to review the smart contract and audit it. The smart contract brings in various functionalities from 0x, Compound, USDC - and is expansive in its scope.

Approved Devs will be paid the bounty for their contribution and help.

Please reach out to us at telegram for access to the private repo:

Telegram ID: @real_alethea

We Are Recruiting

We are actively recruiting and looking for expert smart contract developers to join our team. We will be regularly posting on Gitcoin and these bounties are a way for us to discover and connect with the talented developers on Gitcoin. If you are interested in joining us for the longer term, please do reach out to us for this bounty.

Learn More:

https://twitter.com/gdb/status/1403067980343242753?msID=6c2ed0c4-46b8-4491-9bd0-9230a8f20e5a

https://andrewsteinwold.substack.com/p/ai-nfts-what-is-an-inft-

https://alethea.ai/

https://twitter.com/real_alethea

gitcoinbot commented 3 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.125 ETH (252.25 USD @ $2017.97/ETH) attached to it.

gitcoinbot commented 3 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

Work has been started.

These users each claimed they can complete the work by 265 years, 4 months from now. Please review their action plans below:

1) icebert04 has been approved to start work.

Hi guys,

I see that you need help sending your smart contract to the leading auditing DAOs/firms.

I can help you by conducting an in-depth security assessment.

Feel free to check on my Github repo on projects that I made on the Blockchain. Link attached here... https://github.com/icebert04?tab=repositories

Message me anytime if you have more inquiries.

All the best, Francis 2) cxiane has applied to start work _(Funders only: approve worker | reject worker)_.

all know that ERC20 token has a huge amount of fees to deal with before doing some transactions, what can you do to solve this problem

Learn more on the Gitcoin Issue Details page.

gitcoinbot commented 3 years ago

@icebert04 Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

icebert04 commented 3 years ago

@gitcoinbot Thanks for the reminder. I am doing a Line-by-line review and will be submitting my WIP PR today.

gitcoinbot commented 3 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

Work for 0.125 ETH (269.9 USD @ $2249.17/ETH) has been submitted by:

icebert04 commented 3 years ago

Submitted work on
PR: https://github.com/icebert04/Gitcoin-Bounties/blob/main/bounty_1/contracts/AccessControl.sol

icebert04 commented 3 years ago

@satoshi505 Here is one of the low severity issues that I found...

Potential typo error on line 265 in this particular repo

https://github.com/AletheaAI/alethea-contracts/blob/master/contracts/IntelligentNFTv1.sol

function ownerOf(uint256 recordId) public view override returns (address) { // read the token binding IntelliBinding memory binding = bindings[recordId];

Inheriting from Ownable and calling its constructor onlyOwner modifier makes a function revert if not called by the address registered as owner.

Upon checking on https://docs.soliditylang.org/en/v0.8.6/search.html?q=ownerOf&check_keywords=yes&area=default

The modifier that should be used is onlyOwner rather than ownerOf.

gitcoinbot commented 3 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

The funding of 0.125 ETH (273.95 USD @ $2282.89/ETH) attached to this issue has been approved & issued to @icebert04.

gitcoinbot commented 3 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

The funding of 0.125 ETH (273.95 USD @ $2282.89/ETH) attached to this issue has been approved & issued to @icebert04.