[Qemu-devel] Re: [PATCH 5/8] ide/pci: convert to qdev.

[Gerd Hoffmann]

  Hi,

> > Sure, you have to dereference the pointer.  I still don't see a
> > problem here.  You seem to think this is fragile.  Why do you think
> > so?
>
> Pointer not initialized, it is NULL, point nowhere, etc, etc.
> No pointer, you don't need to even think about all that kind of porblems/checks.

NULL can easily be checked for, and that should also catch the "not 
initialized" case.  Pointing into nowhere is a clear bug which needs 
fixing anyway.

> uint8_t *foo[4]
> is a different beast ... how do you declare a pointer to one array of 4
> uint8_t?

This isn't a pointer to an array, it is a array of pointers ...

cheers,
  Gerd