opcode `opGetByte` and `opSetByte`

[amityadav0]

Subject of the issue

Index check for the set byte should have >=.

In opGetByte

go-algorand/data/transactions/logic/eval.go

Line 2454 in 23bfcd7

if idx >= uint64(len(target.Bytes)) {

we are checking for equal condition because the indexing assumed here starts from 0.

but in case of opSetByte

go-algorand/data/transactions/logic/eval.go

Line 2471 in 23bfcd7

if cx.stack[prev].Uint > uint64(len(cx.stack[pprev].Bytes)) {

This one should be same as opGetByte

Please correct me if i am not understanding it right.

[jannotti]

Thank you. Addressing with #2236