Skip to content

Commit

Permalink
Merge branch 'main' into audit/L07
Browse files Browse the repository at this point in the history
  • Loading branch information
ly0va committed Feb 10, 2025
2 parents 8feebae + 9f2cd09 commit 5257462
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 19 deletions.
12 changes: 11 additions & 1 deletion src/SsoAccount.sol
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ import { SignatureDecoder } from "./libraries/SignatureDecoder.sol";
import { ERC1271Handler } from "./handlers/ERC1271Handler.sol";
import { BatchCaller } from "./batch/BatchCaller.sol";

import { BootloaderAuth } from "./auth/BootloaderAuth.sol";

import { ISsoAccount } from "./interfaces/ISsoAccount.sol";
import { IModuleValidator } from "./interfaces/IModuleValidator.sol";

Expand All @@ -32,7 +34,15 @@ import { IModuleValidator } from "./interfaces/IModuleValidator.sol";
/// @notice This contract is a modular and extensible account implementation with support of
/// multi-ownership, custom modules, validation/execution hooks and different signature validation formats.
/// @dev Contract is expected to be used as Beacon proxy implementation.
contract SsoAccount is Initializable, HookManager, ERC1271Handler, TokenCallbackHandler, BatchCaller, ISsoAccount {
contract SsoAccount is
Initializable,
HookManager,
ERC1271Handler,
TokenCallbackHandler,
BatchCaller,
ISsoAccount,
BootloaderAuth
{
// Helper library for the Transaction struct
using TransactionHelper for Transaction;

Expand Down
12 changes: 0 additions & 12 deletions src/auth/Auth.sol

This file was deleted.

4 changes: 2 additions & 2 deletions src/managers/HookManager.sol
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import { EnumerableSet } from "@openzeppelin/contracts/utils/structs/EnumerableS
import { Transaction } from "@matterlabs/zksync-contracts/l2/system-contracts/libraries/TransactionHelper.sol";
import { ExcessivelySafeCall } from "@nomad-xyz/excessively-safe-call/src/ExcessivelySafeCall.sol";

import { Auth } from "../auth/Auth.sol";
import { SelfAuth } from "../auth/SelfAuth.sol";
import { SsoStorage } from "../libraries/SsoStorage.sol";
import { Errors } from "../libraries/Errors.sol";
import { IExecutionHook, IValidationHook } from "../interfaces/IHook.sol";
Expand All @@ -19,7 +19,7 @@ import { IModule } from "../interfaces/IModule.sol";
* @dev Hook addresses are stored in a linked list
* @author https://getclave.io
*/
abstract contract HookManager is IHookManager, Auth {
abstract contract HookManager is IHookManager, SelfAuth {
using EnumerableSet for EnumerableSet.AddressSet;
// Interface helper library
using ERC165Checker for address;
Expand Down
5 changes: 3 additions & 2 deletions src/managers/OwnerManager.sol
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ pragma solidity ^0.8.24;

import { EnumerableSet } from "@openzeppelin/contracts/utils/structs/EnumerableSet.sol";
import { SsoStorage } from "../libraries/SsoStorage.sol";
import { Auth } from "../auth/Auth.sol";
import { Errors } from "../libraries/Errors.sol";
import { SelfAuth } from "../auth/SelfAuth.sol";
import { IOwnerManager } from "../interfaces/IOwnerManager.sol";

/**
Expand All @@ -13,7 +14,7 @@ import { IOwnerManager } from "../interfaces/IOwnerManager.sol";
* @dev Owners are stored in a linked list
* @author https://getclave.io
*/
abstract contract OwnerManager is IOwnerManager, Auth {
abstract contract OwnerManager is IOwnerManager, SelfAuth {
using EnumerableSet for EnumerableSet.AddressSet;

/// @inheritdoc IOwnerManager
Expand Down
4 changes: 2 additions & 2 deletions src/managers/ValidatorManager.sol
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import { ERC165Checker } from "@openzeppelin/contracts/utils/introspection/ERC16
import { EnumerableSet } from "@openzeppelin/contracts/utils/structs/EnumerableSet.sol";
import { ExcessivelySafeCall } from "@nomad-xyz/excessively-safe-call/src/ExcessivelySafeCall.sol";

import { Auth } from "../auth/Auth.sol";
import { SelfAuth } from "../auth/SelfAuth.sol";
import { Errors } from "../libraries/Errors.sol";
import { SsoStorage } from "../libraries/SsoStorage.sol";
import { IValidatorManager } from "../interfaces/IValidatorManager.sol";
Expand All @@ -18,7 +18,7 @@ import { IModule } from "../interfaces/IModule.sol";
* @dev Validators are stored in an enumerable set
* @author https://getclave.io
*/
abstract contract ValidatorManager is IValidatorManager, Auth {
abstract contract ValidatorManager is IValidatorManager, SelfAuth {
using EnumerableSet for EnumerableSet.AddressSet;
// Interface helper library
using ERC165Checker for address;
Expand Down

0 comments on commit 5257462

Please sign in to comment.