[riot-commits] [RIOT-OS/RIOT] 767e70: clif: After incrementing pos, make sure it is stil...

Leandro Lanzieri noreply at github.com
Wed Feb 10 08:50:30 CET 2021


  Branch: refs/heads/master
  Home:   https://github.com/RIOT-OS/RIOT
  Commit: 767e700a5f056626cb7156d4944130104893d631
      https://github.com/RIOT-OS/RIOT/commit/767e700a5f056626cb7156d4944130104893d631
  Author: Sören Tempel <tempel at uni-bremen.de>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M sys/clif/clif.c
    M tests/unittests/tests-clif/tests-clif.c

  Log Message:
  -----------
  clif: After incrementing pos, make sure it is still in bounds

While the for-loop condition does contain a bounds check, the pointer is
independently increment in the for-loop body. This increment therefore
requires a separate bounds check. Otherwise, the parsing loop may access
data outside the given buffer boundaries.


  Commit: 609c9ada34da5546cffb632a98b7ba157c112658
      https://github.com/RIOT-OS/RIOT/commit/609c9ada34da5546cffb632a98b7ba157c112658
  Author: Leandro Lanzieri <leandro.lanzieri at haw-hamburg.de>
  Date:   2021-02-10 (Wed, 10 Feb 2021)

  Changed paths:
    M sys/clif/clif.c
    M tests/unittests/tests-clif/tests-clif.c

  Log Message:
  -----------
  Merge pull request #15945 from nmeum/pr/clif_get_attr_bounds

clif: After incrementing pos, make sure it is still in bounds


Compare: https://github.com/RIOT-OS/RIOT/compare/1db480006c2d...609c9ada34da


More information about the commits mailing list