Skip to content

Commit

Permalink
Fix stack bounds check in AssignmentEvaluator
Browse files Browse the repository at this point in the history
  • Loading branch information
ethanmoffat committed Jan 29, 2022
1 parent 8d7f154 commit fb99073
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions EOBot/Interpreter/States/AssignmentEvaluator.cs
Original file line number Diff line number Diff line change
Expand Up @@ -22,18 +22,18 @@ public bool Evaluate(ProgramState input)
return false;
}

var expressionResult = (VariableBotToken)input.OperationStack.Pop();
if (input.OperationStack.Count == 0)
return false;
var expressionResult = (VariableBotToken)input.OperationStack.Pop();

// todo: check that assignOp is an assignment operator
var assignOp = input.OperationStack.Pop();
if (input.OperationStack.Count == 0)
return false;
var assignOp = input.OperationStack.Pop();

var variable = (IdentifierBotToken)input.OperationStack.Pop();
if (input.OperationStack.Count == 0)
return false;
var variable = (IdentifierBotToken)input.OperationStack.Pop();

if (variable.ArrayIndex != null)
{
Expand Down

0 comments on commit fb99073

Please sign in to comment.