swim: fix crash on bad member_by_uuid() call
In Lua swim object's method member_by_uuid() could crash if called with no arguments. UUID was then passed as NULL, and dereferenced. The patch makes member_by_uuid() treat NULL like nil UUID and return NULL (member not found). The reason is that swim_member_by_uuid() can't fail. It can only return a member or not. It never sets a diag error. Closes #5951
Showing
- changelogs/unreleased/swim-member_by_uuid-crash.md 4 additions, 0 deletionschangelogs/unreleased/swim-member_by_uuid-crash.md
- src/lib/swim/swim.c 2 additions, 0 deletionssrc/lib/swim/swim.c
- test/swim/swim.result 9 additions, 0 deletionstest/swim/swim.result
- test/swim/swim.test.lua 3 additions, 0 deletionstest/swim/swim.test.lua
- test/unit/swim.c 22 additions, 1 deletiontest/unit/swim.c
- test/unit/swim.result 8 additions, 1 deletiontest/unit/swim.result
Loading
Please register or sign in to comment