[libvirt] [PATCH] virFileInData: Report an error if unable to reposition file
Michal Privoznik
mprivozn at redhat.com
Wed Jul 19 07:52:31 UTC 2017
On 07/18/2017 11:04 PM, John Ferlan wrote:
>
>
> On 07/11/2017 05:05 AM, Michal Privoznik wrote:
>> The purpose of this function is to tell if the current position
>> in given FD is in data section or a hole and how much bytes there
>> is remaining until the end of the section. This is achieved by
>> couple of lseeks(). The most important part is that we reposition
>> the FD back, so that the position is unchanged from the caller
>> POV. And until now the final lseek() back to the original
>> position was done with no check for errors. And I was convinced
>> that that's okay since nothing can go wrong. However, John
>
> Good thing there's only one John ;-) Perhaps better said, "review
> feedback from a related series persuaded me..."
>
>> persuaded me, that it's better to be safe than sorry. Therefore,
>> lets check if the final lseek() succeeded and if it doesn't
>> report an error.
>>
>> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>> ---
>> src/util/virfile.c | 8 ++++++--
>> 1 file changed, 6 insertions(+), 2 deletions(-)
>>
>
> Reviewed-by: John Ferlan <jferlan at redhat.com>
>
> John
>
Fixed and pushed. Thank you.
Michal
More information about the libvir-list
mailing list