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!
