diff options
| author | Roberto Ierusalimschy <roberto@inf.puc-rio.br> | 2021-09-22 13:10:39 -0300 |
|---|---|---|
| committer | Roberto Ierusalimschy <roberto@inf.puc-rio.br> | 2021-09-22 13:10:39 -0300 |
| commit | deac067ed39a44c001599c0d15de09872496b2aa (patch) | |
| tree | d7373651e7d54a8ca5ffa4841379a4d9149164aa /testes | |
| parent | 2ff34717227b8046b0fdcb96206f11f5e888664e (diff) | |
| download | lua-deac067ed39a44c001599c0d15de09872496b2aa.tar.gz lua-deac067ed39a44c001599c0d15de09872496b2aa.tar.bz2 lua-deac067ed39a44c001599c0d15de09872496b2aa.zip | |
Avoid overflows when incrementing parameters in C
Any C function can receive maxinteger as an integer argument, and
therefore cannot increment it without some care (e.g., doing unsigned
arithmetic as the core does).
Diffstat (limited to 'testes')
| -rw-r--r-- | testes/nextvar.lua | 17 | ||||
| -rw-r--r-- | testes/utf8.lua | 6 |
2 files changed, 23 insertions, 0 deletions
diff --git a/testes/nextvar.lua b/testes/nextvar.lua index 076f6361..9e23e572 100644 --- a/testes/nextvar.lua +++ b/testes/nextvar.lua | |||
| @@ -43,6 +43,14 @@ assert(i == 4) | |||
| 43 | assert(type(ipairs{}) == 'function' and ipairs{} == ipairs{}) | 43 | assert(type(ipairs{}) == 'function' and ipairs{} == ipairs{}) |
| 44 | 44 | ||
| 45 | 45 | ||
| 46 | do -- overflow (must wrap-around) | ||
| 47 | local f = ipairs{} | ||
| 48 | local k, v = f({[math.mininteger] = 10}, math.maxinteger) | ||
| 49 | assert(k == math.mininteger and v == 10) | ||
| 50 | k, v = f({[math.mininteger] = 10}, k) | ||
| 51 | assert(k == nil) | ||
| 52 | end | ||
| 53 | |||
| 46 | if not T then | 54 | if not T then |
| 47 | (Message or print) | 55 | (Message or print) |
| 48 | ('\n >>> testC not active: skipping tests for table sizes <<<\n') | 56 | ('\n >>> testC not active: skipping tests for table sizes <<<\n') |
| @@ -499,6 +507,15 @@ do -- testing table library with metamethods | |||
| 499 | end | 507 | end |
| 500 | 508 | ||
| 501 | 509 | ||
| 510 | do -- testing overflow in table.insert (must wrap-around) | ||
| 511 | |||
| 512 | local t = setmetatable({}, | ||
| 513 | {__len = function () return math.maxinteger end}) | ||
| 514 | table.insert(t, 20) | ||
| 515 | local k, v = next(t) | ||
| 516 | assert(k == math.mininteger and v == 20) | ||
| 517 | end | ||
| 518 | |||
| 502 | if not T then | 519 | if not T then |
| 503 | (Message or print) | 520 | (Message or print) |
| 504 | ('\n >>> testC not active: skipping tests for table library on non-tables <<<\n') | 521 | ('\n >>> testC not active: skipping tests for table library on non-tables <<<\n') |
diff --git a/testes/utf8.lua b/testes/utf8.lua index 6010d1ad..461e223c 100644 --- a/testes/utf8.lua +++ b/testes/utf8.lua | |||
| @@ -112,6 +112,12 @@ do | |||
| 112 | end | 112 | end |
| 113 | errorcodes("ab\xff") | 113 | errorcodes("ab\xff") |
| 114 | errorcodes("\u{110000}") | 114 | errorcodes("\u{110000}") |
| 115 | |||
| 116 | -- calling interation function with invalid arguments | ||
| 117 | local f = utf8.codes("") | ||
| 118 | assert(f("", 2) == nil) | ||
| 119 | assert(f("", -1) == nil) | ||
| 120 | assert(f("", math.mininteger) == nil) | ||
| 115 | end | 121 | end |
| 116 | 122 | ||
| 117 | -- error in initial position for offset | 123 | -- error in initial position for offset |
