AZ
А как это решение поможет против вызова из конструктора, когда кода еще нет?
Size: a a a
2021 June 07
SM
Тут надо больше вводных данных. Если конструктор вызывается при деплое, то смысла в этой проверке нет. Если же это factory, то нужно видеть код
AZ
Был еще tx.origin == msg.sender, но это для проверки вызывающего
N
она как раз уязвима
𓄋
++
𓄋
Вроде видел в 8 версии
𓄋
Уже готовое решение
𓄋
Найду скину
N
можно пробовать проверять на наличие баланса
MD
а как она уязвима?
AZ
AZ
В треде код
AZ
Типа вот
pragma solidity 0.4.25;
contract Victim {
function isContract() public view returns(bool){
uint32 size;
address a = msg.sender;
assembly {
size := extcodesize(a)
}
return (size > 0);
}
}
contract Attacker {
bool public iTrickedIt;
Victim v;
constructor(address _v) public {
v = Victim(_v);
// addrss(this) doesn't have code, yet
iTrickedIt = !v.isContract();
}
}𓄋
Бдыыыщ
N
Ага и это логично
RP
Так вам что надо проверять на контракт, вызывающего или произвольный адрес?
Если и то и другое то поставьте оба способа
msg.sender == tx.origin
проверит для вызывающего
и следом extcodesize если что
Если и то и другое то поставьте оба способа
msg.sender == tx.origin
проверит для вызывающего
и следом extcodesize если что
MD
понятно, спасибо
𓄋
Топ
MD
ну да, если нулевой баланс и не контракт - идёшь нахер))