elric1/heimdal — comment on commit 83056ef81093189f57f57eef7aca563e77bb35cc
File: lib/kadm5/context_s.c line 189
By nicowilliams:

Indeed, it's not right. This function is clearly intended to be able to return not-zero to indicate errors. Its caller does not check the result of the call, but the caller's caller (and their callers...) do, so just make all three of these cases result in return ENOMEM and then make the call site for find_db_spec() return whatever find_db_spec() returns.