Re: [libvirt] [PATCH 5/6] Fix error handling in virSecurityManagerGetMountOptions

2012-11-25 Thread Laine Stump
On 11/22/2012 11:48 AM, Daniel P. Berrange wrote: > From: "Daniel P. Berrange" > > The impls of virSecurityManagerGetMountOptions had no way to > return errors, since the code was treating 'NULL' as a success > value. This is somewhat pointless, since the calling code did > not want NULL in the fi

[libvirt] [PATCH 5/6] Fix error handling in virSecurityManagerGetMountOptions

2012-11-22 Thread Daniel P. Berrange
From: "Daniel P. Berrange" The impls of virSecurityManagerGetMountOptions had no way to return errors, since the code was treating 'NULL' as a success value. This is somewhat pointless, since the calling code did not want NULL in the first place and has to translate it into the empty string "". S