sherlock-audit / 2024-02-jala-swap-judging

6 stars 4 forks source link

0x77 - Missing constructor making contract as an abstract #218

Closed sherlock-admin closed 6 months ago

sherlock-admin commented 6 months ago

0x77

medium

Missing constructor making contract as an abstract

Summary

The contract "ChilizWrapperFactory.sol" is missing a constructor

Vulnerability Detail

a missing constructor in the utils/ChilizWrapperFactory.sol making the contract not compiled. Even if we change the version to any version of compiler, it will impose other errors so there is a huge inconsistency in solidity version selection.

Impact

the contract is basically responsible for wrapping, unwrapping a tokens. If it wont work , all the functions regarding this wont obviously.

Code Snippet

missing constructor https://github.com/sherlock-audit/2024-02-jala-swap/blob/main/jalaswap-dex-contract/contracts/utils/ChilizWrapperFactory.sol?plain=1#L14

Tool used

Manual Review Remix slither

Recommendation

add constructor to the contract

   constructor() Ownable(msg.sender) {}

Duplicate of #251