Roland McGrath <[email protected]> writes: >> if (unlikely (nread == -1)) >> - /* We cannot even read the head of the file. Maybe FILDES is associated >> - with an unseekable device. This is nothing we can handle. */ >> - return NULL; >> + { >> + __libelf_seterrno (ELF_E_INVALID_FILE); >> + /* We cannot even read the head of the file. Maybe FILDES is >> associated >> + with an unseekable device. This is nothing we can handle. */ >> + return NULL; >> + } > > That looks sensible, though I'd keep the comment at the top of the block.
Pushed with that fix. Thanks, PM
