1. 07 Feb, 2017 7 commits
  2. 06 Feb, 2017 3 commits
  3. 05 Feb, 2017 5 commits
  4. 04 Feb, 2017 1 commit
    • Nikias Bassen's avatar
      bplist: Fix OOB write on heap buffer and improve recursion check · c4dcf11b
      Nikias Bassen authored
      Issue #92 pointed out an problem with (invalid) bplist files which have
      exactly one structured node whose subnode reference itself.
      The recursion check used a fixed size array with the size of the total number
      of objects. In this case the number of objects is 1 but the recursion check
      code wanted to set the node_index for the level 1 which leads to an OOB write
      on the heap. This commit fixes/improves two things:
      1) Prevent OOB write by using a dynamic data storage for the used node
         indexes (plist_t of type PLIST_ARRAY)
      2) Reduces the memory usage of large binary plists, because not the total
         number of nodes in the binary plist, but the number of recursion levels
         is important for the recursion check.
      c4dcf11b
  5. 03 Feb, 2017 2 commits
  6. 01 Feb, 2017 3 commits
  7. 28 Jan, 2017 9 commits
  8. 25 Jan, 2017 2 commits
  9. 19 Jan, 2017 4 commits
  10. 18 Jan, 2017 3 commits
  11. 16 Jan, 2017 1 commit
    • Nikias Bassen's avatar
      bplist: Disallow key nodes with non-string node types · 7a28a14c
      Nikias Bassen authored
      As reported in #86, the binary plist parser would force the type of the
      key node to be of type PLIST_KEY while the node might be of a different
      i.e. non-string type. A following plist_free() might then call free() on
      an invalid pointer; e.g. if the node is of type integer, its value would
      be considered a pointer, and free() would cause an error.
      We prevent this issue by disallowing non-string key nodes during parsing.
      7a28a14c