blip icon indicating copy to clipboard operation
blip copied to clipboard

Crash on index3d_is_empty with NULL argument

Open XLegland opened this issue 3 years ago • 2 comments

index3d_is_empty(NULL) leads to a crash. We should return if the argument is NULL to avoid crashing

XLegland avatar Oct 21 '22 13:10 XLegland

@XLegland I'm not sure about this. So you would return true? Suggesting that the index is empty? (while in fact NULL) This could lead devs to other errors, not necessarily easier to debug. We could simply add an assert in that function.

aduermael avatar Oct 30 '22 18:10 aduermael

let's use an assert for now

gdevillele avatar Apr 24 '23 08:04 gdevillele