Re: [PATCH] virfile: Report error when changing pipe size fails
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(a)redhat.com>
> ---
> src/util/virfile.c | 18 +++++++++++-------
> 1 file changed, 11 insertions(+), 7 deletions(-)
Reviewed-by: Daniel P. Berrangé <berrange(a)redhat.com>
With regards,
Daniel
testing it right now, thanks!