> On 5 Jan 2025, at 00:29, Ranier Vilela <ranier...@gmail.com> wrote: > > Hi. > > Per Coverity. > > All call sites of function *get_cheapest_path_for_pathkeys* checks > for NULL returns. > > So, it is highly likely that the function will return NULL. > > IMO, the Assert in this particular call, is not fully effective. > > Fix removing the Assert and always check if the return is NULL.
Yet the author wrote an Assert here (over a decade ago), so rather than blindly changing that it seems reasonable to motivate a patch like this with an investigation on what the Assert means here. The fact that Coverity complains is far from conclusive evidence that something is wrong. -- Daniel Gustafsson