Re: [PATCH] virfile: Report error when changing pipe size fails

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 3/28/22 2:24 PM, Claudio Fontana wrote:
> On 3/28/22 2:22 PM, Daniel P. Berrangé wrote:
>> On Mon, Mar 28, 2022 at 01:46:08PM +0200, Michal Privoznik wrote:
>>> When changing the size of pipe that virFileWrapperFdNew() creates
>>> we start at 1MiB and if that fails because it's above the system
>>> wide limit we get EPERM and continue with half of the size.
>>>
>>> However, we might get another error in which case we should
>>> report proper system error and return failure from
>>> virFileWrapperFdNew().
>>>
>>> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
>>> ---
>>>  src/util/virfile.c | 18 +++++++++++-------
>>>  1 file changed, 11 insertions(+), 7 deletions(-)
>>
>> Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
>>
>>
>> With regards,
>> Daniel
>>
> 
> testing it right now, thanks!
> 

Tested-by: Claudio Fontana <cfontana@xxxxxxx>





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux