Index: sandbox/linux/seccomp-bpf/errorcode.cc |
diff --git a/sandbox/linux/seccomp-bpf/errorcode.cc b/sandbox/linux/seccomp-bpf/errorcode.cc |
index 5a45e4c8cf392d71c55d9f4b04ecb31f51564fda..3bc0d95d2faa422b044065636ac1440438b69ffd 100644 |
--- a/sandbox/linux/seccomp-bpf/errorcode.cc |
+++ b/sandbox/linux/seccomp-bpf/errorcode.cc |
@@ -36,21 +36,18 @@ ErrorCode::ErrorCode(Trap::TrapFnc fnc, const void* aux, bool safe, uint16_t id) |
ErrorCode::ErrorCode(int argno, |
ArgType width, |
- Operation op, |
+ uint64_t mask, |
uint64_t value, |
const ErrorCode* passed, |
const ErrorCode* failed) |
: error_type_(ET_COND), |
+ mask_(mask), |
value_(value), |
argno_(argno), |
width_(width), |
- op_(op), |
passed_(passed), |
failed_(failed), |
err_(SECCOMP_RET_INVALID) { |
- if (op < 0 || op >= OP_NUM_OPS) { |
- SANDBOX_DIE("Invalid opcode in BPF sandbox rules"); |
- } |
} |
bool ErrorCode::Equals(const ErrorCode& err) const { |
@@ -63,9 +60,9 @@ bool ErrorCode::Equals(const ErrorCode& err) const { |
if (error_type_ == ET_SIMPLE || error_type_ == ET_TRAP) { |
return err_ == err.err_; |
} else if (error_type_ == ET_COND) { |
- return value_ == err.value_ && argno_ == err.argno_ && |
- width_ == err.width_ && op_ == err.op_ && |
- passed_->Equals(*err.passed_) && failed_->Equals(*err.failed_); |
+ return mask_ == err.mask_ && value_ == err.value_ && argno_ == err.argno_ && |
+ width_ == err.width_ && passed_->Equals(*err.passed_) && |
+ failed_->Equals(*err.failed_); |
} else { |
SANDBOX_DIE("Corrupted ErrorCode"); |
} |
@@ -85,14 +82,14 @@ bool ErrorCode::LessThan(const ErrorCode& err) const { |
if (error_type_ == ET_SIMPLE || error_type_ == ET_TRAP) { |
return err_ < err.err_; |
} else if (error_type_ == ET_COND) { |
- if (value_ != err.value_) { |
+ if (mask_ != err.mask_) { |
+ return mask_ < err.mask_; |
+ } else if (value_ != err.value_) { |
return value_ < err.value_; |
} else if (argno_ != err.argno_) { |
return argno_ < err.argno_; |
} else if (width_ != err.width_) { |
return width_ < err.width_; |
- } else if (op_ != err.op_) { |
- return op_ < err.op_; |
} else if (!passed_->Equals(*err.passed_)) { |
return passed_->LessThan(*err.passed_); |
} else if (!failed_->Equals(*err.failed_)) { |