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>