Re: [Qemu-devel] [PATCH v3 1/6] vmdk: Use g_random_int to generate CID

[Max Reitz <mreitz@redhat.com>]

On 2014-12-03 at 11:28, Fam Zheng wrote:
> This replaces two "time(NULL)" invocations with "g_random_int()".
> According to VMDK spec, CID "is a random 32‐bit value updated the first
> time the content of the virtual disk is modified after the virtual disk
> is opened". Using "seconds since epoch" is just a "lame way" to generate
> it, and not completely safe because of the low precision.
>
> Suggested-by: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Fam Zheng <famz@redhat.com>
> Reviewed-by: Markus Armbruster <armbru@redhat.com>
> ---
>   block/vmdk.c | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/block/vmdk.c b/block/vmdk.c
> index 2cbfd3e..ebb4b70 100644
> --- a/block/vmdk.c
> +++ b/block/vmdk.c
> @@ -28,6 +28,7 @@
>   #include "qemu/module.h"
>   #include "migration/migration.h"
>   #include <zlib.h>
> +#include <glib.h>

This should be covered by qemu-common.h already. But anyway:

Reviewed-by: Max Reitz <mreitz@redhat.com>