Re: [libvirt] [PATCH 1/2] util: Don't overflow in virRandomBits

2018-08-05 Thread Eric Blake
On 08/01/2018 07:16 AM, Daniel P. Berrangé wrote: On Wed, Aug 01, 2018 at 01:44:32PM +0200, Michal Privoznik wrote: The function is supposed to return up to 64bit long integer. In order to do that it calls virRandomBytes() to fill the integer with random bytes and then masks out everything but

Re: [libvirt] [PATCH 1/2] util: Don't overflow in virRandomBits

2018-08-02 Thread Michal Privoznik
On 08/01/2018 04:50 PM, Eric Blake wrote: > On 08/01/2018 07:16 AM, Daniel P. Berrangé wrote: >> On Wed, Aug 01, 2018 at 01:44:32PM +0200, Michal Privoznik wrote: >>> The function is supposed to return up to 64bit long integer. In >>> order to do that it calls virRandomBytes() to fill the integer

Re: [libvirt] [PATCH 1/2] util: Don't overflow in virRandomBits

2018-08-01 Thread Daniel P . Berrangé
On Wed, Aug 01, 2018 at 01:44:32PM +0200, Michal Privoznik wrote: > The function is supposed to return up to 64bit long integer. In > order to do that it calls virRandomBytes() to fill the integer > with random bytes and then masks out everything but requested > bits. However, when doing that it

[libvirt] [PATCH 1/2] util: Don't overflow in virRandomBits

2018-08-01 Thread Michal Privoznik
The function is supposed to return up to 64bit long integer. In order to do that it calls virRandomBytes() to fill the integer with random bytes and then masks out everything but requested bits. However, when doing that it shifts 1U and not 1ULL. So effectively, requesting 32 random bis or more