mirrors/bitcoin-s
1
0
Fork 0
mirror of https://github.com/bitcoin-s/bitcoin-s.git synced 2025-03-13 11:35:40 +01:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #165 from Christewart/script_parse_bug

Browse source 
overflow bug fix didn't quite account for every overflow possibility
This commit is contained in:
Chris Stewart 2018-05-29 13:36:09 -04:00 • committed by GitHub
commit bd7f66dc7b
1 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
core/src/main/scala/org/bitcoins/core/serializers/script/ScriptParser.scala
View file

@ -219,12 +219,11 @@ sealed abstract class ScriptParser extends Factory[List[ScriptToken]] {
val uInt32Push = UInt32(BitcoinSUtil.flipEndianness(scriptConstantHex))
//need this for the case where we have an OP_PUSHDATA4 with a number larger than a int32 can hold
//TODO: Review this more, see this transaction's scriptSig as an example: b30d3148927f620f5b1228ba941c211fdabdae75d0ba0b688a58accbf018f3cc
val bytesForPushOp = Try(uInt32Push.toInt).getOrElse(Int.MaxValue)
val bytesForPushOp = Try(uInt32Push.toInt).getOrElse(tail.size)
val bytesToPushOntoStack = ScriptConstant(scriptConstantHex)
val endIndex = {
val idx = bytesForPushOp + numBytes
//for the case of an overflow. Just assume Int.MaxValue for slice operation which only takes a Int
if (idx >= 0) idx else Int.MaxValue
if (idx >= numBytes) idx else tail.size
}
val scriptConstantBytes = tail.slice(numBytes, endIndex)
val scriptConstant = ScriptConstant(scriptConstantBytes)