Page MenuHomePhabricator

efl: `eina_value_list_vinsert` assumes that the position is always valid

Authored by lucas on Feb 7 2020, 10:07 AM.



The eina_value_list_vinsert at src/lib/eina_inline_value.x was not checking if the desired position was valid:
When inserting in an empty list in any out of bounds position it actually createded a list with the head being the desired value.
When inserting in a non-empty list in an out of bounds position caused a c error.
Now both cases return EINA_FALSE

Ref T8611

Test Plan

Meson configured with -Dbindings=mono,cxx -Dmono-beta=true, and tests runned
with ninja test all.

Diff Detail

rEFL core/efl
No Linters Available
No Unit Test Coverage
Build Status
Buildable 15931
Build 10774: arc lint + arc unit
lucas created this revision.Feb 7 2020, 10:07 AM
lucas requested review of this revision.Feb 7 2020, 10:07 AM
zmike accepted this revision.Feb 7 2020, 10:14 AM
This revision is now accepted and ready to land.Feb 7 2020, 10:14 AM
This revision was automatically updated to reflect the committed changes.