認可用modifierの欠落

可視性修飾子と認可用`modifier`は別物

Solidityには、public、external、internal、private という可視性修飾子がある。

これらは、「その関数をどこから呼び出せるか」を決めるものである。

function hoge() public {}
function hoge() external {}
function hoge() internal {}
function hoge() private {}

たとえば、external は、コントラクトの外部から呼び出せるという意味である。しかし、external は「誰に実行を許可するか」までは決めない。

function transferOwnership(address newOwner) external {
    owner = newOwner;
}

この関数は、外部から呼び出せる。しかし、外部から呼び出してきた相手が、現在の owner かどうかは確認していない。

つまり、external は「外部から呼べる入口を作る」指定であり、「その入口を誰に使わせるか」を決める認可チェックではない。

一方、onlyOwner や onlyRole のようなものは、関数の実行前に認可を行うための modifier である。

認可とは、「その呼び出し元に、その操作を実行する権限があるか」を確認することである。

たとえば、次のような確認が認可チェックである。

require(msg.sender == owner, "Not owner");

これは、「この関数を直接呼び出してきた msg.sender が、現在の owner か」を確認している。この確認を modifier としてまとめると、次のようになる。

modifier onlyOwner() {
    require(msg.sender == owner, "Not owner");
    _;
}

_ は、modifierが付いた関数本体を実行する位置を表す。つまり、上の onlyOwner は次の順序で処理される。

この onlyOwner を関数に付けると、関数本体の実行前に認可チェックが行われる。

function transferOwnership(address newOwner) external onlyOwner {
    owner = newOwner;
}

この場合、意味は次のようになる。

したがって、可視性修飾子と認可用modifierは役割が違う。

public / external / internal / private
  → 可視性修飾子
  → どこから呼び出せるか

onlyOwner / onlyRole
  → 認可用modifier
  → 誰に実行を認可するか

本記事で扱うのは、external のような可視性修飾子の欠落ではない。onlyOwner や onlyRole のような、認可用modifierの欠落である。

スマートコントラクトでは、管理者だけが実行できるべき関数に、適切な認可用modifierを付ける必要がある。

たとえば、次のような関数である。

function transferOwnership(address newOwner) ex