Re: [libvirt] [PATCH 1/2] util: Fix VIR_ERR_ACCESS_DENIED formatting

2018-11-11 Thread Peter Krempa
On Fri, Nov 09, 2018 at 19:39:36 -0500, John Ferlan wrote: > In commit ccc72d5cb I thought I had the brilliant idea that I > could alter an error message in virErrorMsg to take two arguments > "drivername" and NULL. Then passing the failed drivername for > VIR_ERR_ACCESS_DENIED users would

[libvirt] [PATCH 1/2] util: Fix VIR_ERR_ACCESS_DENIED formatting

2018-11-09 Thread John Ferlan
In commit ccc72d5cb I thought I had the brilliant idea that I could alter an error message in virErrorMsg to take two arguments "drivername" and NULL. Then passing the failed drivername for VIR_ERR_ACCESS_DENIED users would magically allow any "real" error message to be included. Of course it