Roland McGrath <roland@hack.frob.com> 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