On 11/25/2015 04:45 AM, Ben Gamari wrote:
This could have easily saved me three hours.
libebl/eblopenbackend.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libebl/eblopenbackend.c b/libebl/eblopenbackend.c index b301400..662842d 100644 --- a/libebl/eblopenbackend.c +++ b/libebl/eblopenbackend.c @@ -376,7 +376,8 @@ openbackend (Elf *elf, const char *emulation, GElf_Half machine) result->dlhandle = NULL; result->elf = elf; result->name = machines[cnt].prefix;
- fill_defaults (result);
fill_defaults (result);
fprintf(stderr, "ebl_openbackend: Failed to find backend DSO to handle machine.\n");
return result; }
It's not really kosher for a library to dump on stderr. No other part of elfutils does this AFAICS. Maybe some other location needs to return better errors from not having the ebl->dlhandle?
(Also, watch your tab changes.)