> 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



Reply via email to