Code

Oom alloc#463

Code

Open

sftse wants to merge 18 commits intotafia:mastertafia/calamine:masterfrom sftse:oom-allocsftse/calamine:oom-allocCopy head branch name to clipboard

+130-94

Commits

Commits on Oct 30, 2024

• test: add high byte strings testcase

  sftse

  committed
  27bfae9

• fix: consume 2*cch bytes when string has high byte flag set

  Show description for 28d6e7c
  sftse

  committed
  28d6e7c

• fix: use utf16 LE to decode unicode strings instead of global xls encoding

  sftse

  committed
  44a05eb

• fix: accept XLUnicodeString without reserved tag

  sftse

  committed
  0576c6a

• test: add xls with oom allocations

  sftse

  committed
  ed0426a

• fix: tests/OOM_alloc.xls: sort cells by (row, col) to ensure row_end - row_start >= 0

  sftse

  committed
  a363ec7

• test: add xls with oom allocations

  sftse

  committed
  6514b1f

• fix: tests/OOM_alloc2.xls: if first column is too large, set to 0 to prevent OOM allocation due to overflow

  sftse

  committed
  d8edc9d

• test: expand test to check for missing text

  sftse

  committed
  23c3621

• fix: add record type 214 that comes out of nowhere and cannot be found in spec

  sftse

  committed
  245aa5f

• test: add xls with oom allocations

  sftse

  committed
  d55126c

• fix: clippy

  sftse

  committed
  c328139

• refactor: better pattern matching

  sftse

  committed
  018b2dd

• fmt

  sftse

  committed
  67d146b

• fix: length checks must cover biff != Biff2-Biff5 && r.len() == 2 case, otherwise this will crash

  sftse

  committed
  88ad585

• fix: remove let _ binding, which confuses into thinking error might be dropped

  sftse

  committed
  1a8568a

• fix: remove Option-wrapping of return type, is never None

  sftse

  committed
  fb267a3

• test: add future FIXME for unrecognized formula

  sftse

  committed
  67363c0